[coreboot-gerrit] Patch set updated for coreboot: 493451a w83627hf/acpi: Make `AddressMax` a multiple of `AddressAlignment`

Christoph Grenz (christophg+cb@grenz-bonn.de) gerrit at coreboot.org
Thu Jul 11 02:36:56 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/2942

-gerrit

commit 493451ace545b1ccaa51e9c4b3776cfc533dbc88
Author: Christoph Grenz <christophg+cb at grenz-bonn.de>
Date:   Sat Jul 6 17:09:39 2013 +0200

    w83627hf/acpi: Make `AddressMax` a multiple of `AddressAlignment`
    
    The parallel port of the W83627HF can be configured on any port
    between 0x100 and 0xFFC with 4 byte alignment for traditional modes
    and 8 byte alignment for EPP mode. As the ACPI specification says
    that the maximum acceptable starting address has to be a multiple
    of the alignment granularity, correct the maximum starting address
    from 0xFFC to 0xFF8.
    
    Change-Id: I272e09d091149791f2867b1d06e4fc27bc1bb2cd
    Signed-off-by: Christoph Grenz <christophg+cb at grenz-bonn.de>
---
 src/superio/winbond/w83627hf/acpi/superio.asl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/superio/winbond/w83627hf/acpi/superio.asl b/src/superio/winbond/w83627hf/acpi/superio.asl
index 05b570b..8f68c70 100644
--- a/src/superio/winbond/w83627hf/acpi/superio.asl
+++ b/src/superio/winbond/w83627hf/acpi/superio.asl
@@ -564,7 +564,7 @@ Device(SIO) {
 			/* Any configurable address (EPP mode) */
 			StartDependentFn (2,0)
 			{
-				IO (Decode16, 0x0100, 0x0FFC, 0x08, 0x08)
+				IO (Decode16, 0x0100, 0x0FF8, 0x08, 0x08)
 				IRQNoFlags () {3,4,5,7,9,10,11,12}
 			}
 			/* Any configurable address (No EPP mode) */



More information about the coreboot-gerrit mailing list