diff mbox

thinkpad-acpi: don't ask about brightness_mode for fw. 1V and 1R

Message ID 1250440221-8185-1-git-send-email-hmh@hmh.eng.br (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Henrique de Moraes Holschuh Aug. 16, 2009, 4:30 p.m. UTC
X40 (firmware 1V) and T41 (firmware 1R) have been confirmed to work
well with the new defaults, so we can stop pestering people to confirm
that fact.

For now, whitelist just these two firmware types.  It is best to have
at least one more firmware type confirmed for Radeon 9xxx and Intel
GMA-2 ThinkPads before removing the confirmation requests entirely.

Reported-by: Robert de Rooy <robert.de.rooy@gmail.com>
Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
---
 drivers/platform/x86/thinkpad_acpi.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

Comments

Henrique de Moraes Holschuh Aug. 23, 2009, 12:54 p.m. UTC | #1
On Sun, 16 Aug 2009, Henrique de Moraes Holschuh wrote:
> X40 (firmware 1V) and T41 (firmware 1R) have been confirmed to work
> well with the new defaults, so we can stop pestering people to confirm
> that fact.

Len, if this could make it to mainline before the kernel is released, I
would be grateful.

It whitelists a large class of machines (all T4x models, and the X40 and
X41).
Henrique de Moraes Holschuh Aug. 28, 2009, 10:02 p.m. UTC | #2
On Sun, 23 Aug 2009, Henrique de Moraes Holschuh wrote:
> On Sun, 16 Aug 2009, Henrique de Moraes Holschuh wrote:
> > X40 (firmware 1V) and T41 (firmware 1R) have been confirmed to work
> > well with the new defaults, so we can stop pestering people to confirm
> > that fact.
> 
> Len, if this could make it to mainline before the kernel is released, I
> would be grateful.
> 
> It whitelists a large class of machines (all T4x models, and the X40 and
> X41).

Ping!
diff mbox

Patch

diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
index 0acc0c9..df5364c 100644
--- a/drivers/platform/x86/thinkpad_acpi.c
+++ b/drivers/platform/x86/thinkpad_acpi.c
@@ -5627,16 +5627,16 @@  static const struct tpacpi_quirk brightness_quirk_table[] __initconst = {
 	/* Models with ATI GPUs known to require ECNVRAM mode */
 	TPACPI_Q_IBM('1', 'Y', TPACPI_BRGHT_Q_EC),	/* T43/p ATI */
 
-	/* Models with ATI GPUs (waiting confirmation) */
-	TPACPI_Q_IBM('1', 'R', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_EC),
+	/* Models with ATI GPUs that can use ECNVRAM */
+	TPACPI_Q_IBM('1', 'R', TPACPI_BRGHT_Q_EC),
 	TPACPI_Q_IBM('1', 'Q', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_EC),
 	TPACPI_Q_IBM('7', '6', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_EC),
 	TPACPI_Q_IBM('7', '8', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_EC),
 
-	/* Models with Intel Extreme Graphics 2 (waiting confirmation) */
+	/* Models with Intel Extreme Graphics 2 */
+	TPACPI_Q_IBM('1', 'U', TPACPI_BRGHT_Q_NOEC),
 	TPACPI_Q_IBM('1', 'V', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_NOEC),
 	TPACPI_Q_IBM('1', 'W', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_NOEC),
-	TPACPI_Q_IBM('1', 'U', TPACPI_BRGHT_Q_ASK|TPACPI_BRGHT_Q_NOEC),
 
 	/* Models with Intel GMA900 */
 	TPACPI_Q_IBM('7', '0', TPACPI_BRGHT_Q_NOEC),	/* T43, R52 */