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

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 09:28:20PM +0200, Ville Syrjälä wrote: > On Wed, Feb 11, 2015 at 02:05:21PM +0200, Ville Syrjälä wrote: > > On Tue, Feb 10, 2015 at 06:38:08PM +, Simon Farnsworth wrote: > > > Older DisplayPort to DVI-D Dual Link adapters designed by Bizlink have > > > bugs > > >

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

2015-03-11 Thread Ville Syrjälä
On Wed, Feb 11, 2015 at 02:05:21PM +0200, Ville Syrjälä wrote: > On Tue, Feb 10, 2015 at 06:38:08PM +, Simon Farnsworth wrote: > > Older DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs > > in their I2C over AUX implementation (fixed in newer revisions). They work > >

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

2015-02-11 Thread Dave Airlie
On 11 February 2015 at 04:42, Simon Farnsworth 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 >

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

2015-02-11 Thread Ville Syrjälä
On Tue, Feb 10, 2015 at 06:38:08PM +, Simon Farnsworth wrote: > Older DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs > in their I2C over AUX implementation (fixed in newer revisions). They work > fine with Windows, but fail with Linux. > > It turns out that they cannot

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

2015-02-11 Thread Jani Nikula
On Tue, 10 Feb 2015, Simon Farnsworth wrote: > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=55228 > Tested-by: Aidan Marks Aidan writes in the bug, "Hi Simon, I'm sorry to report that the v4 patch doesn't work for me at all, it reverts to 1024x768. v3 was great though." BR, Jani.

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

2015-02-11 Thread Daniel Vetter
On Wed, Feb 11, 2015 at 03:36:29PM +1000, Dave Airlie wrote: > On 11 February 2015 at 04:42, Simon Farnsworth > 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

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

2015-02-10 Thread Simon Farnsworth
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

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

2015-02-10 Thread Simon Farnsworth
Older DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in their I2C over AUX implementation (fixed in newer revisions). They work fine with Windows, but fail with Linux. It turns out that they cannot keep an I2C transaction open unless the previous read was 16 bytes; shorter

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

2015-01-28 Thread Jani Nikula
On Wed, 28 Jan 2015, Daniel Vetter wrote: > On Wed, Jan 28, 2015 at 10:59:06AM +0200, Jani Nikula wrote: >> On Tue, 27 Jan 2015, Ville Syrjälä >> wrote: >> > So I've been experimenting a bit with various dongles here, and sadly I've >> > not managed to get any one of them to return short

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

2015-01-28 Thread Daniel Vetter
On Wed, Jan 28, 2015 at 11:33:34AM +0200, Jani Nikula wrote: > On Wed, 28 Jan 2015, Daniel Vetter wrote: > > On Wed, Jan 28, 2015 at 10:59:06AM +0200, Jani Nikula wrote: > >> On Tue, 27 Jan 2015, Ville Syrjälä > >> wrote: > >> > So I've been experimenting a bit with various dongles here, and

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

