Change in coreboot[master]: nb/intel/ironlake: Update to ASL 2.0 syntax

[Angel Pons (Code Review)]

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43154 )

Change subject: nb/intel/ironlake: Update to ASL 2.0 syntax
......................................................................

nb/intel/ironlake: Update to ASL 2.0 syntax

Tested with BUILD_TIMELESS=1, Packard Bell MS2290 remains identical.

Change-Id: I2771407e5fa2dec5d2b554e53f7ad14cb4a4a4aa
Signed-off-by: Angel Pons 
---
M src/northbridge/intel/ironlake/acpi/hostbridge.asl
1 file changed, 28 insertions(+), 30 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/43154/1

diff --git a/src/northbridge/intel/ironlake/acpi/hostbridge.asl b/src/northbridge/intel/ironlake/acpi/hostbridge.asl
index 6a59fb5..324fb35 100644
--- a/src/northbridge/intel/ironlake/acpi/hostbridge.asl
+++ b/src/northbridge/intel/ironlake/acpi/hostbridge.asl
@@ -1,6 +1,5 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-
 Name(_HID,EISAID("PNP0A08"))	// PCIe
 Name(_CID,EISAID("PNP0A03"))	// PCI

@@ -92,17 +91,16 @@
    External (_SB.CP00._PSS)
    Method (PSSS, 1, NotSerialized)
    {
-		Store (One, Local0) /* Start at P1 */
-		Store (SizeOf (_SB.CP00._PSS), Local1)
+		Local0 = 1 /* Start at P1 */
+		Local1 = SizeOf (_SB.CP00._PSS)

-		While (LLess (Local0, Local1)) {
+		While (Local0 < Local1) {
    		/* Store _PSS entry Control value to Local2 */
-			ShiftRight (DeRefOf (Index (DeRefOf (Index
-			      (_SB.CP00._PSS, Local0)), 4)), 8, Local2)
-			If (LEqual (Local2, Arg0)) {
-				Return (Subtract (Local0, 1))
+			Local2 = DeRefOf (Index (DeRefOf (Index (_SB.CP00._PSS, Local0)), 4)) >> 8
+			If (Local2 == Arg0) {
+				Return (Local0 - 1)
    		}
-			Increment (Local0)
+			Local0++
    	}

Return (0)
@@ -122,23 +120,23 @@
    	Store ("Set TDP Down", Debug)

/* Set CTC */
-		Store (CTCD, CTCS)
+		CTCS = CTCD

/* Set TAR */
-		Store (TARD, TARS)
+		TARS = TARD

/* Set PPC limit and notify OS */
-		Store (PSSS (TARD), PPCM)
+		PPCM = PSSS (TARD)
    	PPCN ()

/* Set PL2 to 1.25 * PL1 */
-		Divide (Multiply (CTDD, 125), 100, , PL2V)
+		PL2V = (CTDD * 125) / 100

/* Set PL1 */
-		Store (CTDD, PL1V)
+		PL1V = CTDD

/* Store the new TDP Down setting */
-		Store (CTCD, CTCC)
+		CTCC = CTCD

Release (CTCM)
    	Return (1)
@@ -158,23 +156,23 @@
    	Store ("Set TDP Nominal", Debug)

/* Set PL1 */
-		Store (CTDN, PL1V)
+		PL1V = CTDN

/* Set PL2 to 1.25 * PL1 */
-		Divide (Multiply (CTDN, 125), 100, , PL2V)
+		PL2V = (CTDN * 125) / 100

/* Set PPC limit and notify OS */
-		Store (PSSS (TARN), PPCM)
+		PPCM = PSSS (TARN)
    	PPCN ()

/* Set TAR */
-		Store (TARN, TARS)
+		TARS = TARN

/* Set CTC */
-		Store (CTCN, CTCS)
+		CTCS = CTCN

/* Store the new TDP Nominal setting */
-		Store (CTCN, CTCC)
+		CTCC = CTCN

Release (CTCM)
    	Return (1)
@@ -305,20 +303,20 @@

// Fix up PCI memory region
    // Start with Top of Lower Usable DRAM
-	Store (^MCHC.TLUD, Local0)
-	ShiftRight (Local0, 4, Local0)
-	Store (^MCHC.TUUD, Local1)
+	Local0 = ^MCHC.TLUD
+	Local0 >>= 4
+	Local1 = ^MCHC.TUUD

// Check if ME base is equal
-	If (LEqual (Local0, Local1)) {
+	If (Local0 == Local1) {
    	// Use Top Of Memory instead
-		Store (^MCHC.TOM, Local0)
-		ShiftRight (Local0, 6, Local0)
+		Local0 = ^MCHC.TOM
+		Local0 >>= 6
    }

-	ShiftLeft (Local0, 20, Local0)
-	Store (Local0, PMIN)
-	Add(Subtract(PMAX, PMIN), 1, PLEN)
+	Local0 <<= 20
+	PMIN = Local0
+	PLEN = PMAX - PMIN + 1

Return (MCRS)
 }

[Angel Pons (Code Review)]

Angel Pons has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/43154 )

Change subject: nb/intel/ironlake: Update to ASL 2.0 syntax
......................................................................

Abandoned

Duplicate of CB:43687