From 3e41ee9847f0acd4a57c0c4cb11156eace16fd05 Mon Sep 17 00:00:00 2001
From: Martin Roth <martinroth@google.com>
Date: Tue, 8 Dec 2015 15:24:41 -0700
Subject: ACPI: Fix IASL Warning about unused method for _S3 check

According to the ACPI Spec for CondRefOf, the result argument is
optional.  In all of these locations, it was getting set but not
used, creating a warning in new versions of IASL.  Since it's
an optional argument, just remove it.

dsdt.aml 640:              If (CondRefOf (\_S3, Local0))
Warning  3144 - Method Local is set but never used ^  (Local0)

Change-Id: I758d198c33e585a6a4ad2c1c70f2370a01af5138
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12693
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
---
 src/mainboard/amd/serengeti_cheetah/acpi/amd8111.asl             | 6 +++---
 src/mainboard/amd/serengeti_cheetah/acpi/amd8131.asl             | 4 ++--
 src/mainboard/amd/serengeti_cheetah/acpi/amd8131_2.asl           | 4 ++--
 src/mainboard/amd/serengeti_cheetah/acpi/amd8132_2.asl           | 4 ++--
 src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8111.asl       | 6 +++---
 src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8131_2.asl     | 4 ++--
 src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132.asl       | 4 ++--
 src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132_2.asl     | 4 ++--
 src/mainboard/amd/serengeti_cheetah_fam10/acpi/htx_no_ioapic.asl | 2 +-
 src/mainboard/hp/dl145_g1/acpi/amd8111.asl                       | 8 ++++----
 src/mainboard/hp/dl145_g1/acpi/amd8131.asl                       | 4 ++--
 src/mainboard/iwill/dk8_htx/acpi/amd8111.asl                     | 6 +++---
 src/mainboard/iwill/dk8_htx/acpi/amd8131.asl                     | 4 ++--
 src/mainboard/iwill/dk8_htx/acpi/amd8131_2.asl                   | 4 ++--
 src/mainboard/iwill/dk8_htx/acpi/amd8132_2.asl                   | 4 ++--
 src/mainboard/iwill/dk8_htx/acpi/htx_no_ioapic.asl               | 2 +-
 16 files changed, 35 insertions(+), 35 deletions(-)

(limited to 'src')

diff --git a/src/mainboard/amd/serengeti_cheetah/acpi/amd8111.asl b/src/mainboard/amd/serengeti_cheetah/acpi/amd8111.asl
index 7509a340d4ad..6e98c248f396 100644
--- a/src/mainboard/amd/serengeti_cheetah/acpi/amd8111.asl
+++ b/src/mainboard/amd/serengeti_cheetah/acpi/amd8111.asl
@@ -103,7 +103,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x08, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x08, 0x03 }) }
                     Else { Return (Package (0x02) { 0x08, 0x01 }) }
                 }
 
@@ -112,7 +112,7 @@
                     Name (_ADR, 0x00000000)
                     Method (_PRW, 0, NotSerialized)
                     {
-                        If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+                        If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
                         Else { Return (Package (0x02) { 0x0F, 0x01 }) }
                     }
                 }
@@ -122,7 +122,7 @@
                     Name (_ADR, 0x00000001)
                     Method (_PRW, 0, NotSerialized)
                     {
-                        If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+                        If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
                         Else { Return (Package (0x02) { 0x0F, 0x01 }) }
                     }
                 }
diff --git a/src/mainboard/amd/serengeti_cheetah/acpi/amd8131.asl b/src/mainboard/amd/serengeti_cheetah/acpi/amd8131.asl
index 172f0bf9d161..ba77557a2168 100644
--- a/src/mainboard/amd/serengeti_cheetah/acpi/amd8131.asl
+++ b/src/mainboard/amd/serengeti_cheetah/acpi/amd8131.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -92,7 +92,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/amd/serengeti_cheetah/acpi/amd8131_2.asl b/src/mainboard/amd/serengeti_cheetah/acpi/amd8131_2.asl
index 8b8bc9fab9c8..6eb95ec72485 100644
--- a/src/mainboard/amd/serengeti_cheetah/acpi/amd8131_2.asl
+++ b/src/mainboard/amd/serengeti_cheetah/acpi/amd8131_2.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -68,7 +68,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/amd/serengeti_cheetah/acpi/amd8132_2.asl b/src/mainboard/amd/serengeti_cheetah/acpi/amd8132_2.asl
index e5cfe3c951d9..9c0629f779b5 100644
--- a/src/mainboard/amd/serengeti_cheetah/acpi/amd8132_2.asl
+++ b/src/mainboard/amd/serengeti_cheetah/acpi/amd8132_2.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -68,7 +68,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8111.asl b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8111.asl
index 094adff698d1..46fe1216b8bc 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8111.asl
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8111.asl
@@ -107,7 +107,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x08, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x08, 0x03 }) }
 			Else { Return (Package (0x02) { 0x08, 0x01 }) }
 		}
 
