* [PATCH] m68k: coldfire: fix typo "reseting" in comment
@ 2026-09-04 11:23 Hemanth Selam
2026-09-06 6:40 ` Josh Juran
0 siblings, 1 reply; 5+ messages in thread
From: Hemanth Selam @ 2026-09-04 11:23 UTC (permalink / raw)
To: Greg Ungerer, Geert Uytterhoeven; +Cc: linux-kernel, linux-m68k
Correct "reseting" to "resetting", reported by scripts/checkpatch.pl using
the misspelling list in scripts/spelling.txt. Only touches comments, no
code changes.
Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
arch/m68k/coldfire/reset.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/m68k/coldfire/reset.c b/arch/m68k/coldfire/reset.c
index 6e5f8ab39f32..9507d7810d98 100644
--- a/arch/m68k/coldfire/reset.c
+++ b/arch/m68k/coldfire/reset.c
@@ -16,7 +16,7 @@
#include <asm/mcfsim.h>
/*
- * There are 2 common methods amongst the ColdFure parts for reseting
+ * There are 2 common methods amongst the ColdFure parts for resetting
* the CPU. But there are couple of exceptions, the 5272 and the 547x
* have something completely special to them, and we let their specific
* subarch code handle them.
--
2.48.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] m68k: coldfire: fix typo "reseting" in comment
2026-09-04 11:23 [PATCH] m68k: coldfire: fix typo "reseting" in comment Hemanth Selam
@ 2026-09-06 6:40 ` Josh Juran
2026-09-06 6:43 ` John Paul Adrian Glaubitz
0 siblings, 1 reply; 5+ messages in thread
From: Josh Juran @ 2026-09-06 6:40 UTC (permalink / raw)
To: Hemanth Selam; +Cc: Greg Ungerer, Geert Uytterhoeven, linux-kernel, linux-m68k
On Sep 4, 2026, at 7:23 AM, Hemanth Selam <hemanth.selam@gmail.com> wrote:
> Correct "reseting" to "resetting", reported by scripts/checkpatch.pl using
> the misspelling list in scripts/spelling.txt. Only touches comments, no
> code changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
>
> - * There are 2 common methods amongst the ColdFure parts for reseting
> + * There are 2 common methods amongst the ColdFure parts for resetting
Is there a reason to leave “ColdFure” as is?
Josh
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] m68k: coldfire: fix typo "reseting" in comment
2026-09-06 6:40 ` Josh Juran
@ 2026-09-06 6:43 ` John Paul Adrian Glaubitz
2026-09-06 6:46 ` Josh Juran
0 siblings, 1 reply; 5+ messages in thread
From: John Paul Adrian Glaubitz @ 2026-09-06 6:43 UTC (permalink / raw)
To: Josh Juran, Hemanth Selam
Cc: Greg Ungerer, Geert Uytterhoeven, linux-kernel, linux-m68k
On Sun, 2026-09-06 at 02:40 -0400, Josh Juran wrote:
> On Sep 4, 2026, at 7:23 AM, Hemanth Selam <hemanth.selam@gmail.com> wrote:
>
> > Correct "reseting" to "resetting", reported by scripts/checkpatch.pl using
> > the misspelling list in scripts/spelling.txt. Only touches comments, no
> > code changes.
> >
> > Assisted-by: Cursor:claude-opus-5
> > Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
> >
> > - * There are 2 common methods amongst the ColdFure parts for reseting
> > + * There are 2 common methods amongst the ColdFure parts for resetting
>
> Is there a reason to leave “ColdFure” as is?
The AI assistant probably never heard about ColdFire, so it assumed the spelling
is correct ;-).
Adrian
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer
`. `' Physicist
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] m68k: coldfire: fix typo "reseting" in comment
2026-09-06 6:43 ` John Paul Adrian Glaubitz
@ 2026-09-06 6:46 ` Josh Juran
2026-09-07 9:15 ` Hemanth Selam
0 siblings, 1 reply; 5+ messages in thread
From: Josh Juran @ 2026-09-06 6:46 UTC (permalink / raw)
To: John Paul Adrian Glaubitz
Cc: Hemanth Selam, Greg Ungerer, Geert Uytterhoeven, linux-kernel,
linux-m68k
On Sep 6, 2026, at 2:43 AM, John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote:
> On Sun, 2026-09-06 at 02:40 -0400, Josh Juran wrote:
>> On Sep 4, 2026, at 7:23 AM, Hemanth Selam <hemanth.selam@gmail.com> wrote:
>>
>>> Correct "reseting" to "resetting", reported by scripts/checkpatch.pl using
>>> the misspelling list in scripts/spelling.txt. Only touches comments, no
>>> code changes.
>>>
>>> Assisted-by: Cursor:claude-opus-5
>>> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
>>>
>>> - * There are 2 common methods amongst the ColdFure parts for reseting
>>> + * There are 2 common methods amongst the ColdFure parts for resetting
>>
>> Is there a reason to leave “ColdFure” as is?
>
> The AI assistant probably never heard about ColdFire, so it assumed the spelling
> is correct ;-).
What about the human reviewing its output? Or am I misunderstanding the meaning of "Signed-off-by”?
Josh
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] m68k: coldfire: fix typo "reseting" in comment
2026-09-06 6:46 ` Josh Juran
@ 2026-09-07 9:15 ` Hemanth Selam
0 siblings, 0 replies; 5+ messages in thread
From: Hemanth Selam @ 2026-09-07 9:15 UTC (permalink / raw)
To: Josh Juran
Cc: John Paul Adrian Glaubitz, Greg Ungerer, Geert Uytterhoeven,
linux-kernel, linux-m68k
On Sun, Sep 06, 2026 at 02:46:04AM -0400, Josh Juran wrote:
> What about the human reviewing its output? Or am I misunderstanding
> the meaning of "Signed-off-by"?
You are not misunderstanding it. My Signed-off-by certifies the DCO,
but sending the patch also means I am answerable for what is in it, and
on this one I did not look hard enough.
The patch came out of a mechanical sweep: checkpatch.pl flags words
listed in scripts/spelling.txt, "reseting" is in that list and
"ColdFure" is not, so nothing pointed at it. That explains why the
tool stopped there. It does not explain why I did not, because the two
words are on the same line and reading the comment was my job.
You caught it, and the v2 fixes both words; Greg has since applied it.
The wider lesson I am taking from this is that a large series does not
get to be reviewed in bulk. Each patch has to be read on its own, in
context, before it goes out, and that is what I will be doing for the
rest of this work.
Thanks for pushing on it rather than letting it through.
Hemanth
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-09-07 9:15 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-04 11:23 [PATCH] m68k: coldfire: fix typo "reseting" in comment Hemanth Selam
2026-09-06 6:40 ` Josh Juran
2026-09-06 6:43 ` John Paul Adrian Glaubitz
2026-09-06 6:46 ` Josh Juran
2026-09-07 9:15 ` Hemanth Selam
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®