Re: checkpatch false positive

From: Joe Perches
Date: Tue Oct 21 2014 - 04:28:37 EST


On Tue, 2014-10-21 at 10:14 +0200, Hans de Goede wrote:
> Hi,
>
> Checkpatch gives the following warning:
>
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #31:
> new file mode 100644
>
> total: 0 errors, 1 warnings, 352 lines checked
>
> 0001-input-Add-new-sun4i-lradc-keys-driver.patch has style problems, please review.
>
> If any of these errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
> On a patch of mine, even though it updates MAINTAINERS properly, it would
> be nice if checkpatch would check for a hunk updating MAINTAINERS, and then
> would not issue this warning (note my perl-foo is way too weak to fix this
> myself).
>
> I've attached the patch triggering the warning.

Hi Hans.

It's not really fixable. Of course you are welcome to
try though.

Many patches are discrete and the entire series isn't
visible to a single MAINTAINERS update scan by checkpatch.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/