[coreboot-gerrit] Patch set updated for coreboot: cff127b supermicro/h8qgi/dsdt: Use PIC as default interrupt model

Aladyshev Konstantin (aladyshev22@gmail.com) gerrit at coreboot.org
Tue Apr 8 17:35:30 CEST 2014


Aladyshev Konstantin (aladyshev22 at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5473

-gerrit

commit cff127bef23f7825467583cec9640bf0aba53ec7
Author: Konstantin Aladyshev <aladyshev at nicevt.ru>
Date:   Sat Jul 12 02:36:32 2014 +0400

    supermicro/h8qgi/dsdt: Use PIC as default interrupt model
    
    According ACPI specification:
    
    """
    The \_PIC optional method is used to report to the BIOS the current
    interrupt model used by the OS. The argument passed into the method
    signifies the interrupt model OSPM has chosen, PIC mode, APIC mode,
    or SAPIC mode. Notice that calling this method is optional for OSPM.
    If the method is never called, the BIOS must assume PIC mode.
    
    Arguments: (1)
    Arg0 – An Integer containing a code for the current interrupt model:
    0 –PIC mode
    1 –APIC mode
    2 –SAPIC mode
    """
    
    In current configuration with default value of interrupt model
    PMOD equal 1 (APIC mode), Linux can't boot with "noapic" option.
    Kernel never call _PIC method and PMOD stays equal 1, indicatind
    that APIC routing objects should be evaluated. This mix of PIC
    and APIC leads to boot fail.
    
    Change default value of interrupt model PMOD to 0, for correct
    "noapic" boot.
    
    Change-Id: I7fa6f0c24802751202ed2e7f13411001a600e772
    Signed-off-by: Konstantin Aladyshev <aladyshev at nicevt.ru>
---
 src/mainboard/supermicro/h8qgi/dsdt.asl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/supermicro/h8qgi/dsdt.asl b/src/mainboard/supermicro/h8qgi/dsdt.asl
index c854067..b829584 100644
--- a/src/mainboard/supermicro/h8qgi/dsdt.asl
+++ b/src/mainboard/supermicro/h8qgi/dsdt.asl
@@ -57,7 +57,7 @@ DefinitionBlock (
 	/* Some global data */
 	Name(OSTP, 3)		/* Assume nothing. WinXp = 1, Vista = 2, Linux = 3, WinCE = 4 */
 	Name(OSV, Ones)	/* Assume nothing */
-	Name(PMOD, One)	/* Assume APIC */
+	Name(PMOD, 0)	/* Default interrupt model is PIC */
 
 	/*
 	 * Processor Object



More information about the coreboot-gerrit mailing list