diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2013-09-11 10:32:18 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2013-10-13 16:16:25 -0700 |
| commit | e8ab508c27be9868411b6578507e93e02bdb8cdb (patch) | |
| tree | 41760abf18d0d41759dc077e5ed7bbb7862cab95 /drivers/hwmon/nct6775.c | |
| parent | a0393713530c49697e49ce0456c039228ab7facb (diff) | |
| download | lwn-e8ab508c27be9868411b6578507e93e02bdb8cdb.tar.gz lwn-e8ab508c27be9868411b6578507e93e02bdb8cdb.zip | |
hwmon: (nct6775) Use return value from find_temp_source
smatch complains that we don't use the return value from find_temp_source().
Valid point, only find_temp_source() doesn't return a valid error code.
Have it return a valid error code and use it.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/nct6775.c')
| -rw-r--r-- | drivers/hwmon/nct6775.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hwmon/nct6775.c b/drivers/hwmon/nct6775.c index 6eb03ce2cff4..de57e0d60d23 100644 --- a/drivers/hwmon/nct6775.c +++ b/drivers/hwmon/nct6775.c @@ -1545,7 +1545,7 @@ static int find_temp_source(struct nct6775_data *data, int index, int count) if (src == source) return nr; } - return -1; + return -ENODEV; } static ssize_t @@ -1644,7 +1644,7 @@ store_temp_beep(struct device *dev, struct device_attribute *attr, nr = find_temp_source(data, sattr->index, data->num_temp_beeps); if (nr < 0) - return -ENODEV; + return nr; bit = data->BEEP_BITS[nr + TEMP_ALARM_BASE]; regindex = bit >> 3; |