On Wed, 29 Dec 2010 22:11:09 +0100 Alex Riesen <raa.lkml at gmail.com> wrote:
> On Wed, Dec 29, 2010 at 21:16, Jesse Barnes <jbarnes at virtuousgeek.org> > wrote: > > On Wed, 29 Dec 2010 11:40:04 -0800 > > Linus Torvalds <torvalds at linux-foundation.org> wrote: > >> Chris - why did that lvds_ssc_freq thing suddenly start mattering? Can > >> we please just disable spread-spectrum entirely? Or perhaps only if we > >> notice that it was enabled already? Or something? > > > > Randy, Jeff and Alex, does the below help at all? ?If so, it may be the > > minimal fix we want for 2.6.37. > > Something broke your patch: whitespace instead of tabs. Yeah, just copy & pasted, sorry. > > diff --git a/drivers/gpu/drm/i915/intel_bios.c > > b/drivers/gpu/drm/i915/intel_bios > > index 2b20786..d27d016 100644 > > --- a/drivers/gpu/drm/i915/intel_bios.c > > +++ b/drivers/gpu/drm/i915/intel_bios.c > > @@ -263,6 +263,9 @@ parse_general_features(struct drm_i915_private > > *dev_priv, > > ? ? ? ? ? ? ? ?dev_priv->int_tv_support = general->int_tv_support; > > ? ? ? ? ? ? ? ?dev_priv->int_crt_support = general->int_crt_support; > > ? ? ? ? ? ? ? ?dev_priv->lvds_use_ssc = general->enable_ssc; > > + ? ? ? ? ? ? ? /* force disable until we can parse this correctly */ > > This comment seems to imply that SSC wasn't used at all until recently, right? No, we enabled it previously, but it apparently doesn't work on all platforms, probably because we're either looking in the wrong place for the bit that tells us to enable it or not, or we're getting the wrong frequency on some platforms. Probably both (the VBIOS guys work closely with the Windows driver team that consumes this data, and don't always tell us when they make incompatible changes). > > + ? ? ? ? ? ? ? if (IS_GEN5(dev) || IS_GEN6(dev)) > > + ? ? ? ? ? ? ? ? ? ? ? dev_priv->lvds_use_ssc = 0; > > Doesn't change anything here. Display stays blank. Sounds like your problem is separate from SSC then, more likely related to panel power or backlight control. Have you tried bisecting for the problem between 2.6.35 and 2.6.36? -- Jesse Barnes, Intel Open Source Technology Center