Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix various spelling mistakes and typos as found using codespell #96

Closed
wants to merge 2 commits into from
Closed

Fix various spelling mistakes and typos as found using codespell #96

wants to merge 2 commits into from

Conversation

ColinIanKing
Copy link
Contributor

There are a bunch of spelling mistakes in the json and csv data as well as a variable spelling mistake in scripts/create_perf_json.py. Fix these.

@ColinIanKing
Fix various spelling mistakes and typos as found using codespell
677dc96 
There are a bunch of spelling mistakes in the json and csv data as
well as a variable spelling mistake in scripts/create_perf_json.py.
Fix these.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
@edwarddavidbaker
Copy link
Contributor

Hi Colin,

Thanks for taking the time to open a pull request.

  • Event files
    • We will be pushing another round of updates which should pickup your contributions.
  • Atom_TMA.csv and E-core_TMA_Metrics.csv
  • README.md and create_perf_json.py
    • Looks good to me. Can you submit a separate PR so that they aren't blocked by event and TMA updates?

ZhangJing-hub pushed a commit to ZhangJing-hub/linux that referenced this pull request Sep 8, 2023
@captain5050 @acmel
perf vendor events intel: Fix spelling mistakes
03dfa45 
Update perf JSON files with spelling fixes by Colin Ian King
<colin.i.king@gmail.com> contributed in:

intel/perfmon#96 "Fix various spelling mistakes and typos as found using codespell torvalds#96"

This is added on top of the spelling mistakes and release number
updates in:

  intel/perfmon#98 "EMR, SPR, CLX, SKX, BDX, HSX, BDW-DE, WSM-EP*, NHM-*, JKT, IVT : Release event updates"

Some additional spelling fixes reported by Edward Baker
<edward.baker@intel.com> are added on top of this.

Signed-off-by: Ian Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Colin Ian King <colin.i.king@gmail.com>
Cc: Edward Baker <edward.baker@intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Xing Zhengjun <zhengjun.xing@linux.intel.com>
Link: https://lore.kernel.org/r/20230829001730.1352769-3-irogers@google.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
@edwarddavidbaker edwarddavidbaker mentioned this pull request Sep 11, 2023
Open
intel-lab-lkp pushed a commit to intel-lab-lkp/linux that referenced this pull request Sep 14, 2023
@captain5050 @acmel
perf vendor events intel: Fix spelling mistakes
b333067 
Update perf JSON files with spelling fixes by Colin Ian King
<colin.i.king@gmail.com> contributed in:

intel/perfmon#96 "Fix various spelling mistakes and typos as found using codespell torvalds#96"

This is added on top of the spelling mistakes and release number
updates in:

  intel/perfmon#98 "EMR, SPR, CLX, SKX, BDX, HSX, BDW-DE, WSM-EP*, NHM-*, JKT, IVT : Release event updates"

Some additional spelling fixes reported by Edward Baker
<edward.baker@intel.com> are added on top of this.

Signed-off-by: Ian Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Colin Ian King <colin.i.king@gmail.com>
Cc: Edward Baker <edward.baker@intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Xing Zhengjun <zhengjun.xing@linux.intel.com>
Link: https://lore.kernel.org/r/20230829001730.1352769-3-irogers@google.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
@rpmclaug rpmclaug mentioned this pull request Sep 14, 2023
Merged
edwarddavidbaker added a commit to edwarddavidbaker/perfmon that referenced this pull request Sep 22, 2023
@edwarddavidbaker @ColinIanKing
readme: Fix Perforamnce
08d6039 
Incorporate spelling suggestion from
intel#96.

Co-authored-by: Colin Ian King <colin.i.king@gmail.com>
edwarddavidbaker added a commit to edwarddavidbaker/perfmon that referenced this pull request Sep 22, 2023
@edwarddavidbaker @ColinIanKing
readme: Fix Perforamnce
f35f259 
Incorporate spelling suggestion from
intel#96.

Co-authored-by: Colin Ian King <colin.i.king@gmail.com>
edwarddavidbaker added a commit to edwarddavidbaker/perfmon that referenced this pull request Sep 22, 2023
@edwarddavidbaker @ColinIanKing
readme: Fix Perforamnce
f3ffdb2 
Incorporate spelling suggestion from
intel#96.

Co-authored-by: Colin Ian King <colin.i.king@gmail.com>
@ColinIanKing
Fix even spelling mistakes found using the spell tool
37fcb3c 
More spelling mistakes found using the spell tool on json data. Fix these.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
edwarddavidbaker added a commit to edwarddavidbaker/perfmon that referenced this pull request Oct 4, 2023
@edwarddavidbaker @ColinIanKing
readme: Fix Perforamnce
88671f2 
Incorporate spelling suggestion from
intel#96.

Co-authored-by: Colin Ian King <colin.i.king@gmail.com>
@edwarddavidbaker
Copy link
Contributor

Thanks again for opening this pull request. Closing since we are tracking spelling issues in #100 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@captain5050 captain5050 Awaiting requested review from captain5050 captain5050 is a code owner

@kliang2 kliang2 Awaiting requested review from kliang2 kliang2 is a code owner

@edwarddavidbaker edwarddavidbaker Awaiting requested review from edwarddavidbaker edwarddavidbaker is a code owner

@kshiprab kshiprab Awaiting requested review from kshiprab kshiprab is a code owner

@claudiaromo claudiaromo Awaiting requested review from claudiaromo claudiaromo is a code owner

@rpmclaug rpmclaug Awaiting requested review from rpmclaug rpmclaug is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ColinIanKing @edwarddavidbaker