Re: [PATCH] MAINTAINERS: correct file entry for AD7091R

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 08, 2024 at 10:56:02AM +0300, Dan Carpenter wrote:
> Generally, patches to MAINTAINERS don't need Fixes tags.
>
> It doesn't matter if the patch is in mainline or not.  If it's in a tree
> which is rebased then maintainers will fold the fix into the original
> commit and the Fixes tag will be lost.  But that's fine.

Makes sense. Thank you for the information and guidance.
>
> Also when you're using a Fixes tag, it should all go one one line.
> Don't line wrap it at 75 characters.  Just go over the limit.  That's
> the normal/correct way.
Also seems like checkpatch does not like the Fixes line without the
keyword Commit. So the fixes line below generates an Error.

Fixes : 7564efb37346 ("MAINTAINERS: Add entry for TQ-Systems device
trees and drivers")

ERROR: Please use git commit description style 'commit <12+ chars of
sha1> ("<title line>")' - ie: 'commit 7564efb37346 ("MAINTAINERS: Add
entry for TQ-Systems device trees and drivers")'

Adding the keyword commit gives no error.

Documentation shows to use the Fixes tag without the keyword commit.
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes

>
> regards,
> dan carpenter
>




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux