question on patch submission

Greg KH greg at kroah.com
Fri Apr 28 09:01:31 EDT 2017


On Fri, Apr 28, 2017 at 09:49:05PM +0900, Michael Mera wrote:
> Hello,
> 
> As part of the eudyptula challenge I submitted a coding style patch on
> staging (that you can find here: https://lkml.org/lkml/2017/4/28/148)
> 
> I had read beforehand the submission guidelines, but I still have some
> questions which I think are more appropriate posted here than as an
> answer to the patch thread:
> 
> - the patch is clear with "checkpatch.pl --strict", but there are some
>   tabs in the middle of the line. In my editor and apparently with
>   checkpatch.pl those tabs are used for alignment (so spend less than 8
>   characters) so that the total of the line in less than 80 characters.
>   In the submitted patch however, due to the +- at the beginning of the
>   line they appear larger and thus the line in more than 80 character.
> 
>   My question is: Should I consider the patch clean or interpret the
>   guideline "tabs are 8 characters" strictly (and not tabs "at the
>   beginning on the line" are 8 characters)?

tabs are always intrepted at 8 character "stops" for kernel development.
Look at other patches on the mailing list and in your editor, all should
be fine when you create it using git.

> - when I make modification to the patch message but not the diff lines,
>   should I still increment the patch version, or rather maybe resend it
>   with something like RESEND in the subject?

Yes, make it a new version, as the version is the combination of both.
Make it trivially easy for a maintainer to know which version to accept.
Marking it as "RESEND" implies that either patch submission could be
accepted, but marking it with a version is explicit as to which you
should use.

thanks,

greg k-h



More information about the Kernelnewbies mailing list