ACPICA: For PM1B registers, do not shift value read or written
authorBob Moore <robert.moore@intel.com>
Wed, 18 Feb 2009 06:26:02 +0000 (14:26 +0800)
committerLen Brown <len.brown@intel.com>
Thu, 26 Mar 2009 20:38:25 +0000 (16:38 -0400)
The PM1B registers are mirrors of the PM1A registers with
different bits actually implemented. From the ACPI specification:
"Although the bits can be split between the two register blocks
(each register block has a unique pointer within the FADT), the bit
positions are maintained. The register block with unimplemented
bits (that is, those implemented in the other register block)
always returns zeros, and writes have no side effects"

Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/acpi/acpica/hwregs.c

index 9c8162128c2b943b2d970c6f13e699d367e9cdf3..5a64e577975f313d8c824efa1eb5a08f362e808c 100644 (file)
@@ -372,9 +372,17 @@ acpi_hw_read_multiple(u32 *value,
                }
        }
 
-       /* Shift the B bits above the A bits */
-
-       *value = value_a | (value_b << register_a->bit_width);
+       /*
+        * OR the two return values together. No shifting or masking is necessary,
+        * because of how the PM1 registers are defined in the ACPI specification:
+        *
+        * "Although the bits can be split between the two register blocks (each
+        * register block has a unique pointer within the FADT), the bit positions
+        * are maintained. The register block with unimplemented bits (that is,
+        * those implemented in the other register block) always returns zeros,
+        * and writes have no side effects"
+        */
+       *value = (value_a | value_b);
        return (AE_OK);
 }
 
@@ -406,13 +414,20 @@ acpi_hw_write_multiple(u32 value,
                return (status);
        }
 
-       /* Second register is optional */
-
+       /*
+        * Second register is optional
+        *
+        * No bit shifting or clearing is necessary, because of how the PM1
+        * registers are defined in the ACPI specification:
+        *
+        * "Although the bits can be split between the two register blocks (each
+        * register block has a unique pointer within the FADT), the bit positions
+        * are maintained. The register block with unimplemented bits (that is,
+        * those implemented in the other register block) always returns zeros,
+        * and writes have no side effects"
+        */
        if (register_b->address) {
-
-               /* Normalize the B bits before write */
-
-               status = acpi_write(value >> register_a->bit_width, register_b);
+               status = acpi_write(value, register_b);
        }
 
        return (status);