[PATCH v4 0/5] drm: fix two issues related to HDMI Connector implementation

From: Dmitry Baryshkov
Date: Mon Jul 15 2024 - 02:33:28 EST


Running IGT tests on Qualcomm Dragonboard820c uncovered two issues with
the HDMI Connector implementation and with its integration into the
drm_bridge_connector. Fix those issues.

Note, I'm not fully satisfied with the drm_bridge_connector move. Maybe
it's better to add drm_bridge_funcs::connector_reset() and call it from
__drm_atomic_helper_connector_reset().

Depends on https://lore.kernel.org/dri-devel/20240704-panel-sw43408-fix-v6-1-3ea1c94bbb9b@xxxxxxxxxx

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---
Changes in v4:
- Fixed DRM_MODE_PROP_IMMUTABLE to use MUST in the single-value clause (Maxime)
- Rebased on top of DRM_DSC_HELPERS patch
- Removed 'depends on DRM_DISPLAY_HELPER' (Maxime)
- Link to v3: https://lore.kernel.org/r/20240702-drm-bridge-connector-fix-hdmi-reset-v3-0-12b0e3124ca4@xxxxxxxxxx

Changes in v3:
- Document the DRM_MODE_PROP_IMMUTABLE requirements currently exposed
only via IGT tests (Maxime).
- Move drm_bridge_connector to drm_display_helper.
- Link to v2: https://lore.kernel.org/r/20240623-drm-bridge-connector-fix-hdmi-reset-v2-0-8590d44912ce@xxxxxxxxxx

Changes in v2:
- Actually pass the flags to drm_property_create_range().
- Link to v1: https://lore.kernel.org/r/20240623-drm-bridge-connector-fix-hdmi-reset-v1-0-41e9894dcdec@xxxxxxxxxx

---
Dmitry Baryshkov (5):
drm/display: stop depending on DRM_DISPLAY_HELPER
drm/drm_property: require DRM_MODE_PROP_IMMUTABLE for single-value props
drm/connector: automatically set immutable flag for max_bpc property
drm/bridge-connector: move to DRM_DISPLAY_HELPER module
drm/bridge-connector: reset the HDMI connector state

MAINTAINERS | 2 +-
drivers/gpu/drm/Makefile | 1 -
drivers/gpu/drm/bridge/Kconfig | 1 +
drivers/gpu/drm/display/Kconfig | 25 ++++++++++++----------
drivers/gpu/drm/display/Makefile | 2 ++
.../gpu/drm/{ => display}/drm_bridge_connector.c | 13 ++++++++++-
drivers/gpu/drm/drm_connector.c | 7 +++++-
drivers/gpu/drm/imx/dcss/Kconfig | 2 ++
drivers/gpu/drm/imx/lcdc/Kconfig | 2 ++
drivers/gpu/drm/ingenic/Kconfig | 2 ++
drivers/gpu/drm/kmb/Kconfig | 2 ++
drivers/gpu/drm/mediatek/Kconfig | 2 ++
drivers/gpu/drm/meson/Kconfig | 2 ++
drivers/gpu/drm/msm/Kconfig | 1 +
drivers/gpu/drm/omapdrm/Kconfig | 2 ++
drivers/gpu/drm/renesas/rcar-du/Kconfig | 2 ++
drivers/gpu/drm/renesas/rz-du/Kconfig | 2 ++
drivers/gpu/drm/renesas/shmobile/Kconfig | 2 ++
drivers/gpu/drm/rockchip/Kconfig | 4 ++++
drivers/gpu/drm/tegra/Kconfig | 1 +
drivers/gpu/drm/tidss/Kconfig | 2 ++
drivers/gpu/drm/xlnx/Kconfig | 1 +
include/drm/drm_property.h | 3 +++
23 files changed, 68 insertions(+), 15 deletions(-)
---
base-commit: cfbc154f11aaa32b4b2887323e4372390648046d
change-id: 20240623-drm-bridge-connector-fix-hdmi-reset-0ce86af053aa

Best regards,
--
Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>