Re: [PATCH v2 0/1] bridge/ti-sn65dsi86.c: convert to atomic operations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



(Dropping CC/lists)

Quoting Sam Ravnborg (2022-07-03 21:27:23)
> This patch is what is left on a previous attempt to make the
> ti-sn65dsi86 support DRM_BRIDGE_ATTACH_NO_CONNECTOR.
> 
> I ran into several challenges on the way:
> - I never found a way to retreive bpc when the driver
>   do not create the connector - and the driver needs it.
>   There may be ways to change how bpc is used but as I have no access
>   to HW it is not easy to change much.
> - My understanding of the driver is limited so I dropped trying to
>   do the proper implmentation of format negotiation
> - I had made a small macro to encapsulate a few of the common functions
>   in drm_bridge_funcs - but looking at it with fresh eyes told me
>   this is not a good idea.
> 
> I dropped previous a-b / r-b as the driver has changed a bit since
> last submission.
> Especially to Kieran - sorry for letting you wait so long for almost nothing.

No worries, thanks for digging in.

I'll get my branch back up and rebase on top of this and resume from
there. I can't remember all the details right now - so it will be
interesting to see how to handle the issues you've mentioned above - but
I think some of it is what I was trying to work on. (dealing with
getting the BPC at least..., so I don't know if I considered the issue
you reference about case of the driver not creating a connector).

I don't think I'll be able to resume my DRM work for more than a couple
of weeks now though so I'm going to be a bit longer. Hopefully your
patch here looks like it could be merged independently though.

--
Kieran


> 
> Sorry if this is a duplicate - something went wrong when I sent this
> mail the first time.
> 
>         Sam
> 
> 
> Sam Ravnborg (1):
>       drm/bridge: ti-sn65dsi86: Use atomic variants of drm_bridge_funcs
> 
>  drivers/gpu/drm/bridge/ti-sn65dsi86.c | 23 +++++++++++++++--------
>  1 file changed, 15 insertions(+), 8 deletions(-)
> 
>




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux