MIME-Version: 1.0
References: <20230121184648.38487-1-didi.debian@cknow.org> <CAP-5=fX1-3ZnXk8G2fB0q9F3MjjA+zWWm1LbuYGc5nL5v3bXYg@mail.gmail.com>
 <3632669.kckeHSHuci@prancing-pony> <8858063.18A79QhPUa@prancing-pony>
In-Reply-To: <8858063.18A79QhPUa@prancing-pony>
From:   Ian Rogers <irogers@google.com>
Date:   Mon, 23 Jan 2023 09:24:08 -0800
Message-ID: <CAP-5=fV21x38Ne8MOhRCxV2zO-GhFvg7ZVw4sAfYr8RxveNogQ@mail.gmail.com>
Subject: Re: [PATCH] tools/perf: Various spelling fixes
To:     Diederik de Haas <didi.debian@cknow.org>
Cc:     Edward <edward.baker@intel.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        "open list:PERFORMANCE EVENTS SUBSYSTEM" 
        <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Sun, Jan 22, 2023 at 4:37 AM Diederik de Haas <didi.debian@cknow.org> wrote:
>
> Hi Ian,
>
> On Saturday, 21 January 2023 21:53:51 CET Diederik de Haas wrote:
> > On Saturday, 21 January 2023 20:25:46 CET Ian Rogers wrote:
> > > The JSON files are generated from Intel information here:
> > > https://github.com/intel/perfmon/
> > > Fixing the spelling in the tree means that when we update the
> > > incorrect spelling will be generated again. Intel are looking at the
> > > matter, I've added Edward Baker to the e-mail. To make it clearer,
> > > could you separate the JSON fixes from the tool fixes?
> >
> > Yes, I'll send a v2 patch with the JSON files excluded.
> > And I'll send a PR to the GH repo.
>
> I just did send v2 of the patch with the following command:
> git send-email --annotate -v2 HEAD~1 --to-cmd=scripts/get_maintainer.pl
>
> But that no longer included you(r email)*, so here's the link:
> https://lore.kernel.org/lkml/20230122122034.48020-1-didi.debian@cknow.org/
>
> The PR on GH is here: https://github.com/intel/perfmon/pull/44
>
> *) Still need to figure out how to properly use `git send-email` as I find the
> separation of the email contents and then only subsequently and separately
> seeing the To/CC list rather inconvenient.
> Guess I'll just use --to=/--cc= manually for now.
>
> Cheers,
>   Diederik

Thanks Diederik! There are some updates from Intel's perfmon
repository that we should pull to the perf tool pre 6.3. I believe
there is an update pending for typo fixes and hopefully these can
include yours. I'll try to keep an eye on things and send them to LKML
unless somebody beats me. Thanks Arnaldo for pulling the other change.

Ian