| 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 |
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
On Tue, 10 Dec 2024 16:16:46 +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. > > [...] Applied to drm-misc-next, thanks! [1/4] drm/display: bridge_connector: provide atomic_check for HDMI bridges commit: 8ec116ff21a97918d187491dc8f90d7eb173dc1d [2/4] drm/bridge: ite-it6263: drop atomic_check() callback commit: 487d1ed989e5ec418b45b9294e8cceefe29ce4fa [3/4] drm/bridge: lontium-lt9611: drop atomic_check() callback commit: 15b8f5a297f793f859ed2104755ebcf29384e26b [4/4] drm/bridge: dw-hdmi-qp: drop atomic_check() callback commit: f8a2397baf041a5cee408b082334bb09c7e161df Best regards,
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,