[PATCH] drm/edid: Try harder to fix up base EDID blocks

Chris Wilson chris at chris-wilson.co.uk
Tue Apr 24 12:13:59 PDT 2012


On Mon, 16 Apr 2012 10:40:08 -0400, Adam Jackson <ajax at redhat.com> wrote:
> Requiring the first byte of the EDID base block header to be 0 means we
> don't fix up as many transfer errors as we could.  Instead have the
> callers specify whether it's meant to be block 0 or not, and
> conditionally run header fixup based on that.
> 
> Bugzilla: https://bugzilla.redhat.com/812890
> Signed-off-by: Adam Jackson <ajax at redhat.com>

The patch does watch it says on the tin. I don't pretend to know how
frequent this corruption is, but as it stands the current code fails to
be as effective as it could be.

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the dri-devel mailing list