[M] Change in coreboot[master]: mb/google/dedede/variants/shotzo: Turn off LAN power in S0ix

[Tony Huang (Code Review)]

Tony Huang has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/67373 )

Change subject: mb/google/dedede/variants/shotzo: Turn off LAN power in S0ix
......................................................................

mb/google/dedede/variants/shotzo: Turn off LAN power in S0ix

Turn off the onboard LAN power which is controlled by GPP_A10 to save power in
S0ix states. For an USB device, the S0ix hook is needed for the on/off
operations to take place.

BUG=b:12345
BRANCH=firmware-dedede-13606.B
TEST=emerge-shotzo coreboot
     check LAN LED off in s0ix states

Change-Id: Id01089531503e62231c5ab19e4cd8056198b9acb
Signed-off-by: Tony Huang 
---
M src/mainboard/google/dedede/mainboard.c
M src/mainboard/google/dedede/variants/baseboard/include/baseboard/variants.h
M src/mainboard/google/dedede/variants/shotzo/Makefile.inc
A src/mainboard/google/dedede/variants/shotzo/variant.c
4 files changed, 82 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/67373/1

diff --git a/src/mainboard/google/dedede/mainboard.c b/src/mainboard/google/dedede/mainboard.c
index ca5a21c..0e87758 100644
--- a/src/mainboard/google/dedede/mainboard.c
+++ b/src/mainboard/google/dedede/mainboard.c
@@ -1,6 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0-or-later */

#include <acpi/acpi.h>
+#include <acpi/acpigen.h>
 #include <baseboard/variants.h>
 #include <console/console.h>
 #include <device/device.h>
@@ -63,10 +64,50 @@
    return current;
 }

+static void mainboard_generate_s0ix_hook(void)
+{
+        acpigen_write_if_lequal_op_int(ARG0_OP, 1);
+        {
+                variant_generate_s0ix_hook(S0IX_ENTRY);
+        }
+        acpigen_write_else();
+        {
+                variant_generate_s0ix_hook(S0IX_EXIT);
+        }
+        acpigen_write_if_end();
+}
+
+static void mainboard_fill_ssdt(const struct device *dev)
+{
+
+        acpigen_write_scope("\_SB");
+        acpigen_write_method_serialized("MS0X", 1);
+        mainboard_generate_s0ix_hook();
+        acpigen_write_method_end(); /* Method */
+        acpigen_write_scope_end(); /* Scope */
+
+}
+
+void __weak variant_generate_s0ix_hook(enum s0ix_entry entry)
+{
+
+	/* Add board-specific MS0X entries */
+        /*
+           if (s0ix_entry == S0IX_ENTRY) {
+                implement variant operations here
+           }
+           if (s0ix_entry == S0IX_EXIT) {
+                implement variant operations here
+           }
+         */
+}
+
+
 static void mainboard_enable(struct device *dev)
 {
    dev->ops->init = mainboard_dev_init;
    dev->ops->write_acpi_tables = mainboard_write_acpi_tables;
+	dev->ops->acpi_fill_ssdt = mainboard_fill_ssdt;
 }

struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/google/dedede/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/dedede/variants/baseboard/include/baseboard/variants.h
index aef3075..0b11f26 100644
--- a/src/mainboard/google/dedede/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/google/dedede/variants/baseboard/include/baseboard/variants.h
@@ -15,6 +15,11 @@
 const struct pad_config *variant_sleep_gpio_table(size_t *num);
 const struct pad_config *variant_override_gpio_table(size_t *num);

+enum s0ix_entry {
+        S0IX_EXIT,
+        S0IX_ENTRY,
+};
+
 /**
  * Get board's Hardware features as defined in FW_CONFIG
  *
@@ -44,4 +49,6 @@

/* Modify LTE devictree settings during ramstage. */
 void update_lte_device(struct acpi_gpio *lte_reset_gpio, struct acpi_gpio *lte_enable_gpio);
+
+void variant_generate_s0ix_hook(enum s0ix_entry);
 #endif /*__BASEBOARD_VARIANTS_H__ */
diff --git a/src/mainboard/google/dedede/variants/shotzo/Makefile.inc b/src/mainboard/google/dedede/variants/shotzo/Makefile.inc
index eb2c9bc..655f3c8 100644
--- a/src/mainboard/google/dedede/variants/shotzo/Makefile.inc
+++ b/src/mainboard/google/dedede/variants/shotzo/Makefile.inc
@@ -1,3 +1,4 @@
 ## SPDX-License-Identifier: GPL-2.0-or-later

ramstage-y += gpio.c
+ramstage-y += variant.c
diff --git a/src/mainboard/google/dedede/variants/shotzo/variant.c b/src/mainboard/google/dedede/variants/shotzo/variant.c
new file mode 100644
index 0000000..7f13fbc
--- /dev/null
+++ b/src/mainboard/google/dedede/variants/shotzo/variant.c
@@ -0,0 +1,14 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <baseboard/variants.h>
+#include <variant/gpio.h>
+#include <acpi/acpigen.h>
+
+void variant_generate_s0ix_hook(enum s0ix_entry entry)
+{
+	if (entry == S0IX_ENTRY)
+		acpigen_soc_clear_tx_gpio(GPP_A10);
+	else if (entry == S0IX_EXIT)
+		acpigen_soc_set_tx_gpio(GPP_A10);
+}
+