Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Adam Jackson <ajax@redhat.com>
Cc: intel-gfx@lists.freedesktop.org, hcb@chaoticmind.net
Subject: Re: Thinkpad T420 and single/dual channel lvds
Date: Wed, 14 Mar 2012 17:44:40 +0100	[thread overview]
Message-ID: <s5hpqcfxhc7.wl%tiwai@suse.de> (raw)
In-Reply-To: <1331736306.30864.7.camel@atropine>

At Wed, 14 Mar 2012 10:45:06 -0400,
Adam Jackson wrote:
> 
> [1  <multipart/signed (7bit)>]
> [1.1  <text/plain; UTF-8 (quoted-printable)>]
> On Wed, 2012-03-14 at 13:37 +0100, Helge Bahmann wrote:
> > Hi everyone,
> > 
> > Booting a Thinkpad T420 with the lid closed results and opening it subsequently results in an 
> > unusable picture on the panel (and no amount of resetting makes it usable): The even pixels show 
> > the nominal content of the framebuffer (with the right half missing) whlie the odd pixels show a 
> > red-/blueish flicker.
> > 
> > A little bit of investigation revealed that with the lid closed on boot, the panel ends up driven in 
> > lvds single channel mode, which the panel probably does not like. It appears that the i915 driver 
> > relies on the BIOS setting up the PCH_LVDS register for dual channel and will otherwise not ever 
> > consider switching.
> > 
> > While the following hack makes my display work, it is quite obviously not the right thing to do -- 
> > could you investigate if there is a "proper" way to determine whether the panel is supposed to be 
> > driven dual channel?
> 
> Nice find!
> 
> There may or may not be a bit for this in the VBT in the BIOS.  But the
> more reliably correct thing I suspect would be to just look at the
> preferred mode for the panel and assume it's dual-link LVDS if the pixel
> clock is >112MHz, since that's the crossover frequency.

Coincidently, we hit the same issue with a HP laptop, and wondered how
is the best way to fix.  I hoped BIOS could handle better,
i.e. setting the power bits no matter whether the lid is opened or
not.  But it doesn't set unless the lid is once opened.

(Interestingly, the bits remain even if you close the lid again before
 booting.  Just opening once seems triggering the probing of LVDS
 panel in BIOS and let it setting the right values.)

FWIW, when I check ironalke_crtc_mode_set(), the clock of the HD+
LVDS mode (1600x900) is 107800 (refclk 120000), while a similar
machine with a HD panel (1366x768) shows 76300.  So, I'm not sure
whether 112MHz could be a right threshold.


thanks,

Takashi

  reply	other threads:[~2012-03-14 16:44 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-14 12:37 Thinkpad T420 and single/dual channel lvds Helge Bahmann
2012-03-14 14:45 ` Adam Jackson
2012-03-14 16:44   ` Takashi Iwai [this message]
2012-03-14 18:09     ` Adam Jackson
2012-03-15 13:15       ` Takashi Iwai
2012-03-15 13:25         ` Chris Wilson
2012-03-15 13:30           ` Takashi Iwai
2012-03-15 14:42             ` Takashi Iwai
2012-03-16 15:33               ` Rodrigo Vivi
2012-03-16 19:55               ` Adam Jackson
2012-03-16 20:29                 ` Takashi Iwai
2012-03-18 17:50                   ` Daniel Vetter
2012-03-18 18:01                     ` Andreas Heider
2012-03-18 20:26                       ` Takashi Iwai
2012-03-18 20:24                     ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5hpqcfxhc7.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=ajax@redhat.com \
    --cc=hcb@chaoticmind.net \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox