diff options
| author | Fengguang Wu <fengguang.wu@intel.com> | 2013-09-17 20:54:09 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2013-10-18 09:11:59 -0700 |
| 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 5e398c98db2..872d76744e3 100644 --- a/drivers/hwmon/ds1621.c +++ b/drivers/hwmon/ds1621.c @@ -378,10 +378,7 @@ static int ds1621_probe(struct i2c_client *client, hwmon_dev = devm_hwmon_device_register_with_groups(&client->dev, client->name, data, ds1621_groups); - if (IS_ERR(hwmon_dev)) - return PTR_ERR(hwmon_dev); - - return 0; + return PTR_ERR_OR_ZERO(hwmon_dev); } static const struct i2c_device_id ds1621_id[] = { |