diff options
author | Guenter Roeck <linux@roeck-us.net> | 2013-01-10 08:54:40 -0500 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2013-04-08 00:16:38 -0400 |
commit | 24f9c539b9b7ef8fcc97741027a31f40ef399741 (patch) | |
tree | fb3139765071ebb015436f970ee9d19880c8f7c9 /drivers/hwmon/abituguru.c | |
parent | 692fe501da4b851cbb60edc672679ad1b41e1602 (diff) |
hwmon: Fix 'Macros with complex values' checkpatch errors
Fix:
ERROR: Macros with complex values should be enclosed in parenthesis
when it is seen due to complex code and not due to multi-line variable
declarations.
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: Alistair John Strachan <alistair@devzero.co.uk>
Acked-by: Alistair John Strachan <alistair@devzero.co.uk>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/abituguru.c')
-rw-r--r-- | drivers/hwmon/abituguru.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/drivers/hwmon/abituguru.c b/drivers/hwmon/abituguru.c index b84f180f210c..df0b69987914 100644 --- a/drivers/hwmon/abituguru.c +++ b/drivers/hwmon/abituguru.c | |||
@@ -96,9 +96,12 @@ | |||
96 | #define ABIT_UGURU_MAX_TIMEOUTS 2 | 96 | #define ABIT_UGURU_MAX_TIMEOUTS 2 |
97 | /* utility macros */ | 97 | /* utility macros */ |
98 | #define ABIT_UGURU_NAME "abituguru" | 98 | #define ABIT_UGURU_NAME "abituguru" |
99 | #define ABIT_UGURU_DEBUG(level, format, arg...) \ | 99 | #define ABIT_UGURU_DEBUG(level, format, arg...) \ |
100 | if (level <= verbose) \ | 100 | do { \ |
101 | pr_debug(format , ## arg) | 101 | if (level <= verbose) \ |
102 | pr_debug(format , ## arg); \ | ||
103 | } while (0) | ||
104 | |||
102 | /* Macros to help calculate the sysfs_names array length */ | 105 | /* Macros to help calculate the sysfs_names array length */ |
103 | /* | 106 | /* |
104 | * sum of strlen of: in??_input\0, in??_{min,max}\0, in??_{min,max}_alarm\0, | 107 | * sum of strlen of: in??_input\0, in??_{min,max}\0, in??_{min,max}_alarm\0, |
@@ -1533,7 +1536,7 @@ static int abituguru_resume(struct device *dev) | |||
1533 | } | 1536 | } |
1534 | 1537 | ||
1535 | static SIMPLE_DEV_PM_OPS(abituguru_pm, abituguru_suspend, abituguru_resume); | 1538 | static SIMPLE_DEV_PM_OPS(abituguru_pm, abituguru_suspend, abituguru_resume); |
1536 | #define ABIT_UGURU_PM &abituguru_pm | 1539 | #define ABIT_UGURU_PM (&abituguru_pm) |
1537 | #else | 1540 | #else |
1538 | #define ABIT_UGURU_PM NULL | 1541 | #define ABIT_UGURU_PM NULL |
1539 | #endif /* CONFIG_PM */ | 1542 | #endif /* CONFIG_PM */ |