Re: [bug report] drm/bridge: Add ti-tfp410 DVI transmitter driver

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

 



Hi Jyri,

On Wednesday 07 Dec 2016 22:28:56 Jyri Sarha wrote:
> On 12/07/16 16:46, Daniel Vetter wrote:
> >>> This implementation is a hack in the first place. The code will be
> >>> removed when DRM has a proper generic connector support.
> > 
> > Hm, what's this generic drm connector thing you talk about? What's it for?
> > Asking since I haven't seen that yet nor heard in a discussion ...
> 
> After looking up this comment from Laurent:
> 
> On 11/10/16 14:15, Laurent Pinchart wrote [1]:
> >>> The implementation side is not so critical, because it more easily
> >>> changed, but should I create an independent generic platform-device
> >>> driver for such DVI/HDMI connector or just implement the connector side
> >>> within tfp410 driver?
> > 
> > Longer term I'd like to go for connector drivers, but it might take a
> > bit of infrastructure work. If you can give it a try it would be great !
> > Otherwise I'm fine with handling that in the tfp410 driver for now.
> 
> ... I think I miss understood it. He was probably referring to the
> infrastructure needed in the tfp410 driver. I'll try to get that done
> for 4.11 or 4.12.

I was referring to drivers for connectors, binding to the connectors DT nodes. 
We shouldn't duplicate handling of connector-related features (ddc-i2c-bus 
being the most important one) in every bridge driver, and we need to pave the 
way for bridges chained with other bridges instead of being connected directly 
to a connector.

> [1]
> https://lists.freedesktop.org/archives/dri-devel/2016-November/123337.html

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[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