diff options
author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2014-01-05 03:17:06 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-01-04 19:27:57 -0500 |
commit | 2f53e9047e79b638b155785f03dae8999da93bad (patch) | |
tree | 3dfad42aaed7f7b0556bc5d14eb491e88b7bd456 /drivers/net/phy/mdio_bus.c | |
parent | ffeed1beb62f3742693598dfe2f1c010de249bba (diff) | |
download | op-kernel-dev-2f53e9047e79b638b155785f03dae8999da93bad.zip op-kernel-dev-2f53e9047e79b638b155785f03dae8999da93bad.tar.gz |
phy: coding style fixes
The recent patch from Florian Fainelli fixed all 'checkpatch.pl' errors but left
the numerous warnings:
- including <asm/io.h> instead of <linux/io.h>;
- including <asm/uaccess.h> instead of <linux/uaccess.h>;
- *extern* declaration in .c file;
- block comments using empty /* line;
- block comments not starting with * on the middle lines;
- block comments not having trailing */ on a separate line;
- EXPORT_SYMBOL() not immediately following its function;
- unnecessary {} for signle statement block;
- spaces before tabs.
While fixing these, also fix the following style issues (some of which were
found running 'checkpatch.pl --strict'):
- alignment not matching open paren;
- missing {} on one of the *if* arms where another has them;
- use of sizeof(struct structure) instead of sizeof(*variable);
- multiple assignments on one line;
- empty line before };
- file names in the heading comments;
- missing spaces around operators;
- no {} around multi-line *if* operator's arm;
- unneeded () around subexpressions;
- incomplete kernel-doc comment style;
- comment line exceeding 80 characters;
- missing empty line after declarations.
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/phy/mdio_bus.c')
0 files changed, 0 insertions, 0 deletions