2015-01-28 Thread Jani Nikula
On Tue, 27 Jan 2015, Ville Syrjälä wrote: > So I've been experimenting a bit with various dongles here, and sadly I've > not managed to get any one of them to return short reads :( > > I did find one that allows changing the speed of the i2c bus, but even if > I reduce it to 1khz there are no

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

2015-01-28 Thread Simon Farnsworth
On Wednesday 28 January 2015 11:33:34 Jani Nikula wrote: > On Wed, 28 Jan 2015, Daniel Vetter wrote: > > On Wed, Jan 28, 2015 at 10:59:06AM +0200, Jani Nikula wrote: > >> On Tue, 27 Jan 2015, Ville Syrjälä > >> wrote: --snip-- > >> > I made the msg size configurable via a module param just to

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

2015-01-28 Thread Daniel Vetter
On Wed, Jan 28, 2015 at 10:59:06AM +0200, Jani Nikula wrote: > On Tue, 27 Jan 2015, Ville Syrjälä wrote: > > So I've been experimenting a bit with various dongles here, and sadly I've > > not managed to get any one of them to return short reads :( > > > > I did find one that allows changing the

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

2015-01-27 Thread Ville Syrjälä
On Mon, Jan 26, 2015 at 04:39:29PM +, Simon Farnsworth wrote: > On Monday 26 January 2015 18:11:01 Ville Syrjälä wrote: > > On Mon, Jan 26, 2015 at 03:47:13PM +, Simon Farnsworth wrote: > > > On Monday 26 January 2015 17:33:35 Ville Syrjälä wrote: > > > > On Mon, Jan 26, 2015 at

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

2015-01-26 Thread Ville Syrjälä
On Mon, Jan 26, 2015 at 03:47:13PM +, Simon Farnsworth wrote: > On Monday 26 January 2015 17:33:35 Ville Syrjälä wrote: > > On Mon, Jan 26, 2015 at 03:22:48PM +, Simon Farnsworth wrote: > > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > > their I2C over

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

2015-01-26 Thread Ville Syrjälä
On Mon, Jan 26, 2015 at 05:33:35PM +0200, Ville Syrjälä wrote: > On Mon, Jan 26, 2015 at 03:22:48PM +, Simon Farnsworth wrote: > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > their I2C over AUX implementation. They work fine with Windows, but fail > > with

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

2015-01-26 Thread Ville Syrjälä
On Mon, Jan 26, 2015 at 03:22:48PM +, Simon Farnsworth wrote: > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > their I2C over AUX implementation. They work fine with Windows, but fail > with Linux. > > It turns out that they cannot keep an I2C transaction open

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

2015-01-26 Thread Simon Farnsworth
On Monday 26 January 2015 18:11:01 Ville Syrjälä wrote: > On Mon, Jan 26, 2015 at 03:47:13PM +, Simon Farnsworth wrote: > > On Monday 26 January 2015 17:33:35 Ville Syrjälä wrote: > > > On Mon, Jan 26, 2015 at 03:22:48PM +, Simon Farnsworth wrote: > > > > DisplayPort to DVI-D Dual Link

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

2015-01-26 Thread Simon Farnsworth
On Monday 26 January 2015 17:33:35 Ville Syrjälä wrote: > On Mon, Jan 26, 2015 at 03:22:48PM +, Simon Farnsworth wrote: > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > their I2C over AUX implementation. They work fine with Windows, but fail > > with Linux. >

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

2015-01-26 Thread Simon Farnsworth
DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in their I2C over AUX implementation. They work fine with Windows, but fail with Linux. It turns out that they cannot keep an I2C transaction open unless the previous read was 16 bytes; shorter reads can only be followed by a

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

2015-01-26 Thread Simon Farnsworth
On Friday 23 January 2015 22:21:09 Thierry Reding wrote: > On Fri, Jan 23, 2015 at 09:46:29PM +0200, Ville Syrjälä wrote: > > On Fri, Jan 23, 2015 at 06:40:38PM +, Simon Farnsworth wrote: > > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > > their I2C over AUX

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

2015-01-24 Thread Daniel Vetter
On Fri, Jan 23, 2015 at 10:21:09PM +0100, Thierry Reding wrote: > On Fri, Jan 23, 2015 at 09:46:29PM +0200, Ville Syrjälä wrote: > > On Fri, Jan 23, 2015 at 06:40:38PM +, Simon Farnsworth wrote: > > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > > their I2C

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

2015-01-23 Thread Thierry Reding
On Fri, Jan 23, 2015 at 09:46:29PM +0200, Ville Syrjälä wrote: > On Fri, Jan 23, 2015 at 06:40:38PM +, Simon Farnsworth wrote: > > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > > their I2C over AUX implementation. They work fine with Windows, but fail > > with

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

2015-01-23 Thread Ville Syrjälä
On Fri, Jan 23, 2015 at 06:40:38PM +, Simon Farnsworth wrote: > DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in > their I2C over AUX implementation. They work fine with Windows, but fail > with Linux. > > It turns out that they cannot keep an I2C transaction open

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

2015-01-23 Thread Simon Farnsworth
DisplayPort to DVI-D Dual Link adapters designed by Bizlink have bugs in their I2C over AUX implementation. They work fine with Windows, but fail with Linux. It turns out that they cannot keep an I2C transaction open unless the previous read was 16 bytes; shorter reads can only be followed by a