[PATCHv3] regulator: pwm-regulator: Make use of the helper function dev_err_probe()
Martin Blumenstingl
martin.blumenstingl at googlemail.com
Sat Sep 25 10:12:28 PDT 2021
Hi Anand,
On Tue, Sep 21, 2021 at 11:27 PM Anand Moon <linux.amoon at gmail.com> wrote:
[...]
> @@ -353,13 +353,8 @@ static int pwm_regulator_probe(struct platform_device *pdev)
>
> drvdata->pwm = devm_pwm_get(&pdev->dev, NULL);
> if (IS_ERR(drvdata->pwm)) {
> - ret = PTR_ERR(drvdata->pwm);
> - if (ret == -EPROBE_DEFER)
> - dev_dbg(&pdev->dev,
> - "Failed to get PWM, deferring probe\n");
> - else
> - dev_err(&pdev->dev, "Failed to get PWM: %d\n", ret);
> - return ret;
> + return dev_err_probe(&pdev->dev, PTR_ERR(drvdata->pwm),
> + "Failed to get PWM\n");
> }
>From functional perspective you're patch is looking good now.
I just noticed that the coding-style in the pwm-regulator driver is
not not use any curly brackets for the if block when there's only one
statement
with the curly brackets removed (and if there are no other changes to
this patch) then you can add my:
Acked-by: Martin Blumenstingl <martin.blumenstingl at googlemail.com>
Best regards,
Martin
More information about the linux-amlogic
mailing list