[U-Boot] Policy for checkpatch usage?

Graeme Russ graeme.russ at gmail.com
Wed Apr 20 14:43:11 CEST 2011


On Wednesday, April 20, 2011, Graeme Russ <graeme.russ at gmail.com> wrote:
> On Wednesday, April 20, 2011, Detlev Zundel <dzu at denx.de> wrote:
>> Hi,
>>
>
>>
>> As a base for discussion, what about this:
>>
>>   Use common sense in interpreting the results of checkpatch. Warnings
>>   that clearly only make sense in the Linux kernel can be ignored.  Also
>>   warnings produced for _context lines_ rather than actual changes can
>>   also be ignored.
>
> One man's common sense is another's idiocy
>
> I vote for a zero warnings, zero errors U-Boot specific checkpatch
>

I also think that all patches should be submitted with a checkpatch
summary with an explaination for any errors or warnings - this will at
least save a little effort for the maintainers and reduce the number of
patches bounced only to have the checkpatch problems argued away
by the author anyway

Regards,

Graeme


More information about the U-Boot mailing list