Hello Wei,
Thank you for the patch.
On Wednesday 06 Jul 2016 12:03:32 weiyj...@163.com wrote:
> From: Wei Yongjun
>
> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR.
>
> Signed-off-by: Wei Yongjun
Acked-by: Laurent Pinchart
> ---
> drivers/pinctrl/sh-pfc/pinctrl.c | 5 +
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/pinctrl/sh-pfc/pinctrl.c
> b/drivers/pinctrl/sh-pfc/pinctrl.c index d4e65bc..e208ee0 100644
> --- a/drivers/pinctrl/sh-pfc/pinctrl.c
> +++ b/drivers/pinctrl/sh-pfc/pinctrl.c
> @@ -814,8 +814,5 @@ int sh_pfc_register_pinctrl(struct sh_pfc *pfc)
> pmx->pctl_desc.npins = pfc->info->nr_pins;
>
> pmx->pctl = devm_pinctrl_register(pfc->dev, &pmx->pctl_desc, pmx);
> - if (IS_ERR(pmx->pctl))
> - return PTR_ERR(pmx->pctl);
> -
> - return 0;
> + return PTR_ERR_OR_ZERO(pmx->pctl);
> }
--
Regards,
Laurent Pinchart