From: Lucas Stach Date: Wed, 19 Jun 2024 18:21:55 +0000 (+0200) Subject: drm/bridge: analogix_dp: remove PLL lock check from analogix_dp_config_video X-Git-Tag: v6.11-rc1~141^2~15^2~5 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b8a4cdbb01312ac72e189072a1058f7948c07f9e;p=thirdparty%2Fkernel%2Flinux.git drm/bridge: analogix_dp: remove PLL lock check from analogix_dp_config_video 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 Reviewed-by: Robert Foss Tested-by: Heiko Stuebner Reviewed-by: Robet Foss Signed-off-by: Robert Foss Link: https://patchwork.freedesktop.org/patch/msgid/20240619182200.3752465-9-l.stach@pengutronix.de --- diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c index c852d9517c275..bbf2c0808acef 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c @@ -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)