[PATCH 1/2] fs_enet: fix checkpatch.pl problems
Grant Likely
grant.likely at secretlab.ca
Wed Jun 18 15:20:21 EST 2008
On Tue, Jun 17, 2008 at 5:03 PM, John Rigby <jrigby at freescale.com> wrote:
> Fixed all errors and warnings that checkpatch.pl
> reports if this was a new submission.
> Also changed instances of fec_t to struct fec in
> mac-fec.c and mii-fec.c.
Nit: checkpatch is not law; particularly in the case of whitespace
change. Personally, I think cleaning up whitespace in this regard
causes more problems than it solves. Plus, it makes the important
change (fec_t -> struct fec) get lost in a see of unimportant ones
which makes for hard reviewing.
But I must sleep now; I'll review your other patch tomorrow.
Cheers,
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
More information about the Linuxppc-dev
mailing list