backlight: as3711_bl: Handle deferred probe
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Tue, 5 Mar 2024 08:11:59 +0000 (09:11 +0100)
committerLee Jones <lee@kernel.org>
Thu, 7 Mar 2024 09:03:29 +0000 (09:03 +0000)
Don't pollute dmesg on deferred probe and simplify the code with
dev_err_probe().

Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20240305-backlight-probe-v2-4-609b0cf24bde@linaro.org
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/video/backlight/as3711_bl.c

index 28437c2da0f5894003f3fab8055322539c2688ce..e6f66bb35ef596e9b5a2959a6039e5bce731b10a 100644 (file)
@@ -383,10 +383,8 @@ static int as3711_backlight_probe(struct platform_device *pdev)
 
        if (pdev->dev.parent->of_node) {
                ret = as3711_backlight_parse_dt(&pdev->dev);
-               if (ret < 0) {
-                       dev_err(&pdev->dev, "DT parsing failed: %d\n", ret);
-                       return ret;
-               }
+               if (ret < 0)
+                       return dev_err_probe(&pdev->dev, ret, "DT parsing failed\n");
        }
 
        if (!pdata->su1_fb && !pdata->su2_fb) {