On Mon, Nov 17, 2025 at 08:11:54PM +0100, Nicolas Frattaroli wrote: > From: Derek Foreman <[email protected]> > > Register the color format property in the dw_hdmi_qp-rockchip driver, > and act on requested format changes as part of the connector state in > the vop2 video output driver. > > Signed-off-by: Derek Foreman <[email protected]> > Signed-off-by: Marius Vlad <[email protected]> > Signed-off-by: Nicolas Frattaroli <[email protected]> > --- > drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c | 3 ++ > drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 46 > ++++++++++++++++++++++++++ > drivers/gpu/drm/rockchip/rockchip_drm_vop2.h | 2 ++ > 3 files changed, 51 insertions(+) > > diff --git a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c > b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c > index 7c294751de19..7028166fdace 100644 > --- a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c > +++ b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c > @@ -635,6 +635,9 @@ static int dw_hdmi_qp_rockchip_bind(struct device *dev, > struct device *master, > return dev_err_probe(hdmi->dev, PTR_ERR(connector), > "Failed to init bridge connector\n"); > > + if (!drm_mode_create_hdmi_color_format_property(connector, > supported_colorformats)) > + drm_connector_attach_color_format_property(connector); > +
Why shouldn't we register the property in drmm_connector_hdmi_init directly? Maxime
signature.asc
Description: PGP signature
