diff options
author | Guenter Roeck <linux@roeck-us.net> | 2012-01-15 19:48:48 +0100 |
---|---|---|
committer | Guenter Roeck <guenter.roeck@ericsson.com> | 2012-03-19 02:27:30 +0100 |
commit | 47efe8772f241c4be540355d90e38b79cc1092a4 (patch) | |
tree | dd88666be9e1fa896e1ce5ed5d127fcbca811390 /firmware | |
parent | hwmon: (w83792d) Fix checkpatch issues (diff) | |
download | linux-47efe8772f241c4be540355d90e38b79cc1092a4.tar.xz linux-47efe8772f241c4be540355d90e38b79cc1092a4.zip |
hwmon: (w83793) Fix checkpatch issues
Fixed:
ERROR: code indent should use tabs where possible
ERROR: do not use assignment in if condition
ERROR: space required after that ',' (ctx:VxV)
WARNING: braces {} are not necessary for any arm of this statement
WARNING: braces {} are not necessary for single statement blocks
WARNING: simple_strtol is obsolete, use kstrtol instead
WARNING: simple_strtoul is obsolete, use kstrtoul instead
Modify multi-line comments to follow Documentation/CodingStyle.
Also replaced "<constant> == <variable>" with "<variable> == <constant>".
Translation was done with the following coccinelle script to limit risk.
@@
identifier i;
constant C;
@@
<...
- C == i
+ i == C
...>
Not fixed (false positive):
ERROR: Macros with complex values should be enclosed in parenthesis
Cc: Rudolf Marek <r.marek@assembler.cz>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'firmware')
0 files changed, 0 insertions, 0 deletions