Re: [PATCH 5/8] drm: Pass supported color formats straight onto drm_bridge

[Marius Vlad <marius.vlad@collabora.com>]

[-- Attachment #1: Type: text/plain, Size: 1834 bytes --]

On Fri, Sep 12, 2025 at 05:31:17PM +0200, Maxime Ripard wrote:
> Hi,
> 
> On Thu, Sep 11, 2025 at 08:34:48PM +0300, Marius Vlad wrote:
> > > > diff --git a/drivers/gpu/drm/msm/dsi/dsi_manager.c b/drivers/gpu/drm/msm/dsi/dsi_manager.c
> > > > index ca400924d4ee..4b87f4f78d38 100644
> > > > --- a/drivers/gpu/drm/msm/dsi/dsi_manager.c
> > > > +++ b/drivers/gpu/drm/msm/dsi/dsi_manager.c
> > > > @@ -479,7 +479,7 @@ int msm_dsi_manager_connector_init(struct msm_dsi *msm_dsi,
> > > >  	if (ret)
> > > >  		return ret;
> > > >  
> > > > -	connector = drm_bridge_connector_init(dev, encoder);
> > > > +	connector = drm_bridge_connector_init(dev, encoder, BIT(HDMI_COLORSPACE_RGB));
> > > 
> > > And this totally depends on the bridge chain. If we have a DSI-to-HDMI
> > > bridge somewhere in the middle, we are able to output YUV data to the
> > > HDMI connector.
> > That's actually the usecase for this patch: to allow passing other color
> > formats, but this patch is a transitory patch to further expose the fact
> > drm_bridge_connector_init was embedding BIT(HDMI_COLORSPACE_RGB) for the
> > format. See rockchip implementation for this bit, the last patch in this
> > series.
> 
> I think Dmitry's point is that it needs to be integrated with the
> atomic_get_input_bus_fmt / atomic_get_output_bus_fmt, because not only
> we need to make sure the monitor supports it, and the userspace demands
> it, we also need to make sure every bridge in the chain (and possibly
> the encoder) can implement it.
Oookay. Will be looking at those bits then. Thanks for pointing where
I need to look at.

tbh it isn't super clear to me why is it now that an issue. If you don't
mind replying back, it this patch actually exposing that, or that was a
thing that had to addressed at one point? 
> 
> Maxime



[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]