Linux ACPI
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: Zhang Rui <rui.zhang@intel.com>
Cc: Yves-Alexis Perez <corsac@debian.org>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"dri-devel@lists.sourceforge.net"
	<dri-devel@lists.sourceforge.net>,
	"jbarnes@virtuousgeek.org" <jbarnes@virtuousgeek.org>,
	Thomas Renninger <trenn@suse.de>,
	Matthew Garrett <mjg@redhat.com>,
	Keith Packard <keithp@keithp.com>
Subject: Re: [intel/acpi] backlight - T61
Date: Sun, 30 Nov 2008 22:18:33 -0500 (EST)	[thread overview]
Message-ID: <alpine.LFD.2.00.0811302158150.3314@localhost.localdomain> (raw)
In-Reply-To: <1228095176.2713.18.camel@rzhang-dt>



> IMO, after loading i915 driver, the ACPI backlight I/F can work again,
> right?

No.
hotkeys have never worked in console mode,
but in 2.6.27, echo into /proc and /sys did work in console mode
and in 2.6.28 they do not.  So this is a regression.

> If this is true, why not always loading i915 both in X and console mode?
> the kernel mode setting should not only work for X mode.

I agree that the i915 driver should handle brightness, no matter if we
are in GUI mode or console mode.  Unfortunately, today it
doesn't seem to do anything in console mode.

That is why I pointed out that the regression here was
when we deleted the /proc and /sys nodes that actualy work
from console mode...

On a brightner note, I'm pleased to report that after upgrading
my T61 to FC10 from FC9, the brightness suddenly became consistent
across screen saver events.  ie. the brightness I set via hotkey
before the screen saver event was restored after the screen
comes back from screen blank.

However, testing FC10 with the latest upstream kernel,
including the i915 changes that appeared today...
it seems that GUI mode forgets the brighness when
I switch to console mode.  Eg. If I use /sys or
hotkeys in GUI mode to reduce brightness to 0,
it stays dim when I switch to console mode,
but switches back to max brightness when
I switch back to GUI mode.

-Len



  reply	other threads:[~2008-12-01  3:19 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-28  6:47 [intel/acpi] backlight Yves-Alexis Perez
2008-11-28 19:18 ` [intel/acpi] backlight - T61 Len Brown
2008-11-28 22:38   ` Thomas Renninger
2008-11-28 22:50   ` Yves-Alexis Perez
2008-12-01  1:32     ` Zhang Rui
2008-12-01  3:18       ` Len Brown [this message]
2008-12-01  5:08         ` Zhang Rui
2008-12-03 23:14           ` Zhenyu Wang
2008-12-04  2:50             ` [Intel-gfx] " Zhenyu Wang
2008-12-09 21:24             ` Jesse Barnes
2008-12-01  1:08 ` [intel/acpi] backlight Zhao Yakui
2008-12-01  3:28   ` Len Brown
2008-12-01  8:38     ` Matthew Garrett
2008-12-01 19:27       ` Jesse Barnes
2008-12-01 19:32         ` Matthew Garrett
2008-12-01 11:11   ` Yves-Alexis Perez

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=alpine.LFD.2.00.0811302158150.3314@localhost.localdomain \
    --to=lenb@kernel.org \
    --cc=corsac@debian.org \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=jbarnes@virtuousgeek.org \
    --cc=keithp@keithp.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=mjg@redhat.com \
    --cc=rui.zhang@intel.com \
    --cc=trenn@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