drm/bridge: analogix_dp: remove PLL lock check from analogix_dp_config_video
authorLucas Stach <l.stach@pengutronix.de>
Wed, 19 Jun 2024 18:21:55 +0000 (20:21 +0200)
committerRobert Foss <rfoss@kernel.org>
Thu, 27 Jun 2024 09:52:11 +0000 (11:52 +0200)
This check is way too late in the DP enable flow. The PLL must be
locked much earlier, before any link training can happen. If the
PLL is unlocked at that point in time there is something seriously
wrong in the enable flow.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Robert Foss <rfoss@kernel.org>
Tested-by: Heiko Stuebner <heiko@sntech.de>
Reviewed-by: Robet Foss <rfoss@kernel.org>
Signed-off-by: Robert Foss <rfoss@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240619182200.3752465-9-l.stach@pengutronix.de
drivers/gpu/drm/bridge/analogix/analogix_dp_core.c

index c852d9517c27564881fce0a33202b223a66b76e8..bbf2c0808acef85ebd1310175e17e37e698f1a51 100644 (file)
@@ -717,11 +717,6 @@ static int analogix_dp_config_video(struct analogix_dp_device *dp)
 
        analogix_dp_set_video_color_format(dp);
 
-       if (analogix_dp_get_pll_lock_status(dp) == PLL_UNLOCKED) {
-               dev_err(dp->dev, "PLL is not locked yet.\n");
-               return -EINVAL;
-       }
-
        for (;;) {
                timeout_loop++;
                if (analogix_dp_is_slave_video_stream_clock_on(dp) == 0)