Commit 54d7b7f9 authored by Dmitry Baryshkov's avatar Dmitry Baryshkov
Browse files

drm/display: bridge_connector: use drm_bridge_connector_mode_valid() 



Use new drm_bridge_connector_mode_valid() helper if there is a HDMI
bridge in the bridge chain. This removes the need to perform TMDS char
rate check manually in the bridge driver.

Reviewed-by: default avatarChen-Yu Tsai <wens@csie.org>
Reviewed-by: default avatarMaxime Ripard <mripard@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20241130-hdmi-mode-valid-v5-7-742644ec3b1f@linaro.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent d4581ae8

drivers/gpu/drm/display/drm_bridge_connector.c

+15 −1
Original line number Diff line number Diff line
@@ -18,6 +18,7 @@ 
#include <drm/drm_managed.h>

#include <drm/drm_modeset_helper_vtables.h>

#include <drm/drm_probe_helper.h>

#include <drm/display/drm_hdmi_helper.h>

#include <drm/display/drm_hdmi_state_helper.h>



/**
@@ -299,9 +300,22 @@ static int drm_bridge_connector_get_modes(struct drm_connector *connector) 
	return 0;

}



static enum drm_mode_status

drm_bridge_connector_mode_valid(struct drm_connector *connector,

				struct drm_display_mode *mode)

{

	struct drm_bridge_connector *bridge_connector =

		to_drm_bridge_connector(connector);



	if (bridge_connector->bridge_hdmi)

		return drm_hdmi_connector_mode_valid(connector, mode);



	return MODE_OK;

}



static const struct drm_connector_helper_funcs drm_bridge_connector_helper_funcs = {

	.get_modes = drm_bridge_connector_get_modes,

	/* No need for .mode_valid(), the bridges are checked by the core. */

	.mode_valid = drm_bridge_connector_mode_valid,

	.enable_hpd = drm_bridge_connector_enable_hpd,

	.disable_hpd = drm_bridge_connector_disable_hpd,

};