[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/rex: Add CPU power limit values

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Tarun Tuli, Subrata Banik, Kapil Porwal.

Sumeet R Pawnikar has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74621 )

Change subject: mb/google/rex: Add CPU power limit values
......................................................................

mb/google/rex: Add CPU power limit values

Add CPU power limit values to override for rex board

BRANCH=None
BUG=b:270664854
TEST=Built and verified power limit values for 15W SKU on Rex board

Change-Id: Ia20421f73758b7b0266fba957fb8200439ce1dab
Signed-off-by: Sumeet Pawnikar 
---
M src/mainboard/google/rex/mainboard.c
M src/mainboard/google/rex/variants/baseboard/include/baseboard/variants.h
M src/mainboard/google/rex/variants/rex0/Makefile.inc
A src/mainboard/google/rex/variants/rex0/ramstage.c
4 files changed, 43 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/74621/1

diff --git a/src/mainboard/google/rex/mainboard.c b/src/mainboard/google/rex/mainboard.c
index c1c22db..7d5f01d 100644
--- a/src/mainboard/google/rex/mainboard.c
+++ b/src/mainboard/google/rex/mainboard.c
@@ -30,6 +30,12 @@
    fw_config_gpio_padbased_override(padbased_table);
    gpio_configure_pads_with_padbased(padbased_table);
    free(padbased_table);
+	variant_devtree_update();
+}
+
+void __weak variant_devtree_update(void)
+{
+	/* Override dev tree settings per board */
 }

void __weak variant_generate_s0ix_hook(enum s0ix_entry entry)
diff --git a/src/mainboard/google/rex/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/rex/variants/baseboard/include/baseboard/variants.h
index b51517c..3cb1fb2 100644
--- a/src/mainboard/google/rex/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/google/rex/variants/baseboard/include/baseboard/variants.h
@@ -29,4 +29,7 @@

void variant_generate_s0ix_hook(enum s0ix_entry entry);

+/* Modify devictree settings during ramstage */
+void variant_devtree_update(void);
+
 #endif /*__BASEBOARD_VARIANTS_H__ */
diff --git a/src/mainboard/google/rex/variants/rex0/Makefile.inc b/src/mainboard/google/rex/variants/rex0/Makefile.inc
index cdbf407..95a700a 100644
--- a/src/mainboard/google/rex/variants/rex0/Makefile.inc
+++ b/src/mainboard/google/rex/variants/rex0/Makefile.inc
@@ -2,4 +2,5 @@
 romstage-y += gpio.c
 ramstage-y += gpio.c
 ramstage-y += variant.c
+ramstage-y += ramstage.c
 ramstage-$(CONFIG_FW_CONFIG) += fw_config.c
diff --git a/src/mainboard/google/rex/variants/rex0/ramstage.c b/src/mainboard/google/rex/variants/rex0/ramstage.c
new file mode 100644
index 0000000..c2d32af
--- /dev/null
+++ b/src/mainboard/google/rex/variants/rex0/ramstage.c
@@ -0,0 +1,17 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+
+#include <baseboard/variants.h>
+#include <device/pci_ids.h>
+#include <intelblocks/power_limit.h>
+
+const struct cpu_power_limits limits[] = {
+	/* SKU_ID, TDP (Watts), pl1_min, pl1_max, pl2_min, pl2_max, pl4 */
+	/* Following values are for performance config as per document #640982 */
+	{ PCI_DID_INTEL_MTL_P_ID_2, 15,  6000, 15000,  57000,  57000, 101000 },
+};
+
+void variant_devtree_update(void)
+{
+	size_t total_entries = ARRAY_SIZE(limits);
+	variant_update_power_limits(limits, total_entries);
+}