Re: [Intel-gfx] [PATCH 2/3] drm/i915: check PPS regs for sanity when using eDP

2012-04-11 Thread Daniel Vetter
On Tue, Apr 10, 2012 at 10:26:56PM +0100, Chris Wilson wrote: > On Tue, 10 Apr 2012 11:58:04 -0700, Jesse Barnes > wrote: > > If these regs don't have valid values, the panel won't come up, and may > > even cause a system hang. So do a basic sanity check when an eDP panel > > is detected. > > >

Re: [Intel-gfx] [PATCH 2/3] drm/i915: check PPS regs for sanity when using eDP

2012-04-10 Thread Chris Wilson
On Tue, 10 Apr 2012 11:58:04 -0700, Jesse Barnes wrote: > If these regs don't have valid values, the panel won't come up, and may > even cause a system hang. So do a basic sanity check when an eDP panel > is detected. > > Signed-off-by: Jesse Barnes Other than adding a cleanup: error path (de

Re: [Intel-gfx] [PATCH 2/3] drm/i915: check PPS regs for sanity when using eDP

2012-04-10 Thread Jesse Barnes
On Tue, 10 Apr 2012 11:58:04 -0700 Jesse Barnes wrote: > If these regs don't have valid values, the panel won't come up, and may > even cause a system hang. So do a basic sanity check when an eDP panel > is detected. > > Signed-off-by: Jesse Barnes > --- > drivers/gpu/drm/i915/intel_dp.c |

[Intel-gfx] [PATCH 2/3] drm/i915: check PPS regs for sanity when using eDP

2012-04-10 Thread Jesse Barnes
If these regs don't have valid values, the panel won't come up, and may even cause a system hang. So do a basic sanity check when an eDP panel is detected. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/intel_dp.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --gi