From e2ea7f22c6355d15515c049ca0dc4352173a0c01 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Tue, 18 Feb 2025 20:51:47 +0000 Subject: [PATCH] soc/intel/cnvi: Remove _S0W and DSW Methods coreboot already has a way to configure wakeup from wireless through the `wifi/generic` driver, so remove these to avoid conflicts. Change-Id: I744ef37690b7a2478ec29a43b987b43592df2235 Signed-off-by: Sean Rhodes Reviewed-on: https://review.coreboot.org/c/coreboot/+/86506 Reviewed-by: Matt DeVillier Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/cnvi/cnvi.c | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/src/soc/intel/common/block/cnvi/cnvi.c b/src/soc/intel/common/block/cnvi/cnvi.c index 87e9bbb72e..a4263af646 100644 --- a/src/soc/intel/common/block/cnvi/cnvi.c +++ b/src/soc/intel/common/block/cnvi/cnvi.c @@ -86,19 +86,6 @@ static void cnvw_fill_ssdt(const struct device *dev) acpigen_write_field("CWAR", fields2, ARRAY_SIZE(fields2), FIELD_BYTEACC | FIELD_NOLOCK | FIELD_PRESERVE); -/* - * Method (_S0W, 0, NotSerialized) // _S0W: S0 Device Wake State - * { - * Return (ACPI_DEVICE_SLEEP_D3_HOT) - * } - */ - acpigen_write_method("_S0W", 0); - { - acpigen_write_return_integer(ACPI_DEVICE_SLEEP_D3_HOT); - } - acpigen_pop_len(); - - /* * Name (RSTT, Zero) */ @@ -326,10 +313,6 @@ static void cnvw_fill_ssdt(const struct device *dev) * Method (_PS3, 0, Serialized) * { * } - * - * Method (_DSW, 3) - * { - * } */ acpigen_write_method_serialized("_PS0", 0); acpigen_pop_len(); @@ -337,9 +320,6 @@ static void cnvw_fill_ssdt(const struct device *dev) acpigen_write_method_serialized("_PS3", 0); acpigen_pop_len(); - acpigen_write_method("_DSW", 3); - acpigen_pop_len(); - /* * Method (CFLR, 0, NotSerialized) * {