[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Ville Syrj?l? writes: > On Thu, Nov 22, 2012 at 07:28:44PM +0100, Egbert Eich wrote: > > > > Something similar should be done for drm_edid_is_valid() - even if the > > driver doesn't bother (for instance because this function is only called > > once when the device structures are initialized

[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Ville Syrjälä
On Thu, Nov 22, 2012 at 07:28:44PM +0100, Egbert Eich wrote: > Ville Syrj?l? writes: > > > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > > index dd0df60..aa9b34d 100644 > > > --- a/drivers/gpu/drm/drm_edid.c > > > +++ b/drivers/gpu/drm/drm_edid.c > > > @@ -15

[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Ville Syrj?l? writes: > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > index dd0df60..aa9b34d 100644 > > --- a/drivers/gpu/drm/drm_edid.c > > +++ b/drivers/gpu/drm/drm_edid.c > > @@ -157,6 +157,17 @@ int drm_edid_header_is_valid(const u8 *raw_edid) > > } >

[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Ville Syrjälä
On Thu, Nov 22, 2012 at 09:44:42AM -0500, Egbert Eich wrote: > Consolidate the null_edid_counter and the bad_edid_counter > into EDID error state flags which for the last EDID read > are accessible from user. > Errors are looged it the same error has not been present > in the previous read of the E

Re: [PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Ville Syrj?l? writes: > On Thu, Nov 22, 2012 at 07:28:44PM +0100, Egbert Eich wrote: > > > > Something similar should be done for drm_edid_is_valid() - even if the > > driver doesn't bother (for instance because this function is only called > > once when the device structures are initialized

Re: [PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Ville Syrjälä
On Thu, Nov 22, 2012 at 07:28:44PM +0100, Egbert Eich wrote: > Ville Syrj�l� writes: > > > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > > index dd0df60..aa9b34d 100644 > > > --- a/drivers/gpu/drm/drm_edid.c > > > +++ b/drivers/gpu/drm/drm_edid.c > > > @@ -15

Re: [PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Ville Syrj?l? writes: > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > index dd0df60..aa9b34d 100644 > > --- a/drivers/gpu/drm/drm_edid.c > > +++ b/drivers/gpu/drm/drm_edid.c > > @@ -157,6 +157,17 @@ int drm_edid_header_is_valid(const u8 *raw_edid) > > } >

[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Consolidate the null_edid_counter and the bad_edid_counter into EDID error state flags which for the last EDID read are accessible from user. Errors are looged it the same error has not been present in the previous read of the EDID. This will reset the EDID error status for example when the monitor

Re: [PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Ville Syrjälä
On Thu, Nov 22, 2012 at 09:44:42AM -0500, Egbert Eich wrote: > Consolidate the null_edid_counter and the bad_edid_counter > into EDID error state flags which for the last EDID read > are accessible from user. > Errors are looged it the same error has not been present > in the previous read of the E

[PATCH v3] DRM/KMS/EDID: Consolidate EDID Error Handling (v3)

2012-11-22 Thread Egbert Eich
Consolidate the null_edid_counter and the bad_edid_counter into EDID error state flags which for the last EDID read are accessible from user. Errors are looged it the same error has not been present in the previous read of the EDID. This will reset the EDID error status for example when the monitor