]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/bridge: imx8qxp-pxl2dpi: remove excess error message
authorLuca Ceresoli <luca.ceresoli@bootlin.com>
Tue, 16 Dec 2025 17:58:47 +0000 (18:58 +0100)
committerLuca Ceresoli <luca.ceresoli@bootlin.com>
Tue, 30 Dec 2025 09:11:33 +0000 (10:11 +0100)
imx8qxp_pxl2dpi_find_next_bridge() already emits a DRM_DEV_ERROR() for
every error except -EPROBE_DEFER. The caller emits another one, which is
redundant. Remove the message in the caller and keep the two in
imx8qxp_pxl2dpi_find_next_bridge() as they are more informative about the
error cause.

Reviewed-by: Maxime Ripard <mripard@kernel.org>
Link: https://patch.msgid.link/20251216-drm-bridge-alloc-getput-drm_of_find_bridge-v3-14-b5165fab8058@bootlin.com
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c

index a4941f39a2e6d2fc616aaa38b8a32a8fbef9476b..e03a411cb9db4d9a315f9924cfaca8213e03efa9 100644 (file)
@@ -415,13 +415,8 @@ static int imx8qxp_pxl2dpi_bridge_probe(struct platform_device *pdev)
        }
 
        p2d->next_bridge = imx8qxp_pxl2dpi_find_next_bridge(p2d);
-       if (IS_ERR(p2d->next_bridge)) {
-               ret = PTR_ERR(p2d->next_bridge);
-               if (ret != -EPROBE_DEFER)
-                       DRM_DEV_ERROR(dev, "failed to find next bridge: %d\n",
-                                     ret);
-               return ret;
-       }
+       if (IS_ERR(p2d->next_bridge))
+               return PTR_ERR(p2d->next_bridge);
 
        ret = imx8qxp_pxl2dpi_set_pixel_link_sel(p2d);
        if (ret)