diff options
author | Nicholas Sudsgaard <devel+coreboot@nsudsgaard.com> | 2024-03-11 21:46:20 +0900 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2024-03-16 15:19:29 +0000 |
commit | f77a28ac1f472cc649f92d45191322ef3e11c343 (patch) | |
tree | 0fd0ae13c9992e97737dc4f60ad49fc7e356dba4 /src | |
parent | 99b069baa6afac418b6751c5e95c1587f9bec095 (diff) |
ec/hp/kbc1126/acpi: Drop unnecessary method arguments
Method(..., 0, NotSerialized) is the default[1] and can be reduced to
Method(...) which reduces some noise.
TEST=Timeless build produces the same binary
[1] https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/19_ASL_Reference/ACPI_Source_Language_Reference.html#method-declare-control-method
Change-Id: Ic24e004500a7fa2a5a5b38a3f6f0e13e4ce7dfac
Signed-off-by: Nicholas Sudsgaard <devel+coreboot@nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81205
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <inforichland@gmail.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/ec/hp/kbc1126/acpi/ac.asl | 4 | ||||
-rw-r--r-- | src/ec/hp/kbc1126/acpi/battery.asl | 24 | ||||
-rw-r--r-- | src/ec/hp/kbc1126/acpi/ec.asl | 12 | ||||
-rw-r--r-- | src/ec/hp/kbc1126/acpi/lid.asl | 5 |
4 files changed, 23 insertions, 22 deletions
diff --git a/src/ec/hp/kbc1126/acpi/ac.asl b/src/ec/hp/kbc1126/acpi/ac.asl index f058cef9f2..59681cf33a 100644 --- a/src/ec/hp/kbc1126/acpi/ac.asl +++ b/src/ec/hp/kbc1126/acpi/ac.asl @@ -52,7 +52,7 @@ Device (AC) Return (0x0F) } - Method (_PSR, 0, NotSerialized) + Method (_PSR) { Local0 = GACS () PWRS = Local0 /* GNVS.PWRS */ @@ -63,7 +63,7 @@ Device (AC) } } -Method (_Q06, 0, NotSerialized) +Method (_Q06) { Printf ("EC: AC STATUS") PWUP (0x05, (0x02 | 0x01)) diff --git a/src/ec/hp/kbc1126/acpi/battery.asl b/src/ec/hp/kbc1126/acpi/battery.asl index e657f4ca6f..ff542a1610 100644 --- a/src/ec/hp/kbc1126/acpi/battery.asl +++ b/src/ec/hp/kbc1126/acpi/battery.asl @@ -40,7 +40,7 @@ Mutex (BTMX, 0x00) #define NLB1 0xC8 /* design capacity of warning */ #define NLB2 0x64 /* design capacity of low */ -Method (ITLB, 0, NotSerialized) +Method (ITLB) { Local0 = (NLB1 + 9) % 0x0a Local1 = (NLB1 + 9) / 0x0a @@ -127,17 +127,17 @@ Device (BAT0) Name (_HID, EisaId ("PNP0C0A")) Name (_UID, 0x01) - Method (_STA, 0, NotSerialized) + Method (_STA) { Return (BSTA (0x01)) } - Method (_BIF, 0, NotSerialized) + Method (_BIF) { Return (BTIF (0x00)) } - Method (_BST, 0, NotSerialized) + Method (_BST) { Return (BTSZ (0x00)) } @@ -153,17 +153,17 @@ Device (BAT1) Name (_HID, EisaId ("PNP0C0A")) Name (_UID, 0x02) - Method (_STA, 0, NotSerialized) + Method (_STA) { Return (BSTA (0x02)) } - Method (_BIF, 0, NotSerialized) + Method (_BIF) { Return (BTIF (0x01)) } - Method (_BST, 0, NotSerialized) + Method (_BST) { Return (BTSZ (0x01)) } @@ -440,7 +440,7 @@ Method (SBTN, 2, Serialized) } } -Method (_Q03, 0, NotSerialized) +Method (_Q03) { Printf ("EC: _Q03") Acquire (BTMX, 0xFFFF) @@ -450,7 +450,7 @@ Method (_Q03, 0, NotSerialized) SBTN (Local0, 0x80) } -Method (_Q08, 0, NotSerialized) +Method (_Q08) { Printf ("EC: PRIMARY BATTERY ATTACHED/DETACHED") PWUP (0x06, 0x01) @@ -470,7 +470,7 @@ Method (_Q08, 0, NotSerialized) } } -Method (_Q09, 0, NotSerialized) +Method (_Q09) { Printf ("EC: PRIMARY BATTERY STATUS") PWUP (0x04, 0x01) @@ -480,7 +480,7 @@ Method (_Q09, 0, NotSerialized) } } -Method (_Q18, 0, NotSerialized) +Method (_Q18) { Printf ("EC: SECONDARY BATTERY ATTACHED/DETACHED") PWUP (0x06, 0x02) @@ -500,7 +500,7 @@ Method (_Q18, 0, NotSerialized) } } -Method (_Q19, 0, NotSerialized) +Method (_Q19) { Printf ("EC: SECONDARY BATTERY STATUS") PWUP (0x04, 0x02) diff --git a/src/ec/hp/kbc1126/acpi/ec.asl b/src/ec/hp/kbc1126/acpi/ec.asl index 4c0056dcc6..afe0a70a40 100644 --- a/src/ec/hp/kbc1126/acpi/ec.asl +++ b/src/ec/hp/kbc1126/acpi/ec.asl @@ -12,7 +12,7 @@ Device (EC0) IO (Decode16, 0x66, 0x66, 1, 1) }) - Method (_STA, 0, NotSerialized) + Method (_STA) { Return (0x0F) } @@ -200,28 +200,28 @@ Device (EC0) /* The following are _Qxx methods in vendor DSDT whose function is unknown. Just leave a debug message here. */ - Method (_Q04, 0, NotSerialized) + Method (_Q04) { Printf ("EC: _Q04") PNOT() } - Method (_Q05, 0, NotSerialized) + Method (_Q05) { Printf ("EC: _Q05") } - Method (_Q0B, 0, NotSerialized) + Method (_Q0B) { Printf ("EC: _Q0B") } - Method (_Q0C, 0, NotSerialized) + Method (_Q0C) { Printf ("EC: _Q0C") } - Method (_Q0D, 0, NotSerialized) + Method (_Q0D) { Printf ("EC: _Q0D") } diff --git a/src/ec/hp/kbc1126/acpi/lid.asl b/src/ec/hp/kbc1126/acpi/lid.asl index 1a5134ff9a..ac5c55b021 100644 --- a/src/ec/hp/kbc1126/acpi/lid.asl +++ b/src/ec/hp/kbc1126/acpi/lid.asl @@ -3,14 +3,15 @@ Device (LID) { Name (_HID, EisaId ("PNP0C0D")) - Method (_LID, 0, NotSerialized) + + Method (_LID) { Local0 = CLID Return (Local0) } } -Method (_Q0A, 0, NotSerialized) +Method (_Q0A) { Printf ("EC: LID STATUS") Notify (LID, 0x80) |