Ren Kuo has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84219?usp=email )
Change subject: mb/google/brox/jubilant: Fix booting to kernel without battery ......................................................................
mb/google/brox/jubilant: Fix booting to kernel without battery
Modify jubilant cpu power limit setting depend on the brox baseboad settgins: https://review.coreboot.org/c/coreboot/+/83752
BUG=b:364441688 BRANCH=None TEST=Able to successfully boot on jubilant photo SKU1 and SKU2 boards with 65W adaptors of cold boot.
Change-Id: I9a143d9faaa6c57b0d314c0ff6c0e55f556d7216 Signed-off-by: Ren Kuo ren.kuo@quanta.corp-partner.google.com --- M src/mainboard/google/brox/variants/jubilant/ramstage.c 1 file changed, 0 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/84219/1
diff --git a/src/mainboard/google/brox/variants/jubilant/ramstage.c b/src/mainboard/google/brox/variants/jubilant/ramstage.c index 86418da..f4e0bcf 100644 --- a/src/mainboard/google/brox/variants/jubilant/ramstage.c +++ b/src/mainboard/google/brox/variants/jubilant/ramstage.c @@ -32,16 +32,6 @@ }, };
-const struct system_power_limits sys_limits[] = { - /* SKU_ID, TDP (Watts), psys_pl2 (Watts) */ - { PCI_DID_INTEL_RPL_P_ID_3, 15, 60 }, - { PCI_DID_INTEL_RPL_P_ID_4, 15, 60 }, -}; - -const struct psys_config psys_config = { - .efficiency = 86, -}; - void __weak variant_devtree_update(void) { printk(BIOS_DEBUG, "WEAK: %s/%s called\n", __FILE__, __func__); @@ -50,5 +40,4 @@ size_t limits_size = ARRAY_SIZE(performance_efficient_limits);
variant_update_power_limits(limits, limits_size); - variant_update_psys_power_limits(limits, sys_limits, limits_size, &psys_config); }