On Mon, Nov 17, 2025 at 08:11:49PM +0100, Nicolas Frattaroli wrote: > The drm_bridge "supported_formats" member stores a bitmask of supported > HDMI output formats if the bridge is in fact an HDMI bridge.
It would be nice to convert the supported_formats field to a bitmask of DRM_MODE_COLOR_FORMAT_* values. > > However, until now, the synopsys dw-hdmi-qp driver did not set this > member in the bridge it creates. > > Set it based on the platform data's supported_formats member, and > default to BIT(HDMI_COLORSPACE_RGB) if it's absent, which preserves the > previous behaviour. > > Signed-off-by: Nicolas Frattaroli <[email protected]> > --- > drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > index fe4c026280f0..cf888236bd65 100644 > --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > @@ -1269,6 +1269,11 @@ struct dw_hdmi_qp *dw_hdmi_qp_bind(struct > platform_device *pdev, > dev_warn(dev, "Set ref_clk_rate to vendor default\n"); > } > > + if (plat_data->supported_formats) > + hdmi->bridge.supported_formats = plat_data->supported_formats; > + else > + hdmi->bridge.supported_formats = BIT(HDMI_COLORSPACE_RGB); > + > dw_hdmi_qp_init_hw(hdmi); > > ret = devm_request_threaded_irq(dev, plat_data->main_irq, -- Regards, Laurent Pinchart
