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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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:
> 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 Wilson, Intel Open Source Technology Centre
dri-devel mailing list

[Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [XFree86] [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]

Add to Google Powered by Linux