mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [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®