drm/bridge: ti-sn65dsi83: Fix enable error path
authorAlexander Stein <alexander.stein@ew.tq-group.com>
Thu, 4 May 2023 06:53:16 +0000 (08:53 +0200)
committerRobert Foss <rfoss@kernel.org>
Thu, 4 May 2023 07:52:19 +0000 (09:52 +0200)
If PLL locking failed, the regulator needs to be disabled again.

Fixes: 5664e3c907e2 ("drm/bridge: ti-sn65dsi83: Add vcc supply regulator support")
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Robert Foss <rfoss@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20230504065316.2640739-1-alexander.stein@ew.tq-group.com
drivers/gpu/drm/bridge/ti-sn65dsi83.c

index 75286c9..1f5c079 100644 (file)
@@ -478,6 +478,7 @@ static void sn65dsi83_atomic_enable(struct drm_bridge *bridge,
                dev_err(ctx->dev, "failed to lock PLL, ret=%i\n", ret);
                /* On failure, disable PLL again and exit. */
                regmap_write(ctx->regmap, REG_RC_PLL_EN, 0x00);
+               regulator_disable(ctx->vcc);
                return;
        }