mbox series

[0/4] drm/bridge: move drm_atomic_helper_connector_hdmi_check() to drm_bridge_connector

Message ID 20241210-bridge_hdmi_check-v1-0-a8fdd8c5afa5@linaro.org (mailing list archive)
Headers show
Series drm/bridge: move drm_atomic_helper_connector_hdmi_check() to drm_bridge_connector | expand

Message

Dmitry Baryshkov Dec. 10, 2024, 2:16 p.m. UTC 
Reduce boilerplate code and move calls to
drm_atomic_helper_connector_hdmi_check() to the drm_bridge_connector.
Initially the drm_bridge_connector didn't use HDMI state helpers
directly, so each driver had to call that function on its own. Since the
commit 9a71cf8b6fa4 ("drm/bridge-connector: reset the HDMI connector
state") it depends on DRM_DISPLAY_HDMI_STATE_HELPER, allowing us to use
the helper from the generic code too.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
Dmitry Baryshkov (4):
      drm/display: bridge_connector: provide atomic_check for HDMI bridges
      drm/bridge: ite-it6263: drop atomic_check() callback
      drm/bridge: lontium-lt9611: drop atomic_check() callback
      drm/bridge: dw-hdmi-qp: drop atomic_check() callback

 drivers/gpu/drm/bridge/ite-it6263.c            | 10 ----------
 drivers/gpu/drm/bridge/lontium-lt9611.c        | 10 ----------
 drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c   | 17 -----------------
 drivers/gpu/drm/display/drm_bridge_connector.c | 13 +++++++++++++
 4 files changed, 13 insertions(+), 37 deletions(-)
---
base-commit: 84e541b1e58e04d808e1bb13ef566ffbe87aa89b
change-id: 20241205-bridge_hdmi_check-9fd86dd5b290

Best regards,

Comments

Maxime Ripard Dec. 11, 2024, 5:11 p.m. UTC | #1 
On Tue, Dec 10, 2024 at 04:16:46PM +0200, Dmitry Baryshkov wrote:
> Reduce boilerplate code and move calls to
> drm_atomic_helper_connector_hdmi_check() to the drm_bridge_connector.
> Initially the drm_bridge_connector didn't use HDMI state helpers
> directly, so each driver had to call that function on its own. Since the
> commit 9a71cf8b6fa4 ("drm/bridge-connector: reset the HDMI connector
> state") it depends on DRM_DISPLAY_HDMI_STATE_HELPER, allowing us to use
> the helper from the generic code too.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Acked-by: Maxime Ripard <mripard@kernel.org>

Maxime