Re: [PATCH] checkpatch: do not check for "/dev/null" while parsing addition of files

From: Joe Perches
Date: Wed May 05 2021 - 13:24:20 EST


On Wed, 2021-05-05 at 13:42 +0530, Manivannan Sadhasivam wrote:
> "/dev/null" is used to signal created or deleted files in the diff header.
> While parsing the addition of files, checkpatch should look for
> "+++ b/file" and ignore "+++ /dev/null".

nack. checkpatch shouldn't care about a/ b/ uses here.

>
> Without this change, checkpatch falsely reports below warning for the
> patch that does devicetree YAML conversion:

Send me a copy of the source patch that creates this false positive.

>
> ```
> WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst