* [PATCH] scripts/spelling.txt: remove 'thead' as a typo
@ 2023-07-23 1:03 Drew Fustini
2023-07-23 14:50 ` Guo Ren
0 siblings, 1 reply; 2+ messages in thread
From: Drew Fustini @ 2023-07-23 1:03 UTC (permalink / raw)
To: Andrew Morton, Randy Dunlap, Colin Ian King, Diederik de Haas,
Ian Rogers, SeongJae Park, Drew Fustini, Luca Ceresoli,
linux-kernel
Cc: Conor Dooley, Guo Ren, Jisheng Zhang
T-Head is a vendor of processor core IP, and they have recently
introduced the RISC-V TH1520 SoC. Remove 'thead' as a typo of 'thread'
to avoid checkpatch incorrectly warning that 'thead' is typo in patches
that add support for T-Head designs in the kernel.
Cc: Conor Dooley <conor@kernel.org>
Cc: Guo Ren <guoren@kernel.org>
Cc: Jisheng Zhang <jszhang@kernel.org>
Link: https://www.t-head.cn/
Signed-off-by: Drew Fustini <dfustini@baylibre.com>
---
scripts/spelling.txt | 1 -
1 file changed, 1 deletion(-)
For example, checkpatch prints the following warnings for 89b0186ab532
("dt-bindings: riscv: Add T-HEAD TH1520 board compatibles"):
$ ./scripts/checkpatch.pl --strict -g 89b0186ab532
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
new file mode 100644
WARNING: 'thead' may be misspelled - perhaps 'thread'?
+$id: http://devicetree.org/schemas/riscv/thead.yaml#
^^^^^
WARNING: 'thead' may be misspelled - perhaps 'thread'?
+ - const: thead,th1520
^^^^^
total: 0 errors, 3 warnings, 0 checks, 29 lines checked
The 'thead' typo false positive adds noise to automation like the
linux-riscv patchwork bot:
https://patchwork.kernel.org/project/linux-riscv/patch/20230722-upstream-beaglev-ahead-dts-v1-2-ccda511357f4@baylibre.com/
diff --git a/scripts/spelling.txt b/scripts/spelling.txt
index fc7ba95e86a0..855c4863124b 100644
--- a/scripts/spelling.txt
+++ b/scripts/spelling.txt
@@ -1541,7 +1541,6 @@ temeprature||temperature
temorary||temporary
temproarily||temporarily
temperture||temperature
-thead||thread
theads||threads
therfore||therefore
thier||their
--
2.34.1
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] scripts/spelling.txt: remove 'thead' as a typo
2023-07-23 1:03 [PATCH] scripts/spelling.txt: remove 'thead' as a typo Drew Fustini
@ 2023-07-23 14:50 ` Guo Ren
0 siblings, 0 replies; 2+ messages in thread
From: Guo Ren @ 2023-07-23 14:50 UTC (permalink / raw)
To: Drew Fustini
Cc: Andrew Morton, Randy Dunlap, Colin Ian King, Diederik de Haas,
Ian Rogers, SeongJae Park, Luca Ceresoli, linux-kernel,
Conor Dooley, Jisheng Zhang
Good catch, thank you very much.
Acked-by: Guo Ren <guoren@kernel.org>
On Sun, Jul 23, 2023 at 9:04 AM Drew Fustini <dfustini@baylibre.com> wrote:
>
> T-Head is a vendor of processor core IP, and they have recently
> introduced the RISC-V TH1520 SoC. Remove 'thead' as a typo of 'thread'
> to avoid checkpatch incorrectly warning that 'thead' is typo in patches
> that add support for T-Head designs in the kernel.
>
> Cc: Conor Dooley <conor@kernel.org>
> Cc: Guo Ren <guoren@kernel.org>
> Cc: Jisheng Zhang <jszhang@kernel.org>
> Link: https://www.t-head.cn/
> Signed-off-by: Drew Fustini <dfustini@baylibre.com>
> ---
> scripts/spelling.txt | 1 -
> 1 file changed, 1 deletion(-)
>
> For example, checkpatch prints the following warnings for 89b0186ab532
> ("dt-bindings: riscv: Add T-HEAD TH1520 board compatibles"):
>
> $ ./scripts/checkpatch.pl --strict -g 89b0186ab532
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> new file mode 100644
>
> WARNING: 'thead' may be misspelled - perhaps 'thread'?
> +$id: http://devicetree.org/schemas/riscv/thead.yaml#
> ^^^^^
>
> WARNING: 'thead' may be misspelled - perhaps 'thread'?
> + - const: thead,th1520
> ^^^^^
> total: 0 errors, 3 warnings, 0 checks, 29 lines checked
>
> The 'thead' typo false positive adds noise to automation like the
> linux-riscv patchwork bot:
> https://patchwork.kernel.org/project/linux-riscv/patch/20230722-upstream-beaglev-ahead-dts-v1-2-ccda511357f4@baylibre.com/
>
> diff --git a/scripts/spelling.txt b/scripts/spelling.txt
> index fc7ba95e86a0..855c4863124b 100644
> --- a/scripts/spelling.txt
> +++ b/scripts/spelling.txt
> @@ -1541,7 +1541,6 @@ temeprature||temperature
> temorary||temporary
> temproarily||temporarily
> temperture||temperature
> -thead||thread
> theads||threads
> therfore||therefore
> thier||their
> --
> 2.34.1
>
--
Best Regards
Guo Ren
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-07-23 14:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-23 1:03 [PATCH] scripts/spelling.txt: remove 'thead' as a typo Drew Fustini
2023-07-23 14:50 ` Guo Ren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®