Hi, net/net.c currently raises a lot of checkpatch issues: 76 errors, 197 warnings. This patchset fixes most of them, leaving 0 errors and 25 warnings.
Since the changes are quite massive I split them in separate patches, one per each category of errors/warning. The 25 warnings still present are of the following types: - WARNING: consider using kstrto* in preference to simple_strtoul which is Linux-specific; - WARNING: externs should be avoided in .c files these are due to loop-dependencies between net/net.c and other files; - WARNING: line over 80 characters described in the commit message for patch 1; - WARNING: Use of volatile is usually wrong [...] I've had a quick look at these, but I don't know in detail how the networking code and Ethernet drivers (and maybe DMA?) interact, so I cannot remove them being sure that things will continue working. Luca Ceresoli (8): net/net.c: cosmetic: fix lines over 80 characters net/net.c: cosmetic: variable initializations net/net.c: cosmetic: fix whitespace issues net/net.c: cosmetic: fix brace issues net/net.c: cosmetic: fix pointer syntax issues net/net.c: cosmetic: parentheses not required for return net/net.c: cosmetic: fix indentation net/net.c: cosmetic: do not use assignment in if condition net/net.c | 600 ++++++++++++++++++++++++++++++++++--------------------------- 1 files changed, 333 insertions(+), 267 deletions(-) _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
