[coreboot-gerrit] Patch set updated for coreboot: AGESA binaryPI: Remove unused IMC ACPI methods IMSP and IMWK

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Sat Mar 4 10:25:48 CET 2017


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18537

-gerrit

commit a1e9cc82fc3d8f8357b9614fa1fccf48884681ec
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Sat Jan 28 10:17:38 2017 +0200

    AGESA binaryPI: Remove unused IMC ACPI methods IMSP and IMWK
    
    Note that IMC must sleep while SPI writes are in progress.
    Instead of using these ACPI methods, flashrom currently does
    raw IO to achieve the same.
    
    Change-Id: Ifca4e8328c54d1074b4799ddecfece24607214db
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/mainboard/amd/bettong/acpi/AmdImc.asl       | 44 -------------------------
 src/mainboard/amd/db-ft3b-lc/acpi/AmdImc.asl    | 44 -------------------------
 src/mainboard/amd/gardenia/acpi/AmdImc.asl      | 44 -------------------------
 src/mainboard/amd/lamar/acpi/AmdImc.asl         | 44 -------------------------
 src/mainboard/amd/olivehill/acpi/AmdImc.asl     | 44 -------------------------
 src/mainboard/amd/olivehillplus/acpi/AmdImc.asl | 44 -------------------------
 src/mainboard/amd/parmer/acpi/AmdImc.asl        | 44 -------------------------
 src/mainboard/amd/thatcher/acpi/AmdImc.asl      | 44 -------------------------
 8 files changed, 352 deletions(-)

diff --git a/src/mainboard/amd/bettong/acpi/AmdImc.asl b/src/mainboard/amd/bettong/acpi/AmdImc.asl
index 970858d..a9190da 100644
--- a/src/mainboard/amd/bettong/acpi/AmdImc.asl
+++ b/src/mainboard/amd/bettong/acpi/AmdImc.asl
@@ -63,48 +63,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/db-ft3b-lc/acpi/AmdImc.asl b/src/mainboard/amd/db-ft3b-lc/acpi/AmdImc.asl
index d3bb74e..9426dfa 100644
--- a/src/mainboard/amd/db-ft3b-lc/acpi/AmdImc.asl
+++ b/src/mainboard/amd/db-ft3b-lc/acpi/AmdImc.asl
@@ -62,48 +62,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/gardenia/acpi/AmdImc.asl b/src/mainboard/amd/gardenia/acpi/AmdImc.asl
index 970858d..a9190da 100644
--- a/src/mainboard/amd/gardenia/acpi/AmdImc.asl
+++ b/src/mainboard/amd/gardenia/acpi/AmdImc.asl
@@ -63,48 +63,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/lamar/acpi/AmdImc.asl b/src/mainboard/amd/lamar/acpi/AmdImc.asl
index 26f7d61..2bb4426 100644
--- a/src/mainboard/amd/lamar/acpi/AmdImc.asl
+++ b/src/mainboard/amd/lamar/acpi/AmdImc.asl
@@ -64,48 +64,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/olivehill/acpi/AmdImc.asl b/src/mainboard/amd/olivehill/acpi/AmdImc.asl
index aa941ba..621db82 100644
--- a/src/mainboard/amd/olivehill/acpi/AmdImc.asl
+++ b/src/mainboard/amd/olivehill/acpi/AmdImc.asl
@@ -63,48 +63,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/olivehillplus/acpi/AmdImc.asl b/src/mainboard/amd/olivehillplus/acpi/AmdImc.asl
index d3bb74e..9426dfa 100644
--- a/src/mainboard/amd/olivehillplus/acpi/AmdImc.asl
+++ b/src/mainboard/amd/olivehillplus/acpi/AmdImc.asl
@@ -62,48 +62,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/parmer/acpi/AmdImc.asl b/src/mainboard/amd/parmer/acpi/AmdImc.asl
index 5e6c02b..186f9b7 100644
--- a/src/mainboard/amd/parmer/acpi/AmdImc.asl
+++ b/src/mainboard/amd/parmer/acpi/AmdImc.asl
@@ -63,48 +63,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }
diff --git a/src/mainboard/amd/thatcher/acpi/AmdImc.asl b/src/mainboard/amd/thatcher/acpi/AmdImc.asl
index 5e6c02b..186f9b7 100644
--- a/src/mainboard/amd/thatcher/acpi/AmdImc.asl
+++ b/src/mainboard/amd/thatcher/acpi/AmdImc.asl
@@ -63,48 +63,4 @@ Method (ITZE, 0)
 	WACK()
 }
 
-//Sleep
-Method (IMSP, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(1, MRG1)
-	Store(0, MRG2)
-	Store(0x98, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0xB4, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-}
-
-//Wake
-Method (IMWK, 0)
-{
-	Store(0, MRG0)
-	Store(0xB5, MRG1)
-	Store(0, MRG2)
-	Store(0x96, MSTI)
-	WACK()
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(0, MRG2)
-	Store(0x80, MSTI)
-	WACK()
-
-	Or(MRG2, 0x01, Local0)
-
-	Store(0, MRG0)
-	Store(0, MRG1)
-	Store(Local0, MRG2)
-	Store(0x81, MSTI)
-	WACK()
 }



More information about the coreboot-gerrit mailing list