linux-perf-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] tools/perf: Various spelling fixes
       [not found] ` <CAP-5=fX1-3ZnXk8G2fB0q9F3MjjA+zWWm1LbuYGc5nL5v3bXYg@mail.gmail.com>
@ 2023-01-21 20:53   ` Diederik de Haas
  0 siblings, 0 replies; only message in thread
From: Diederik de Haas @ 2023-01-21 20:53 UTC (permalink / raw)
  To: Edward, Ian Rogers
  Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
	Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim,
	Maxime Coquelin, Alexandre Torgue, Kan Liang, Zhengjun Xing,
	Sandipan Das, open list:PERFORMANCE EVENTS SUBSYSTEM,
	open list:PERFORMANCE EVENTS SUBSYSTEM,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

[-- Attachment #1: Type: text/plain, Size: 1194 bytes --]

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.

> Could you also add the mistakes found by this list to:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/scri
> pts/spelling.txt which will mean the checkpatch.pl tool will flag the
> mistakes should regressions happen.

Ha! That's based on Debian's lintian tool (too) :-D
I'll make/send a separate patch to include the ones lintian found.
Lintian found several more, but I filtered the list I send to belong to `tools/
perf` hoping/assuming the To list wouldn't be so long (which didn't work out 
as well as I hoped ;-)).

> Beside those 2 suggestions, the corrections look great and thank you
> for taking the time to do this.

Cool :-)

Cheers,
  Diederik

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-01-21 20:54 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20230121184648.38487-1-didi.debian@cknow.org>
     [not found] ` <CAP-5=fX1-3ZnXk8G2fB0q9F3MjjA+zWWm1LbuYGc5nL5v3bXYg@mail.gmail.com>
2023-01-21 20:53   ` [PATCH] tools/perf: Various spelling fixes Diederik de Haas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).