Etiquette of submitting patches for fixing coding style.

Graeme Russ graeme.russ at gmail.com
Fri Oct 26 16:56:24 EDT 2012


Hi Eugene,

On Oct 27, 2012 7:18 AM, "Eugene Voronkov" <eugene.voronkov at gmail.com>
wrote:
>
>
>
> On Fri, Oct 26, 2012 at 5:38 AM, Sumeet pawnikar <sumeet4linux at gmail.com>
wrote:
>>
>>
>>
>> On Fri, Oct 26, 2012 at 1:57 AM, Eugene Voronkov <
eugene.voronkov at gmail.com> wrote:
>>>
>>> I watched Kroah-Hartman's video[1] on submitting patches where he walks
through the process of fixing coding style.  I feel like this would be a
good way for me to jump into the process but I need more information.  At
what point do code style patches stop being more trouble then they're worth
to the maintainers?  For example, running checkpatch.pl against all files
is showing around 3 non-trivial style violations per file.  Would a patch
fixing 12 violations across 4 files be worth submitting?
>>>
>>>
>>> 1. http://www.youtube.com/watch?v=LLBrBBImJt4
>>>
>>>
>>> _______________________________________________
>>> Kernelnewbies mailing list
>>> Kernelnewbies at kernelnewbies.org
>>> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
>>>
>> Suggestion would be, divide your single patch in separate individual
patches with respect to functionality/violations fix.
>
>
>
> So for example, one patch removes braces from if/else conditionals with
single statement.  Another patch fixes incorrect spacing.  Correct?

Best to fix all checkpatch errors and warnings in the same file in the same
patch. If you ate fixing two related files (two files in the same driver
for example) then I would put them both in the same patch as well.

If the two files are unrelated, keep the patches separate.

Don't mix functional changes with cosmetic (checkpatch) fixups.

Regards,

Graeme
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20121027/5fd4ba7e/attachment.html 


More information about the Kernelnewbies mailing list