Re: [git pull] drm fixes

Takashi Iwai
2010-12-22T17:07:20Z

At Wed, 22 Dec 2010 16:59:06 +0000,
Chris Wilson wrote:
>
> On Wed, 22 Dec 2010 17:24:36 +0100, Takashi Iwai <tiwai@suse.de> wrote:
> > The commit 448f53a1ede54eb854d036abf54573281412d650
> > drm/i915/bios: Reverse order of 100/120 Mhz SSC clocks
> >
> > causes a regression on a SandyBridge machine here.
> > The laptop display (LVDS) becomes blank. Reverting the commit fixes
> > the problem.
>
> The question is whose BIOS is wrong? The Lenovo U160's or the
> Sandybridge SDV? And why does it work for that other OS? <Insert
> rhetorical question of the day here.>
>
> It's back to the square one for one or the other platform...

Yeah, we can blame BIOS :) And, this is likely the BIOS on my machine
here that is broken.

But this seems like an issue that you can't rely solely on VBT. We
can never guarantee that BIOS is correct (who can?), and there is no
way to avoid this change as long as it's hard-coded. We've hit
another regression by VBT check (e-DP wrongly detected; kernel bug
24822), so I think judging the behavior only from BIOS is rather
dangerous.

thanks,

Takashi