Re: [Intel-gfx] [PATCH] drm/i915: fixup infoframe support for sdvo

2012-05-20 Thread Daniel Vetter
On Sun, May 20, 2012 at 11:17:36AM -0300, Eugeni Dodonov wrote: > On Sat, May 12, 2012 at 3:22 PM, Daniel Vetter wrote: > > > At least the worst offenders: > > - SDVO specifies that the encoder should compute the ecc. Testing also > > shows that we must not send the ecc field, so copy the dip_inf

Re: [Intel-gfx] [PATCH] drm/i915: fixup infoframe support for sdvo

2012-05-20 Thread Eugeni Dodonov
On Sat, May 12, 2012 at 3:22 PM, Daniel Vetter wrote: > At least the worst offenders: > - SDVO specifies that the encoder should compute the ecc. Testing also > shows that we must not send the ecc field, so copy the dip_infoframe > struct to a temporay place and avoid the ecc field. This way the

Re: [Intel-gfx] [PATCH] drm/i915: fixup infoframe support for sdvo

2012-05-19 Thread Daniel Vetter
On Sat, May 12, 2012 at 08:22:00PM +0200, Daniel Vetter wrote: > At least the worst offenders: > - SDVO specifies that the encoder should compute the ecc. Testing also > shows that we must not send the ecc field, so copy the dip_infoframe > struct to a temporay place and avoid the ecc field. Th

[Intel-gfx] [PATCH] drm/i915: fixup infoframe support for sdvo

2012-05-12 Thread Daniel Vetter
At least the worst offenders: - SDVO specifies that the encoder should compute the ecc. Testing also shows that we must not send the ecc field, so copy the dip_infoframe struct to a temporay place and avoid the ecc field. This way the avi infoframe is exactly 17 bytes long, which agrees with