[S] Change in coreboot[master]: mb/google/skyrim: Move SPL setting to variants

[Chris Wang (Code Review)]

Chris Wang has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/73426 )

Change subject: mb/google/skyrim: Move SPL setting to variants
......................................................................

mb/google/skyrim: Move SPL setting to variants

Move the sustained_power_limit_mW setting from the baseboard
to variants. This setting will be needed before STT is enabled,
but once STT is enabled, this setting should be removed.

BUG=b:265267957
BRANCH=none
TEST=Build/Boot to ChromeOS
Signed-off-by: Chris Wang

Change-Id: I7b9779600cfa8c7581732e936a714728fd618d20
---
M src/mainboard/google/skyrim/variants/baseboard/devicetree.cb
M src/mainboard/google/skyrim/variants/crystaldrift/overridetree.cb
M src/mainboard/google/skyrim/variants/markarth/overridetree.cb
M src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
4 files changed, 32 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/73426/1

diff --git a/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb b/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb
index c92131c..6892742 100644
--- a/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb
+++ b/src/mainboard/google/skyrim/variants/baseboard/devicetree.cb
@@ -260,8 +260,6 @@
    register "fast_ppt_limit_mW"                     = "30000"
    register "slow_ppt_time_constant_s"              = "5"
    register "stapm_time_constant_s"                 = "275"
-	# Remove the sustained_power_limit_mW when STT is enabled
-	register "sustained_power_limit_mW"              = "15000"
    register "thermctl_limit_degreeC"                = "100"
    register "vrm_current_limit_mA"                  = "28000"
    register "vrm_maximum_current_limit_mA"          = "50000"
diff --git a/src/mainboard/google/skyrim/variants/crystaldrift/overridetree.cb b/src/mainboard/google/skyrim/variants/crystaldrift/overridetree.cb
index 7a56f93..9f58641 100644
--- a/src/mainboard/google/skyrim/variants/crystaldrift/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/crystaldrift/overridetree.cb
@@ -1,5 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-or-later

chip soc/amd/mendocino
+
+	# Set Package Power Parameters
+	# Remove the sustained_power_limit_mW when STT is enabled
+	register "sustained_power_limit_mW"              = "15000"
+
    device domain 0 on end
 end	# chip soc/amd/mendocino
diff --git a/src/mainboard/google/skyrim/variants/markarth/overridetree.cb b/src/mainboard/google/skyrim/variants/markarth/overridetree.cb
index f50820b..b7132a8 100644
--- a/src/mainboard/google/skyrim/variants/markarth/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/markarth/overridetree.cb
@@ -2,6 +2,10 @@

chip soc/amd/mendocino

+	# Set Package Power Parameters
+	# Remove the sustained_power_limit_mW when STT is enabled
+	register "sustained_power_limit_mW"              = "15000"
+
    device domain 0 on

register "dxio_tx_vboost_enable" = "1"
diff --git a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
index 0b9781e..3c33741 100644
--- a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
@@ -27,6 +27,11 @@
 end

chip soc/amd/mendocino
+
+	# Set Package Power Parameters
+	# Remove the sustained_power_limit_mW when STT is enabled
+	register "sustained_power_limit_mW"              = "15000"
+
    device domain 0 on
    	device ref gpp_bridge_a on  # Internal GPP Bridge 0 to Bus A
    		device ref xhci_1 on # XHCI1 controller