On Thu, Feb 09, 2012 at 10:23:37AM -0800, Keith Packard wrote:
> <#part sign=pgpmime>
> On Thu, 09 Feb 2012 17:33:23 +0000, Chris Wilson <ch...@chris-wilson.co.uk> 
> wrote:
> > 
> > That code already existed in intel_lvds_init():
> >   if (HAS_PCH_SPLIT(dev)) {
> >      if (I915_READ(PCH_LVDS) & LVDS_DETECTED) == 0)
> >         return false
> >      ...
> >   }
> 
> Tricky. Do we have any idea how reliable this pin is? We've had such bad
> luck in the past...

It's imo guaranteed to blow up somewhere, but we already have the no_lvds
quirk infrastructure in place for that. Bug reporter confirmed that this
patch works for him, so I've decided to include this into the current
-next cycle to get feedback (and no_lvds quirk patches) as early as
possible.

Thanks for the patch and review.
-Daniel
-- 
Daniel Vetter
Mail: dan...@ffwll.ch
Mobile: +41 (0)79 365 57 48
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to