@@ -116,7 +116,7 @@
 			Name (_ADR, 0x00000000)
 			Method (_PRW, 0, NotSerialized)
 			{
-				If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+				If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
 				Else { Return (Package (0x02) { 0x0F, 0x01 }) }
 			}
 		}
@@ -126,7 +126,7 @@
 			Name (_ADR, 0x00000001)
 			Method (_PRW, 0, NotSerialized)
 			{
-				If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+				If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
 				Else { Return (Package (0x02) { 0x0F, 0x01 }) }
 			}
 		}
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8131_2.asl b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8131_2.asl
index 9139853fd085..8e40ab0101c2 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8131_2.asl
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8131_2.asl
@@ -22,7 +22,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
 			Else { Return (Package (0x02) { 0x29, 0x01 }) }
 		}
 
@@ -75,7 +75,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
 			Else { Return (Package (0x02) { 0x22, 0x01 }) }
 		}
 
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132.asl b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132.asl
index ba0b4e207c4e..e4167e21a861 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132.asl
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132.asl
@@ -23,7 +23,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
 			Else { Return (Package (0x02) { 0x29, 0x01 }) }
 		}
 
@@ -103,7 +103,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
 			Else { Return (Package (0x02) { 0x22, 0x01 }) }
 		}
 
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132_2.asl b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132_2.asl
index 8753f2b23bc7..30a3fcf9ac22 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132_2.asl
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/amd8132_2.asl
@@ -23,7 +23,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
 			Else { Return (Package (0x02) { 0x29, 0x01 }) }
 		}
 
@@ -110,7 +110,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
 			Else { Return (Package (0x02) { 0x22, 0x01 }) }
 		}
 
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/htx_no_ioapic.asl b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/htx_no_ioapic.asl
index aae917173d5c..9caa3b2466c1 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi/htx_no_ioapic.asl
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi/htx_no_ioapic.asl
@@ -23,7 +23,7 @@
 
 		Method (_PRW, 0, NotSerialized)
 		{
-			If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+			If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
 			Else { Return (Package (0x02) { 0x29, 0x01 }) }
 		}
 
