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

Christoph Grenz (christophg+cb@grenz-bonn.de) gerrit at coreboot.org
Sat Jul 6 17:25:07 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 c25869764dd514d4b969e31804b6bc6173e56ee6
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 427d63c..8531339 100644
--- a/src/superio/winbond/w83627hf/acpi/superio.asl
+++ b/src/superio/winbond/w83627hf/acpi/superio.asl
@@ -555,7 +555,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