From: Takashi Iwai <tiwai@suse.de>
To: Melchior FRANZ <melchior.franz@gmail.com>
Cc: linux-kernel@vger.kernel.org, Michael Chang <mchang@novell.com>,
Joey Lee <jlee@novell.com>,
chris@chris-wilson.co.uk, dri-devel@lists.freedesktop.org
Subject: Re: i915/kms/backlight-combo mode problem
Date: Sun, 15 May 2011 12:08:48 +0200 [thread overview]
Message-ID: <s5h7h9s46of.wl%tiwai@suse.de> (raw)
In-Reply-To: <201105101308.24662@rk-nord.at>
At Tue, 10 May 2011 13:08:23 +0200,
Melchior FRANZ wrote:
>
> * Michael Chang -- Tuesday 10 May 2011:
> > Could you please try this patch and get the log ? We wonder why
> > is_backlight_combination_mode () returns false.
>
> This information was already buried in the bugzilla thread:
>
> https://bugzilla.kernel.org/show_bug.cgi?id=31522
> "It turned out that on this machine INTEL_INFO(dev)->gen equals 4,
> and is_backlight_combination_mode() returns 0x40000000."
>
>
> But to say it again in your words: :-)
>
> [drm:is_backlight_combination_mode], BLM_COMBINATION_MODE = 1073741824 (0x40000000)
>
> 6x during boot-up, and several times later when changing the backlight
> brightness.
>
>
> This was with 8b061610dac3a3b89770c85ad63b481a47b0c38e. And now
> I have a little shocker for you (and me): because this was a
> vanilla kernel (apart from these debug messages), the screen went
> black again, like I knew it. But pressing the "brightness down"
> key turns the backlight on! I can't believe that I haven't tested
> that. I guess I've only tried "brightness up" and "display toggle".
> Those don't turn backlight on. Or maybe somethine else relevant
> meanwhile changed in the i915 drivers. (I've regularly been
> updating to HEAD.)
>
> So, the problem was just the initial state all the time?
Looks so, indeed. Now, the question is what's the real cause.
IIRC, you reported that the backlight gets normal when you revert my
commit in 2.6.38.x. So, this was regarded as a regression at first.
But, one question remains: whether the backlight level control worked
with the reverted kernel?
Judging from the attempts so far, it looks like that only LBPC can
adjust the level on your machine. If it's true, 2.6.38.0 shouldn't be
able to adjust the level. If you can still change the level without
LBPC, the former analysis was incorrect.
Also, with the latest 2.6.38.x, you found that the backlight gets back
when you adjust the level down. Another question now is what happens
if you again turn it up to the max level. Is the backlight still on?
If the backlight is kept on even with the max level, it implies that
the problem is only the initial value; once when set correctly, it'll
work fine after that. OTOH, if the backlight gets off again at max,
it means that the max value (LBPC 0xfe) is a sort of out-of-range.
Then LBPC calculation in the driver has to be modified.
thanks,
Takashi
next prev parent reply other threads:[~2011-05-15 10:08 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-27 4:30 Linux 2.6.39-rc5 Linus Torvalds
2011-04-29 15:58 ` Melchior FRANZ
2011-04-29 17:09 ` Takashi Iwai
2011-04-29 17:41 ` i915/kms/backlight-combo mode problem (was: Re: Linux 2.6.39-rc5) Melchior FRANZ
2011-04-29 19:47 ` Takashi Iwai
2011-04-29 20:09 ` Melchior FRANZ
2011-04-30 8:11 ` Takashi Iwai
2011-04-30 8:32 ` Melchior FRANZ
2011-04-30 10:27 ` Takashi Iwai
2011-04-30 11:34 ` Melchior FRANZ
2011-04-30 14:37 ` Joey Lee
2011-05-02 12:08 ` Takashi Iwai
2011-05-02 13:12 ` Melchior FRANZ
2011-05-04 7:07 ` Joey Lee
[not found] ` <201105051745.27992@rk-nord.at>
[not found] ` <s5htyd9kumz.wl%tiwai@suse.de>
2011-05-05 20:42 ` 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 [this message]
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
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 ` i915/kms/backlight-combo mode problem Chris Diamand
2012-01-21 22:31 ` Keith Packard
2012-01-22 10:46 ` Chris Diamand
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=s5h7h9s46of.wl%tiwai@suse.de \
--to=tiwai@suse.de \
--cc=chris@chris-wilson.co.uk \
--cc=dri-devel@lists.freedesktop.org \
--cc=jlee@novell.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mchang@novell.com \
--cc=melchior.franz@gmail.com \
/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
all inboxes | Powered by JetHome®