diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-09-11 00:19:50 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2013-09-11 16:01:40 -0400 |
commit | 3499e5b2e14b792fe411302fea3b6fcc4ba40ef2 (patch) | |
tree | 2f5091c35d0b628d20f5963757e41a448fcc7318 /drivers/hwmon | |
parent | 12ca0b569eac0bf8389b3eac74ba1d7e9f624ab8 (diff) |
hwmon: (amc6821) Fix return value
Propagate return value obtained from i2c_smbus_read_byte_data()
instead of hardcoding.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Cc: T. Mertelj <tomaz.mertelj@guest.arnes.si>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/amc6821.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hwmon/amc6821.c b/drivers/hwmon/amc6821.c index 4fe49d2bfe1d..fcb18535fd56 100644 --- a/drivers/hwmon/amc6821.c +++ b/drivers/hwmon/amc6821.c | |||
@@ -364,7 +364,7 @@ static ssize_t set_pwm1_enable( | |||
364 | if (config < 0) { | 364 | if (config < 0) { |
365 | dev_err(&client->dev, | 365 | dev_err(&client->dev, |
366 | "Error reading configuration register, aborting.\n"); | 366 | "Error reading configuration register, aborting.\n"); |
367 | return -EIO; | 367 | return config; |
368 | } | 368 | } |
369 | 369 | ||
370 | switch (val) { | 370 | switch (val) { |
@@ -665,7 +665,7 @@ static ssize_t set_fan1_div( | |||
665 | if (config < 0) { | 665 | if (config < 0) { |
666 | dev_err(&client->dev, | 666 | dev_err(&client->dev, |
667 | "Error reading configuration register, aborting.\n"); | 667 | "Error reading configuration register, aborting.\n"); |
668 | return -EIO; | 668 | return config; |
669 | } | 669 | } |
670 | mutex_lock(&data->update_lock); | 670 | mutex_lock(&data->update_lock); |
671 | switch (val) { | 671 | switch (val) { |