[PATCH] drm/dp: Use large transactions for I2C over AUX

Daniel Vetter daniel at ffwll.ch
Tue Feb 10 23:25:49 PST 2015


On Wed, Feb 11, 2015 at 03:36:29PM +1000, Dave Airlie wrote:
> On 11 February 2015 at 04:42, Simon Farnsworth
> <simon.farnsworth at onelan.co.uk> wrote:
> > A note:
> >
> > This is *not* enough to bring us to parity with Windows with these
> > adapters. There's also something wrong with our HPD handling that trips them
> > up - I suspect, based on AUX traces from one that's happy, that it's our
> > handling of short HPDs that's imperfect, as the device does a short HPD when
> > it sees the DVI-D HPD sense change.
> >
> 
> We don't handle soft irqs well outside of MST, I think the Apple VGA dongle
> also causes a soft irq when you plug a vga monitor into the dongle, when
>  the dongle is already plugged in.

Yeah we probably need some helper function to handle sst short pulse. The
problem there is that with sst the driver owns the drm connector and not
the helper like with mst, so I'm not sure how we should communicate back
to the driver what has been detected exactly. Maybe just a pile of out
pointers for edid and stuff, or we create an sst_connector_state struct or
something. Of course mst should then use the same for handling sst leafs
so that we can share all the special magic for e.g. the apple vga dongle.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the dri-devel mailing list