Re: [PATCH v2 1/8] drm/connector: let drivers declare infoframes as unsupported

[Liu Ying <victor.liu@nxp.com>]

On 08/19/2025, Dmitry Baryshkov wrote:
> Currently DRM framework expects that the HDMI connector driver supports
> all infoframe types: it generates the data as required and calls into
> the driver to program all of them, letting the driver to soft-fail if
> the infoframe is unsupported. This has a major drawback on userspace
> API: the framework also registers debugfs files for all Infoframe types,
> possibly surprising the users when infoframe is visible in the debugfs
> file, but it is not visible on the wire.
> 
> Let drivers declare that they support only a subset of infoframes,
> creating a more consistent interface.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
>  drivers/gpu/drm/display/drm_bridge_connector.c     |  6 ++++
>  drivers/gpu/drm/display/drm_hdmi_state_helper.c    | 37 ++++++++++++++++++++--
>  drivers/gpu/drm/drm_connector.c                    |  4 +++
>  drivers/gpu/drm/drm_debugfs.c                      | 16 +++++++---
>  drivers/gpu/drm/rockchip/inno_hdmi.c               |  1 +
>  drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c             |  1 +
>  drivers/gpu/drm/tests/drm_connector_test.c         | 28 ++++++++++++++++
>  drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c |  8 +++++
>  drivers/gpu/drm/vc4/vc4_hdmi.c                     |  5 +++
>  include/drm/drm_bridge.h                           |  7 ++++
>  include/drm/drm_connector.h                        | 22 +++++++++++++
>  11 files changed, 128 insertions(+), 7 deletions(-)

Acked-by: Liu Ying <victor.liu@nxp.com>