Return the value of devm_drm_bridge_add() in order to propagate the
error properly, if it fails due to resource allocation failure or bridge
registration failure.

This ensures that the probe function fails safely rather than proceeding
with a potentially incomplete bridge setup.

Fixes: bf7e97910b9f ("drm/imx: parallel-display: add the bridge before 
attaching it")
Signed-off-by: Chen Ni <[email protected]>
---
 drivers/gpu/drm/imx/ipuv3/parallel-display.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/imx/ipuv3/parallel-display.c 
b/drivers/gpu/drm/imx/ipuv3/parallel-display.c
index 6fbf505d2801..590120a33fa0 100644
--- a/drivers/gpu/drm/imx/ipuv3/parallel-display.c
+++ b/drivers/gpu/drm/imx/ipuv3/parallel-display.c
@@ -256,7 +256,9 @@ static int imx_pd_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, imxpd);
 
-       devm_drm_bridge_add(dev, &imxpd->bridge);
+       ret = devm_drm_bridge_add(dev, &imxpd->bridge);
+       if (ret)
+               return ret;
 
        return component_add(dev, &imx_pd_ops);
 }
-- 
2.25.1

Reply via email to