diff --git a/src/mainboard/hp/dl145_g1/acpi/amd8111.asl b/src/mainboard/hp/dl145_g1/acpi/amd8111.asl
index aa136ffbf2bb..77389f006267 100644
--- a/src/mainboard/hp/dl145_g1/acpi/amd8111.asl
+++ b/src/mainboard/hp/dl145_g1/acpi/amd8111.asl
@@ -87,7 +87,7 @@ Device (TP2P) {
 	// result :
 	// [0] Bit index into GPEx_EN in the GPE block described by FADT.
 	// [1] The lowest power state from which the system can be awakened.
-		//If (CondRefOf (\_S3, Local0)) {
+		//If (CondRefOf (\_S3)) {
 		//	Return (Package (0x02) { 0x08, 0x03 })
 		//} Else {
 			Return (Package (0x02) { 0x08, 0x01 })
@@ -96,7 +96,7 @@ Device (TP2P) {
 	Device (ETHR) {
 		Name (_ADR, 0x00010000)
 		Method (_PRW, 0, NotSerialized) { // Power Resource for Wake
-			//If (CondRefOf (\_S3, Local0)) {
+			//If (CondRefOf (\_S3)) {
 			//	Return (Package (0x02) { 0x08, 0x03 })
 			//} Else {
 				Return (Package (0x02) { 0x08, 0x01 })
@@ -109,7 +109,7 @@ Device (TP2P) {
 			And (GNBL, 0x7FFF, GNBL)
 		}
 		Method (_PRW, 0, NotSerialized) { // Power Resource for Wake
-			//If (CondRefOf (\_S3, Local0)) {
+			//If (CondRefOf (\_S3)) {
 			//	Return (Package (0x02) { 0x0F, 0x03 })
 			//} Else {
 				Return (Package (0x02) { 0x0F, 0x01 })
@@ -122,7 +122,7 @@ Device (TP2P) {
 			And (GNBL, 0x7FFF, GNBL)
 		}
 		Method (_PRW, 0, NotSerialized) { // Power Resource for Wake
-			//If (CondRefOf (\_S3, Local0)) {
+			//If (CondRefOf (\_S3)) {
 			//	Return (Package (0x02) { 0x0F, 0x03 })
 			//} Else {
 				Return (Package (0x02) { 0x0F, 0x01 })
diff --git a/src/mainboard/hp/dl145_g1/acpi/amd8131.asl b/src/mainboard/hp/dl145_g1/acpi/amd8131.asl
index 54aae73affd1..29808aabd9ab 100644
--- a/src/mainboard/hp/dl145_g1/acpi/amd8131.asl
+++ b/src/mainboard/hp/dl145_g1/acpi/amd8131.asl
@@ -9,7 +9,7 @@ Device (PG0A) {
 		Return (DADD(GHCD(HCIN, 0), 0x00000000))
 	}
 	Method (_PRW, 0, NotSerialized) {
-		//If (CondRefOf (\_S3, Local0)) {
+		//If (CondRefOf (\_S3)) {
 		//	Return (Package (0x02) { 0x29, 0x03 })
 		//} Else {
 			Return (Package (0x02) { 0x29, 0x01 })
@@ -75,7 +75,7 @@ Device (PG0B) {
 		Return (DADD(GHCD(HCIN, 0), 0x00010000))
 	}
 	Method (_PRW, 0, NotSerialized) {
-		//If (CondRefOf (\_S3, Local0)) {
+		//If (CondRefOf (\_S3)) {
 		//	Return (Package (0x02) { 0x22, 0x03 })
 		//} Else {
 			Return (Package (0x02) { 0x22, 0x01 })
diff --git a/src/mainboard/iwill/dk8_htx/acpi/amd8111.asl b/src/mainboard/iwill/dk8_htx/acpi/amd8111.asl
index a470e89bf1e5..4d41c6079a10 100644
--- a/src/mainboard/iwill/dk8_htx/acpi/amd8111.asl
+++ b/src/mainboard/iwill/dk8_htx/acpi/amd8111.asl
@@ -103,7 +103,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x08, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x08, 0x03 }) }
                     Else { Return (Package (0x02) { 0x08, 0x01 }) }
                 }
 
@@ -112,7 +112,7 @@
                     Name (_ADR, 0x00000000)
                     Method (_PRW, 0, NotSerialized)
                     {
-                        If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+                        If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
                         Else { Return (Package (0x02) { 0x0F, 0x01 }) }
                     }
                 }
@@ -122,7 +122,7 @@
                     Name (_ADR, 0x00000001)
                     Method (_PRW, 0, NotSerialized)
                     {
-                        If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x0F, 0x03 }) }
+                        If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x0F, 0x03 }) }
                         Else { Return (Package (0x02) { 0x0F, 0x01 }) }
                     }
                 }
diff --git a/src/mainboard/iwill/dk8_htx/acpi/amd8131.asl b/src/mainboard/iwill/dk8_htx/acpi/amd8131.asl
index dd82e38df547..6d93e020484e 100644
--- a/src/mainboard/iwill/dk8_htx/acpi/amd8131.asl
+++ b/src/mainboard/iwill/dk8_htx/acpi/amd8131.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -92,7 +92,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/iwill/dk8_htx/acpi/amd8131_2.asl b/src/mainboard/iwill/dk8_htx/acpi/amd8131_2.asl
index 8b8bc9fab9c8..6eb95ec72485 100644
--- a/src/mainboard/iwill/dk8_htx/acpi/amd8131_2.asl
+++ b/src/mainboard/iwill/dk8_htx/acpi/amd8131_2.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -68,7 +68,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/iwill/dk8_htx/acpi/amd8132_2.asl b/src/mainboard/iwill/dk8_htx/acpi/amd8132_2.asl
index e5cfe3c951d9..9c0629f779b5 100644
--- a/src/mainboard/iwill/dk8_htx/acpi/amd8132_2.asl
+++ b/src/mainboard/iwill/dk8_htx/acpi/amd8132_2.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
@@ -68,7 +68,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x22, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x22, 0x03 }) }
                     Else { Return (Package (0x02) { 0x22, 0x01 }) }
                 }
 
diff --git a/src/mainboard/iwill/dk8_htx/acpi/htx_no_ioapic.asl b/src/mainboard/iwill/dk8_htx/acpi/htx_no_ioapic.asl
index 01625650c0c9..7301fef99896 100644
--- a/src/mainboard/iwill/dk8_htx/acpi/htx_no_ioapic.asl
+++ b/src/mainboard/iwill/dk8_htx/acpi/htx_no_ioapic.asl
@@ -12,7 +12,7 @@
 
                 Method (_PRW, 0, NotSerialized)
                 {
-                    If (CondRefOf (\_S3, Local0)) { Return (Package (0x02) { 0x29, 0x03 }) }
+                    If (CondRefOf (\_S3)) { Return (Package (0x02) { 0x29, 0x03 }) }
                     Else { Return (Package (0x02) { 0x29, 0x01 }) }
                 }
 
-- 
cgit v1.2.3