[PATCH v2 1/1] backlight: pwm_bl: Use dev_err_probe
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Fri Nov 17 07:44:27 UTC 2023
Hello Alexander,
On Fri, Nov 17, 2023 at 08:33:16AM +0100, Alexander Stein wrote:
> Use dev_err_probe to simplify error paths. Also Let dev_err_probe handle
s/Let/let/
> the -EPROBE_DEFER case and also add an entry to
I'd s/also //
> /sys/kernel/debug/devices_deferred when deferred.
>
> Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>
> [...]
> @@ -568,7 +570,7 @@ static int pwm_backlight_probe(struct platform_device *pdev)
> ret = pwm_backlight_brightness_default(&pdev->dev, data,
> state.period);
> if (ret < 0) {
> - dev_err(&pdev->dev,
> + dev_err_probe(&pdev->dev, ret,
> "failed to setup default brightness table\n");
Missing realign of the continuation line.
> goto err_alloc;
> }
Otherwise LGTM
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20231117/923c1859/attachment.sig>
More information about the dri-devel
mailing list