diff options
author | Guenter Roeck <linux@roeck-us.net> | 2012-01-24 17:55:00 -0800 |
---|---|---|
committer | Guenter Roeck <guenter.roeck@ericsson.com> | 2012-03-18 18:27:44 -0700 |
commit | f352df652ff37c434ed32bc5b7eb7ce430b73abc (patch) | |
tree | ef8d38b83e133f2424557d46dc9c2b2ca72732ce | |
parent | 3230f7049e6325c27d6d40f2a1547cf75bc4ddff (diff) | |
download | op-kernel-dev-f352df652ff37c434ed32bc5b7eb7ce430b73abc.zip op-kernel-dev-f352df652ff37c434ed32bc5b7eb7ce430b73abc.tar.gz |
hwmon: (hwmon-vid) Fix checkpatch issues
Fixed:
ERROR: code indent should use tabs where possible
ERROR: space required before the open parenthesis '('
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Not fixed (url):
WARNING: line over 80 characters
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>
Acked-by: Jean Delvare <khali@linux-fr.org>
-rw-r--r-- | drivers/hwmon/hwmon-vid.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/drivers/hwmon/hwmon-vid.c b/drivers/hwmon/hwmon-vid.c index 55f86c4..9f26400 100644 --- a/drivers/hwmon/hwmon-vid.c +++ b/drivers/hwmon/hwmon-vid.c @@ -83,18 +83,18 @@ int vid_from_reg(int val, u8 vrm) { int vid; - switch(vrm) { + switch (vrm) { - case 100: /* VRD 10.0 */ + case 100: /* VRD 10.0 */ /* compute in uV, round to mV */ val &= 0x3f; - if((val & 0x1f) == 0x1f) + if ((val & 0x1f) == 0x1f) return 0; - if((val & 0x1f) <= 0x09 || val == 0x0a) + if ((val & 0x1f) <= 0x09 || val == 0x0a) vid = 1087500 - (val & 0x1f) * 25000; else vid = 1862500 - (val & 0x1f) * 25000; - if(val & 0x20) + if (val & 0x20) vid -= 12500; return (vid + 500) / 1000; @@ -119,7 +119,7 @@ int vid_from_reg(int val, u8 vrm) case 90: /* VRM 9.0 */ val &= 0x1f; return val == 0x1f ? 0 : - 1850 - val * 25; + 1850 - val * 25; case 85: /* VRM 8.5 */ val &= 0x1f; @@ -134,7 +134,7 @@ int vid_from_reg(int val, u8 vrm) val &= 0x1f; return val == 0x1f ? 0 : val & 0x10 ? 5100 - (val) * 100 : - 2050 - (val) * 50; + 2050 - (val) * 50; case 17: /* Intel IMVP-II */ val &= 0x1f; return val & 0x10 ? 975 - (val & 0xF) * 25 : @@ -157,7 +157,7 @@ int vid_from_reg(int val, u8 vrm) return 0; } } - +EXPORT_SYMBOL(vid_from_reg); /* * After this point is the code to automatically determine which @@ -299,8 +299,6 @@ u8 vid_which_vrm(void) return 0; } #endif - -EXPORT_SYMBOL(vid_from_reg); EXPORT_SYMBOL(vid_which_vrm); MODULE_AUTHOR("Rudolf Marek <r.marek@assembler.cz>"); |