linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Chris Diamand <chris.diamand@gmail.com>
To: Keith Packard <keithp@keithp.com>
Cc: linux-kernel@vger.kernel.org, tiwai@suse.de, melchior.franz@gmail.com
Subject: Re: i915/kms/backlight-combo mode problem
Date: Sat, 21 Jan 2012 21:25:56 +0000	[thread overview]
Message-ID: <4F1B2D64.7020705@gmail.com> (raw)
In-Reply-To: <86k44kkity.fsf@sumi.keithp.com>

On 21/01/12 18:32, Keith Packard wrote:
> On Sat, 21 Jan 2012 17:09:25 +0000, Chris Diamand<chris.diamand@gmail.com>  wrote:
>> Hi,
>>
>> I have the problem described in the thread here
>> (https://lkml.org/lkml/2011/4/29/217) where the backlight
>> values are reversed so although stuff is displayed on the screen, the
>> backlight is off. This occurs with kernels from about 2.6.39 onwards.
>>
>> After booting a "broken" kernel (>~2.6.39), the backlight can be lit
>> temporarily with:
>> setpci -s 00:02.0 F4.B=0
>> and then turned off with:
>> setpci -s 00:02.0 F4.B=FF
> Can you experiment with other values? I'd like to know if it's
> completely reversed, or if there's some other interaction here.
>
> Testing 0x80, 0x7f, 0x01, 0xfe would all be interesting to me.
>
0x00 is the brightest. It gets very gradually dimmer up to about 0xB0, 
when the gaps between
brightness levels are slightly bigger.
0xFE is very low but still works.
Barely any difference between 0x80 and 0x7f, same with 0x00 and 0x01, etc.

> I suspect that writes to the LPBC value are being trapped by SMI and
> 'doing things' underneath.
>
>> Yesterday I tried this patch, https://lkml.org/lkml/2011/4/30/37 by
>> manually applying it to the 3.2.1
>> kernel source - this fixes the problem.
> Can you still turn the backlight off with this patch in place?
Yes, with Fn-F6. It turns back on again when the mouse is moved or a key 
is pressed.
>> ** Unfortunately the LKML thread stops after this patch and given that
>> the problem is still present in the latest source, this code wasn't
>> pushed into the kernel. Is there a reason for this or was it just
>> forgotten about? What can I do about this? I am happy to make a git
>> patch with the latest source if this will help.
> The patch would completely bypass LPBC-based brightness controls if the
> backlight happened to be off when the driver started, which isn't
> exactly what you want.
>



  reply	other threads:[~2012-01-21 21:25 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-21 17:09 i915/kms/backlight-combo mode problem (was: Re: Linux 2.6.39-rc5) Chris Diamand
2012-01-21 18:32 ` Keith Packard
2012-01-21 21:25   ` Chris Diamand [this message]
2012-01-21 22:31     ` i915/kms/backlight-combo mode problem Keith Packard
2012-01-22 10:46       ` Chris Diamand
  -- strict thread matches above, loose matches on Subject: below --
2011-04-27  4:30 Linux 2.6.39-rc5 Linus Torvalds
     [not found] ` <s5htyd9kumz.wl%tiwai@suse.de>
2011-05-05 20:42   ` i915/kms/backlight-combo mode problem (was: Re: Linux 2.6.39-rc5) Melchior FRANZ
2011-05-06  8:52     ` i915/kms/backlight-combo mode problem Melchior FRANZ
2011-05-06 12:41       ` Joey Lee
2011-05-06 16:23         ` Melchior FRANZ
2011-05-07 20:22           ` Melchior FRANZ
2011-05-08 13:41             ` Joey Lee
2011-05-08 14:05               ` Melchior FRANZ
2011-05-09  8:50                 ` Joey Lee
2011-05-09  9:00                   ` Takashi Iwai
2011-05-09  9:35                     ` Joey Lee
2011-05-09 10:08                   ` Melchior FRANZ
     [not found]                     ` <BANLkTimtvAWeS-t3bWjJmdVtyaPDw2uWDA@mail.gmail.com>
2011-05-10  3:29                       ` Michael Chang
2011-05-10 11:08                         ` Melchior FRANZ
     [not found]                           ` <BANLkTimQJM3N1WdBhYVOH0UyyOTFcxOfUQ@mail.gmail.com>
2011-05-13 21:20                             ` Joey Lee
2011-05-15 12:50                               ` Melchior FRANZ
2011-05-15 10:08                           ` Takashi Iwai
2011-05-16 12:21                             ` Melchior FRANZ
2011-05-17  7:56                               ` Michael Chang
2011-05-17  8:58                                 ` Melchior FRANZ
2011-05-17 10:12                                   ` Michael Chang
2011-05-17 11:40                                     ` Melchior FRANZ
2011-05-09  8:58             ` 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=4F1B2D64.7020705@gmail.com \
    --to=chris.diamand@gmail.com \
    --cc=keithp@keithp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=melchior.franz@gmail.com \
    --cc=tiwai@suse.de \
    /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;
as well as URLs for NNTP newsgroup(s).