Re: [PATCH 06/24] Input: pwm-vibra - Simplify with dev_err_probe()

From: Krzysztof Kozlowski
Date: Thu Aug 27 2020 - 14:40:57 EST


On Thu, Aug 27, 2020 at 12:06:46PM +0300, Andy Shevchenko wrote:
> On Wed, Aug 26, 2020 at 9:18 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
> >
> > Common pattern of handling deferred probe can be simplified with
> > dev_err_probe(). Less code and also it prints the error value.
>
> > err = PTR_ERR_OR_ZERO(vibrator->vcc);
> > + if (err)
> > + return dev_err_probe(&pdev->dev, err, "Failed to request regulator\n");
>
> Same comment, no need to have additional PTR_ERR_OZR_ZERO().

Yes.

Best regards,
Krzysztof