Re: [PATCH rc6 00/30] target/avr merger

[Peter Maydell]

On Sun, 5 Jul 2020 at 19:31, Thomas Huth <huth@tuxfamily.org> wrote:
>
> Am Sun, 5 Jul 2020 07:29:06 -0700 (PDT)
> schrieb no-reply@patchew.org:
>
> > Patchew URL:
> > 20200705140315.260514-1-huth@tuxfamily.org/">https://patchew.org/QEMU/20200705140315.260514-1-huth@tuxfamily.org/
> [...]
> > === OUTPUT BEGIN ===
> > 1/30 Checking commit b5e3116a3591 (target/avr: Add basic parameters
> > of the new platform) WARNING: added, moved or deleted file(s), does
> > MAINTAINERS need updating? #42:
> > new file mode 100644
>
> These warnings are really annoying. The target/avr folder is added to
> MAINTAINERS in the first patch, so they should not occur...?

checkpatch's logic for this is extremely simplistic: it will
warn if:
 * the patch adds, moves or deletes a file
 * the patch does not itself modify MAINTAINERS

This naturally leads to a ton of false-positives.

thanks
-- PMM