Re: [PATCH] hwmon: Use PTR_ERR_OR_ZERO instead of reimplementing its function
From: Guenter Roeck
Date: Sat Sep 15 2018 - 20:43:24 EST
On Tue, Sep 11, 2018 at 08:03:37PM +0800, zhong jiang wrote:
> PTR_ERR_OR_ZERO has implemented the same function. We prefer to use
> inlined function rather than code-opened implementation.
>
> Signed-off-by: zhong jiang <zhongjiang@xxxxxxxxxx>
Applied to hwmon-next. For future patches, please also refer to the
affected driver in the Subject line, unless the change is in the
subsystem core code.
Thanks,
Guenter
> ---
> drivers/hwmon/asus_atk0110.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/hwmon/asus_atk0110.c b/drivers/hwmon/asus_atk0110.c
> index a6636fe..a7cf008 100644
> --- a/drivers/hwmon/asus_atk0110.c
> +++ b/drivers/hwmon/asus_atk0110.c
> @@ -1210,10 +1210,8 @@ static int atk_register_hwmon(struct atk_data *data)
> data->hwmon_dev = hwmon_device_register_with_groups(dev, "atk0110",
> data,
> data->attr_groups);
> - if (IS_ERR(data->hwmon_dev))
> - return PTR_ERR(data->hwmon_dev);
>
> - return 0;
> + return PTR_ERR_OR_ZERO(data->hwmon_dev);
> }
>
> static int atk_probe_if(struct atk_data *data)
> --
> 1.7.12.4
>