Hi,
just a few e-mails ago along this thread Albert Aribaud wrote:
My opinion is that you should make sure that at least the code you touch is checkpatch-clean, so yes, you should fix that; but there is no need to submit 'checkpatch-compliance' patches. Just fix the line here so that checkpatch does not complain.
So I proceeded along that way.
Now Detlev Zundel wrote:
... Hm, I see. Still, can we have one commit (with "cosmetic" in the changelog) that silences checkpatch but does not have any functional changes? We really try hard to separate cosmetic from functional changes. This makes reviewing (and debugging) so much easier.
While I appreciate the careful review of my patches, I cannot hide that it is discouraging for new contributors to be requested for contradictory modifications.
There should be one precise policy, and that should be clearly documented.
http://www.denx.de/wiki/U-Boot/CodingStyle is the place where I would expect to find it.
Luca