drm/msm/mdp5: fix error return value

Signed-off-by: Rob Clark <robdclark@gmail.com>

1 files changed, 16 insertions, 6 deletions

diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c
index ee8446c1b5f6..42caf7fcb0b9 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c
@@ -280,12 +280,22 @@ struct msm_kms *mdp5_kms_init(struct drm_device *dev)
                 goto fail;
         }
 
-        ret = get_clk(pdev, &mdp5_kms->axi_clk, "bus_clk") ||
-                        get_clk(pdev, &mdp5_kms->ahb_clk, "iface_clk") ||
-                        get_clk(pdev, &mdp5_kms->src_clk, "core_clk_src") ||
-                        get_clk(pdev, &mdp5_kms->core_clk, "core_clk") ||
-                        get_clk(pdev, &mdp5_kms->lut_clk, "lut_clk") ||
-                        get_clk(pdev, &mdp5_kms->vsync_clk, "vsync_clk");
+        ret = get_clk(pdev, &mdp5_kms->axi_clk, "bus_clk");
+        if (ret)
+                goto fail;
+        ret = get_clk(pdev, &mdp5_kms->ahb_clk, "iface_clk");
+        if (ret)
+                goto fail;
+        ret = get_clk(pdev, &mdp5_kms->src_clk, "core_clk_src");
+        if (ret)
+                goto fail;
+        ret = get_clk(pdev, &mdp5_kms->core_clk, "core_clk");
+        if (ret)
+                goto fail;
+        ret = get_clk(pdev, &mdp5_kms->lut_clk, "lut_clk");
+        if (ret)
+                goto fail;
+        ret = get_clk(pdev, &mdp5_kms->vsync_clk, "vsync_clk");
         if (ret)
                 goto fail;