Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86506?usp=email )
Change subject: soc/intel/cnvi: Remove _S0W and DSW Methods ......................................................................
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 sean@starlabs.systems Reviewed-on: https://review.coreboot.org/c/coreboot/+/86506 Reviewed-by: Matt DeVillier matt.devillier@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/intel/common/block/cnvi/cnvi.c 1 file changed, 0 insertions(+), 20 deletions(-)
Approvals: build bot (Jenkins): Verified Matt DeVillier: Looks good to me, approved
diff --git a/src/soc/intel/common/block/cnvi/cnvi.c b/src/soc/intel/common/block/cnvi/cnvi.c index 87e9bbb..a4263af 100644 --- a/src/soc/intel/common/block/cnvi/cnvi.c +++ b/src/soc/intel/common/block/cnvi/cnvi.c @@ -87,19 +87,6 @@ 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) */ acpigen_write_name_integer("RSTT", 0); @@ -326,10 +313,6 @@ * Method (_PS3, 0, Serialized) * { * } - * - * Method (_DSW, 3) - * { - * } */ acpigen_write_method_serialized("_PS0", 0); acpigen_pop_len(); @@ -337,9 +320,6 @@ acpigen_write_method_serialized("_PS3", 0); acpigen_pop_len();
- acpigen_write_method("_DSW", 3); - acpigen_pop_len(); - /* * Method (CFLR, 0, NotSerialized) * {