[U-Boot] Policy for checkpatch usage?

Eric Cooper
2011-04-21T15:04:07Z

On Thu, Apr 21, 2011 at 04:56:36PM +0200, Fabian Cenedese wrote:
> Is that even possible? The cosmetic patch itself will be surrounded
> by context lines which may fire up a warning. So these lines need
> to be changed as well to satisy checkpatch. But this new patch
> will again include several context lines... until you have to fix up the
> whole file. Or did I misunderstand?

What's wrong with (cosmetically) fixing all the files that a patch
touches? That way the project gets incremental cleanup of the code
base as it evolves.

(It would be easy to automate a check for whitespace-only patches to
ease the job of the custodians. Line-breaking and other style changes
might still require eyeballing.)

--
Eric Cooper e c c @ c m u . e d u