[coreboot-gerrit] Patch set updated for coreboot: aa5d337 w83627hf/acpi: Fix endianess error in floppy drive enumeration code

Christoph Grenz (christophg+cb@grenz-bonn.de) gerrit at coreboot.org
Thu Jul 4 03:56:30 CEST 2013


Christoph Grenz (christophg+cb at grenz-bonn.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3602

-gerrit

commit aa5d3373563760390f3c34a0d8d0366eff4eaefd
Author: Christoph Grenz <christophg+cb at grenz-bonn.de>
Date:   Thu Jul 4 02:51:42 2013 +0200

    w83627hf/acpi: Fix endianess error in floppy drive enumeration code
    
    The enumeration results are stored as five DWORDs in one 20 byte buffer.
    Bytes 3, 7, 11 and 15 were used to set the lowest bit of each DWORD.
    ACPI uses little endian, so 1, 4, 8 and 12 are the correct indices.
    
    Change-Id: I793225cb1bb62fd148ecfa1e61e02f5d7be62cdb
    Signed-off-by: Christoph Grenz <christophg+cb at grenz-bonn.de>
---
 src/superio/winbond/w83627hf/acpi/superio.asl | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/superio/winbond/w83627hf/acpi/superio.asl b/src/superio/winbond/w83627hf/acpi/superio.asl
index ff95dfc..5058d89 100644
--- a/src/superio/winbond/w83627hf/acpi/superio.asl
+++ b/src/superio/winbond/w83627hf/acpi/superio.asl
@@ -351,10 +351,10 @@ Device(SIO) {
 			}
 			OperationRegion (FIO2, SystemIO, 0x3F7, 0x01)
 
-			CreateByteField (_FDE, 3, FD1)
-			CreateByteField (_FDE, 7, FD2)
-			CreateByteField (_FDE, 11, FD3)
-			CreateByteField (_FDE, 15, FD4)
+			CreateByteField (_FDE, 1, FD1)
+			CreateByteField (_FDE, 4, FD2)
+			CreateByteField (_FDE, 8, FD3)
+			CreateByteField (_FDE, 12, FD4)
 
 			Store(One, ACT1)
 			Store(0, SELE)



More information about the coreboot-gerrit mailing list