x86, quirk: Fix SB600 revision check
authorAndreas Herrmann <andreas.herrmann3@amd.com>
Tue, 15 Mar 2011 14:31:37 +0000 (15:31 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 16 Mar 2011 13:03:32 +0000 (14:03 +0100)
Commit 7f74f8f28a2bd9db9404f7d364e2097a0c42cc12
(x86 quirk: Fix polarity for IRQ0 pin2 override on SB800
systems) introduced a regression. It removed some SB600 specific
code to determine the revision ID without adapting a
corresponding revision ID check for SB600.

See this mail thread:

  http://marc.info/?l=linux-kernel&m=129980296006380&w=2

This patch adapts the corresponding check to cover all SB600
revisions.

Tested-by: Wang Lei <f3d27b@gmail.com>
Signed-off-by: Andreas Herrmann <andreas.herrmann3@amd.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: stable@kernel.org # 38.x, 37.x, 32.x
LKML-Reference: <20110315143137.GD29499@alberich.amd.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/early-quirks.c

index 9efbdcc..3755ef4 100644 (file)
@@ -159,7 +159,12 @@ static void __init ati_bugs_contd(int num, int slot, int func)
        if (rev >= 0x40)
                acpi_fix_pin2_polarity = 1;
 
-       if (rev > 0x13)
+       /*
+        * SB600: revisions 0x11, 0x12, 0x13, 0x14, ...
+        * SB700: revisions 0x39, 0x3a, ...
+        * SB800: revisions 0x40, 0x41, ...
+        */
+       if (rev >= 0x39)
                return;
 
        if (acpi_use_timer_override)