diff options
author | Fengguang Wu <fengguang.wu@intel.com> | 2013-09-17 23:54:09 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2013-10-18 12:11:59 -0400 |
commit | 31ab1ad70b25f4e19f6442e8f6d68025ba43b942 (patch) | |
tree | 68b4ea88de2db77ab07de274036c8f6b758c0986 /drivers/hwmon | |
parent | 9c09bd8d896f8dcce14d0f032cb449a00a3e3141 (diff) |
hwmon: (ds1621) fix coccinelle warnings
drivers/hwmon/ds1621.c:381:1-3: WARNING: PTR_RET can be used
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
Generated by: coccinelle/api/ptr_ret.cocci
CC: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/ds1621.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/hwmon/ds1621.c b/drivers/hwmon/ds1621.c index 5e398c98db2e..872d76744e30 100644 --- a/drivers/hwmon/ds1621.c +++ b/drivers/hwmon/ds1621.c | |||
@@ -378,10 +378,7 @@ static int ds1621_probe(struct i2c_client *client, | |||
378 | hwmon_dev = devm_hwmon_device_register_with_groups(&client->dev, | 378 | hwmon_dev = devm_hwmon_device_register_with_groups(&client->dev, |
379 | client->name, data, | 379 | client->name, data, |
380 | ds1621_groups); | 380 | ds1621_groups); |
381 | if (IS_ERR(hwmon_dev)) | 381 | return PTR_ERR_OR_ZERO(hwmon_dev); |
382 | return PTR_ERR(hwmon_dev); | ||
383 | |||
384 | return 0; | ||
385 | } | 382 | } |
386 | 383 | ||
387 | static const struct i2c_device_id ds1621_id[] = { | 384 | static const struct i2c_device_id ds1621_id[] = { |