[M] Change in coreboot[main]: mb/google/brox: Remove Mainboard Prepare to Sleep(MPTS) ACPI method

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/83788?usp=email )

Change subject: mb/google/brox: Remove Mainboard Prepare to Sleep(MPTS) ACPI method
......................................................................

mb/google/brox: Remove Mainboard Prepare to Sleep(MPTS) ACPI method

Brox does not have WWAN or discrete GPU. Hence no need to power them off
during suspend. Hence also remove the MPTS ACPI method.

BUG=None
TEST=Build Brox firmware and boot to OS.

Change-Id: Ia239c3f038ce31934efb0a391350fa0f786e3fcd
Signed-off-by: Karthikeyan Ramasubramanian 
---
M src/mainboard/google/brox/mainboard.c
1 file changed, 0 insertions(+), 64 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/83788/1

diff --git a/src/mainboard/google/brox/mainboard.c b/src/mainboard/google/brox/mainboard.c
index 7ac4f24..19ba730 100644
--- a/src/mainboard/google/brox/mainboard.c
+++ b/src/mainboard/google/brox/mainboard.c
@@ -13,9 +13,6 @@
 #include <soc/ramstage.h>
 #include <stdio.h>

-WEAK_DEV_PTR(rp6_wwan);
-WEAK_DEV_PTR(dgpu);
-
 #define HDMI_HPD GPP_A18

static void add_fw_config_oem_string(const struct fw_config *config, void *arg)
@@ -105,35 +102,6 @@
    mainboard_ec_init();
 }

-static void mainboard_generate_wwan_shutdown(const struct device *dev)
-{
-	const struct drivers_wwan_fm_config *config = config_of(dev);
-	const struct device *parent = dev->upstream->dev;
-
-	if (!config)
-		return;
-	if (config->rtd3dev) {
-		acpigen_write_store();
-		acpigen_emit_namestring(acpi_device_path_join(parent, "RTD3._STA"));
-		acpigen_emit_byte(LOCAL0_OP);
-		acpigen_write_if_lequal_op_int(LOCAL0_OP, ONE_OP);
-		{
-			acpigen_emit_namestring(acpi_device_path_join(dev, "DPTS"));
-			acpigen_emit_byte(ARG0_OP);
-		}
-		acpigen_write_if_end();
-	}
-}
-
-static void mainboard_generate_dgpu_shutdown(const struct device *dev)
-{
-	/* Call `_OFF` from the Power Resource associated with the dGPU's PEG port. */
-	const struct device *parent = dev->upstream->dev;
-
-	if (parent)
-		acpigen_emit_namestring(acpi_device_path_join(parent, "PGPR._OFF"));
-}
-
 static void mainboard_generate_hdmi_hpd_disable(void)
 {
    /* Change pad mode to GPIO control - GPMO(HDMI_HPD, 0); */
@@ -160,36 +128,6 @@
    acpigen_write_integer(1);
 }

-static void mainboard_generate_mpts(void)
-{
-	const struct device *wwan = DEV_PTR(rp6_wwan);
-	const struct device *dgpu = DEV_PTR(dgpu);
-
-	/*
-	 * If HAVE_WWAN_POWER_SEQUENCE is selected, MPTS will be added to the
-	 * DSDT via wwan_power.asl. We can't add MPTS to the SSDT as well,
-	 * since the duplicate definition will result in a kernel error.
-	 *
-	 * This special case can be removed in the future if the power-off
-	 * sequences for all WWAN devices used on brox are moved to the SSDT.
-	 */
-	if (CONFIG(HAVE_WWAN_POWER_SEQUENCE)) {
-		if (wwan || dgpu)
-			printk(BIOS_ERR, "Skip adding duplicate MPTS entry to SSDT\n");
-		return;
-	}
-
-	acpigen_write_scope("\_SB");
-	acpigen_write_method_serialized("MPTS", 1);
-	if (wwan)
-		mainboard_generate_wwan_shutdown(wwan);
-	if (dgpu)
-		mainboard_generate_dgpu_shutdown(dgpu);
-
-	acpigen_write_method_end(); /* Method */
-	acpigen_write_scope_end(); /* Scope */
-}
-
 static void mainboard_generate_s0ix_hook(void)
 {
    acpigen_write_if_lequal_op_int(ARG0_OP, 1);
@@ -211,8 +149,6 @@

static void mainboard_fill_ssdt(const struct device *dev)
 {
-	mainboard_generate_mpts();
-
    /* for variant to fill additional SSDT */
    variant_fill_ssdt(dev);