[U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"

Luca Ceresoli
2011-04-19T21:56:54Z

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