diff options
author | Jean Delvare <khali@linux-fr.org> | 2007-09-26 16:16:56 -0400 |
---|---|---|
committer | Mark M. Hoffman <mhoffman@lightlink.com> | 2007-10-09 22:56:31 -0400 |
commit | b965d4b7f614522170af6a7e450be0333792ccd2 (patch) | |
tree | 31f8ae8e50239905627687e454bb3c1f2cac30f1 /drivers/hwmon/lm87.c | |
parent | 428a7039c5717695935b946af9413e59f68928a4 (diff) |
hwmon: (lm87) Fix a division by zero
Missing parentheses in the definition of FAN_FROM_REG cause a division
by zero for a specific register value.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>
Diffstat (limited to 'drivers/hwmon/lm87.c')
-rw-r--r-- | drivers/hwmon/lm87.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/lm87.c b/drivers/hwmon/lm87.c index e60076e8b597..ef8a9b251cd8 100644 --- a/drivers/hwmon/lm87.c +++ b/drivers/hwmon/lm87.c | |||
@@ -129,7 +129,7 @@ static u8 LM87_REG_TEMP_LOW[3] = { 0x3A, 0x38, 0x2C }; | |||
129 | (((val) < 0 ? (val)-500 : (val)+500) / 1000)) | 129 | (((val) < 0 ? (val)-500 : (val)+500) / 1000)) |
130 | 130 | ||
131 | #define FAN_FROM_REG(reg,div) ((reg) == 255 || (reg) == 0 ? 0 : \ | 131 | #define FAN_FROM_REG(reg,div) ((reg) == 255 || (reg) == 0 ? 0 : \ |
132 | 1350000 + (reg)*(div) / 2) / ((reg)*(div)) | 132 | (1350000 + (reg)*(div) / 2) / ((reg)*(div))) |
133 | #define FAN_TO_REG(val,div) ((val)*(div) * 255 <= 1350000 ? 255 : \ | 133 | #define FAN_TO_REG(val,div) ((val)*(div) * 255 <= 1350000 ? 255 : \ |
134 | (1350000 + (val)*(div) / 2) / ((val)*(div))) | 134 | (1350000 + (val)*(div) / 2) / ((val)*(div))) |
135 | 135 | ||