On Mon, 16 Apr 2012 10:40:08 -0400, Adam Jackson <ajax@xxxxxxxxxx> 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@xxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel