diff options
| author |   Gabriele Gorla <gorlik@penguintown.net> | 2010-12-08 16:27:22 +0100 |
|---|---|---|
| committer |   Jean Delvare <khali@endymion.delvare> | 2010-12-08 16:27:22 +0100 |
| commit | 8b0f1840a46449e1946fc88860ef3ec8d6b1c2c7 (patch) | |
| tree | d313520d2c32bfe3a603cbaca59ca84cc34e0a04 /drivers/hwmon/adm1026.c | |
| parent | hwmon: (adm1026) Fix setting fan_div (diff) | |
| download | linux-dev-8b0f1840a46449e1946fc88860ef3ec8d6b1c2c7.tar.xz linux-dev-8b0f1840a46449e1946fc88860ef3ec8d6b1c2c7.zip | |
hwmon: (adm1026) Allow 1 as a valid divider value
Allow 1 as a valid div value as specified in the ADM1026 datasheet.
Signed-off-by: Gabriele Gorla <gorlik@penguintown.net>
Cc: stable@kernel.org
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'drivers/hwmon/adm1026.c')
| -rw-r--r-- | drivers/hwmon/adm1026.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/hwmon/adm1026.c b/drivers/hwmon/adm1026.c index 6f257ce1ae07..be0fdd58aa29 100644 --- a/drivers/hwmon/adm1026.c +++ b/drivers/hwmon/adm1026.c @@ -920,9 +920,7 @@ static ssize_t set_fan_div(struct device *dev, struct device_attribute *attr, val = simple_strtol(buf, NULL, 10); new_div = DIV_TO_REG(val); - if (new_div == 0) { - return -EINVAL; - } + mutex_lock(&data->update_lock); orig_div = data->fan_div[nr]; data->fan_div[nr] = DIV_FROM_REG(new_div); |