[Intel-gfx] [PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Daniel Vetter
On Mon, Mar 09, 2015 at 05:29:44PM +0200, Ville Syrjälä wrote: > On Mon, Mar 09, 2015 at 11:44:05AM +, Chris Wilson wrote: > > On a noisy link, we may retry the EDID reads multiple times per block > > and still succeed. In such cases, we don't want to flood the kernel logs > > with *ERROR* me

[PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Ville Syrjälä
On Mon, Mar 09, 2015 at 03:39:01PM +, Chris Wilson wrote: > On Mon, Mar 09, 2015 at 05:29:44PM +0200, Ville Syrjälä wrote: > > On Mon, Mar 09, 2015 at 11:44:05AM +, Chris Wilson wrote: > > > On a noisy link, we may retry the EDID reads multiple times per block > > > and still succeed. In

[PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Ville Syrjälä
On Mon, Mar 09, 2015 at 11:44:05AM +, Chris Wilson wrote: > On a noisy link, we may retry the EDID reads multiple times per block > and still succeed. In such cases, we don't want to flood the kernel logs > with *ERROR* messages as we then succeed. Refactor the EDID dumping and > push it into t

[PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Chris Wilson
On Mon, Mar 09, 2015 at 06:04:46PM +0200, Ville Syrjälä wrote: > On Mon, Mar 09, 2015 at 03:39:01PM +, Chris Wilson wrote: > > On Mon, Mar 09, 2015 at 05:29:44PM +0200, Ville Syrjälä wrote: > > > On Mon, Mar 09, 2015 at 11:44:05AM +, Chris Wilson wrote: > > > > On a noisy link, we may r

[PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Chris Wilson
On Mon, Mar 09, 2015 at 05:29:44PM +0200, Ville Syrjälä wrote: > On Mon, Mar 09, 2015 at 11:44:05AM +, Chris Wilson wrote: > > On a noisy link, we may retry the EDID reads multiple times per block > > and still succeed. In such cases, we don't want to flood the kernel logs > > with *ERROR* me

[PATCH] drm: Only warn about an invalid EDID block prior to rejection

2015-03-09 Thread Chris Wilson
On a noisy link, we may retry the EDID reads multiple times per block and still succeed. In such cases, we don't want to flood the kernel logs with *ERROR* messages as we then succeed. Refactor the EDID dumping and push it into the caller rather than the validator so we can suppress the warnings fr