Re: [PATCH v1] checkpatch: Warn about sign offs if there's an accidental patch separator

[Ian Rogers]

On Sun, Jan 4, 2026 at 2:22 PM Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> On Sun, 2026-01-04 at 13:13 -0800, Ian Rogers wrote:
> > On Sun, Jan 4, 2026 at 10:54 AM Joe Perches <joe@xxxxxxxxxxx> wrote:
> > >
> > > On Mon, 2026-01-05 at 01:37 +0800, Kuan-Wei Chiu wrote:
> > > > +Cc Andrew Morton,
> > > []
> > >
> > > > That said, any Ack/Review or feedback from checkpatch.pl
> > > > maintainers/reviewers would still be helpful and appreciated.
> > >
> > >
> > > I think most of this is pointless and incorrect besides.
> > > A patch separator is specifically 3 dashes not a line that
> > > starts with 3 or more dashes.
> >
> > Thanks Joe, do you have a reference for this?
> []
> > > See the original problem
> > > https://lore.kernel.org/lkml/20251203214706.112174-7-irogers@xxxxxxxxxx/
>
> Did you look at this?

I was the author of it. The patch passed checkpatch.pl but then
created a commit with missing tags - ie it was broken. This patch is
fixing this issue so that checkpatch.pl will warn about the missing
tags prior to the patch being sent to LKML, which is kind of
checkpatch.pl's purpose. This is all detailed in the commit message.

Thanks,
Ian Rogers