Re: [PATCH] fix ptr_ret.cocci warnings

From: Olimpiu Dejeu
Date: Tue Mar 21 2017 - 10:59:36 EST


On Fri, Mar 17, 2017 at 10:48 AM, kbuild test robot <lkp@xxxxxxxxx> wrote:
> drivers/video/backlight/arcxcnn_bl.c:183:1-3: WARNING: PTR_ERR_OR_ZERO can be used
>
>
> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
>
> Generated by: scripts/coccinelle/api/ptr_ret.cocci
>
> CC: Olimpiu Dejeu <olimpiu@xxxxxxxxxxxxxx>
> Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx>
> ---
>
> arcxcnn_bl.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> --- a/drivers/video/backlight/arcxcnn_bl.c
> +++ b/drivers/video/backlight/arcxcnn_bl.c
> @@ -180,10 +180,7 @@ static int arcxcnn_backlight_register(st
>
> lp->bl = devm_backlight_device_register(lp->dev, name, lp->dev, lp,
> &arcxcnn_bl_ops, props);
> - if (IS_ERR(lp->bl))
> - return PTR_ERR(lp->bl);
> -
> - return 0;
> + return PTR_ERR_OR_ZERO(lp->bl);
> }
>
> static void arcxcnn_parse_dt(struct arcxcnn *lp)

Acked-by: Olimpiu Dejeu <olimpiu@xxxxxxxxxxxxxx>