From a34b5bc6ed972dd1720345011b07e4a97b9e4594 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sun, 6 May 2018 21:06:18 -0500 Subject: southbridge/intel/bd82x6x: Remove unused argument from ACPI method The method POSC was only using 2 of the 3 arguments passed in to it. Remove the unused argument. Change-Id: I6bbc2a034c79581fd338276eea56aac6d1affa58 Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/26124 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- src/southbridge/intel/bd82x6x/acpi/pch.asl | 8 +++++++- src/southbridge/intel/bd82x6x/acpi/usb.asl | 13 +++++++++---- 2 files changed, 16 insertions(+), 5 deletions(-) (limited to 'src') diff --git a/src/southbridge/intel/bd82x6x/acpi/pch.asl b/src/southbridge/intel/bd82x6x/acpi/pch.asl index cfa27d2b46..9f5033d759 100644 --- a/src/southbridge/intel/bd82x6x/acpi/pch.asl +++ b/src/southbridge/intel/bd82x6x/acpi/pch.asl @@ -254,10 +254,16 @@ Scope(\) Method (_OSC, 4) { + /* + * Arg0 - A Buffer containing a UUID + * Arg1 - An Integer containing a Revision ID of the buffer format + * Arg2 - An Integer containing a count of entries in Arg3 + * Arg3 - A Buffer containing a list of DWORD capabilities + */ /* Check for XHCI */ If (LEqual (Arg0, ToUUID("7c9512a9-1705-4cb4-af7d-506a2423ab71"))) { - Return (^XHC.POSC(Arg1, Arg2, Arg3)) + Return (^XHC.POSC(Arg2, Arg3)) } /* Check for PCIe */ diff --git a/src/southbridge/intel/bd82x6x/acpi/usb.asl b/src/southbridge/intel/bd82x6x/acpi/usb.asl index 5f78e3e904..c5bc48d73d 100644 --- a/src/southbridge/intel/bd82x6x/acpi/usb.asl +++ b/src/southbridge/intel/bd82x6x/acpi/usb.asl @@ -140,13 +140,18 @@ Device (XHC) Name (_PRW, Package(){ 13, 4 }) // Power Resources for Wake - Method(POSC,3,Serialized) + Method(POSC,2,Serialized) { + /* + * Arg0 - An Integer containing a count of entries in Arg3 + * Arg1 - A Buffer containing a list of DWORD capabilities + */ + // Create DWord field from the Capabilities Buffer - CreateDWordField(Arg2,0,CDW1) + CreateDWordField(Arg1,0,CDW1) // Check revision - If(LNotEqual(Arg1,One)) { + If(LNotEqual(Arg0,One)) { // Set unknown revision bit Or(CDW1,0x8,CDW1) } @@ -179,7 +184,7 @@ Device (XHC) And(X2PR, 0xFFFFFFF0, Local0) Or(Local0, Local1, X2PR) } - Return(Arg2) + Return(Arg1) } // Leave USB ports on for to allow Wake from USB -- cgit v1.2.3