diff options
author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2014-01-03 16:52:22 +0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-01-04 05:49:20 +0400 |
commit | 128296fc3fa2f607050cc2c5d6dc90237f3cedb3 (patch) | |
tree | 4f30efaa8fa90561c44869a2c430289351b23586 /include/linux/socket.h | |
parent | 45ea3932e261af147187ceefb43456c7f8e3b288 (diff) | |
download | linux-128296fc3fa2f607050cc2c5d6dc90237f3cedb3.tar.xz |
sh_eth: coding style fixes
Running 'scripts/checkpatch.pl' on the driver files gives numerous warnings:
- block comments using empty /* line;
- unneeded \ at end of lines;
- message string split across lines;
- use of __attribute__((aligned(n))) instead of __aligned(n) macro;
- use of __attribute__((packed)) instead of __packed macro.
Additionally, running 'scripts/checkpatch.pl --strict' gives more complaints:
- including the paragraph about writing to FSF into the heading comment;
- alignment not matching open paren;
- multiple assignments on one line;
- use of CamelCase names;
- missing {} on one of the *if* arms where another has them;
- spinlock definition without a comment.
While fixing these, also do some more style cleanups:
- remove useless () around expressions;
- add {} around multi-line *if* operator's arm;
- remove space before comma;
- add spaces after /* and before */;
- properly align continuation lines of broken up expressions;
- realign comments to the structure fields.
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/socket.h')
0 files changed, 0 insertions, 0 deletions