Re: [PATCH v3 1/3] drm/layerscape: Add sii9022a driver

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

 



+ Nicolas, Laurent and Rob

Hi,

On Wed, 9 Mar 2016 11:49:38 +0000
Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote:

> Hi Meng,
> 
> On 9 March 2016 at 08:31, Meng Yi <meng.yi@xxxxxxx> wrote:
> > The SiI9022A is an ultra low-power HDMI transmitter. It supports
> > resolutions from standard definition 480i/p and 576i/p all the way
> > to high-definition 720p, 1080i, and 1080p, the highest resolution
> > supported 4K today. It also supports all PC resolutions up
> > to UXGA for netbooks
> >
> An alternative approach (making a drm bridge) for the sii902x devices
> was taken by Boris a bit before your initial submission of this driver
> [1].
> I believe that you guys can coordinate and once a decision is made
> whether a i2c or bridge driver (personally leaning towards the
> latter), you can work together and review each others' work.

The reason I decided to expose the device as a drm_bridge is because
there was several discussions about merging drm_bridge and
drm_encoder_slave, and IIRC, the proposal was to move to the drm_bridge
approach.

Another reason I've chosen to go for the drm_bridge solution is because
I wanted to avoid as much as possible the development of a new glue for
each new encoder bridge type (HDMI/DVI, DP, ...), and, AFAICT drm_bridge
allow that, while it's a bit more complicated with drm_encoder_slave.

> 
> Alternatively you can take a review/comment on a similar work (SiI8620
> bridge device) by Andrzej, who would return the favour and check this
> series :-)

Yes, I'll try to have a look.

Thanks,

Boris


-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
_______________________________________________
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