summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSubrata Banik <subratabanik@google.com>2023-11-29 00:42:26 +0530
committerSubrata Banik <subratabanik@google.com>2023-12-04 17:04:02 +0000
commitffd9dd55afe8c3743e6585643d756733357d20aa (patch)
tree875e0cd45439d380028250a05da08beef9d22d67
parent5fe04f33ee595be1816e935cae5e29b7f037261a (diff)
mb/google/rex/var/screebo: Override power limits
This patch allows variants to override the default baseboard PLx limits. Additionally, rearrange the include header files alphabetically. BUG=b:313667378 TEST=Able to boot google/screebo with modified power limits. Before: [DEBUG] WEAK: src/mainboard/google/rex/variants/baseboard/rex/ ramstage.c/variant_devtree_update called [INFO ] Overriding power limits PL1 (mW) (10000, 15000) PL2 (mW) (40000, 40000) PL4 (W) (84) After: [INFO ] Overriding power limits PL1 (mW) (10000, 15000) PL2 (mW) (40000, 40000) PL4 (W) (84) Change-Id: Ic66872c530963238a0bf5eebbd5b5a76a7985e5c Signed-off-by: Subrata Banik <subratabanik@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/79330 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Kapil Porwal <kapilporwal@google.com>
-rw-r--r--src/mainboard/google/rex/variants/screebo/variant.c65
1 files changed, 64 insertions, 1 deletions
diff --git a/src/mainboard/google/rex/variants/screebo/variant.c b/src/mainboard/google/rex/variants/screebo/variant.c
index 1dd8b980e3..edddd43a0a 100644
--- a/src/mainboard/google/rex/variants/screebo/variant.c
+++ b/src/mainboard/google/rex/variants/screebo/variant.c
@@ -1,8 +1,9 @@
/* SPDX-License-Identifier: GPL-2.0-only */
+#include <baseboard/variants.h>
#include <chip.h>
+#include <ec/google/chromeec/ec.h>
#include <fw_config.h>
-#include <baseboard/variants.h>
#include <sar.h>
const char *get_wifi_sar_cbfs_filename(void)
@@ -29,3 +30,65 @@ void variant_update_soc_chip_config(struct soc_intel_meteorlake_config *config)
config->tcss_aux_ori = 0x04;
}
}
+
+const struct cpu_tdp_power_limits variant_perf_efficient_limits[] = {
+ {
+ .mch_id = PCI_DID_INTEL_MTL_P_ID_2,
+ .cpu_tdp = 15,
+ .pl1_min_power = 10000,
+ .pl1_max_power = 15000,
+ .pl2_min_power = 40000,
+ .pl2_max_power = 40000,
+ .pl4_power = 84000
+ },
+ {
+ .mch_id = PCI_DID_INTEL_MTL_P_ID_5,
+ .cpu_tdp = 15,
+ .pl1_min_power = 10000,
+ .pl1_max_power = 15000,
+ .pl2_min_power = 40000,
+ .pl2_max_power = 40000,
+ .pl4_power = 84000
+ },
+};
+
+const struct cpu_tdp_power_limits variant_power_efficient_limits[] = {
+ {
+ .mch_id = PCI_DID_INTEL_MTL_P_ID_2,
+ .cpu_tdp = 15,
+ .pl1_min_power = 10000,
+ .pl1_max_power = 15000,
+ .pl2_min_power = 40000,
+ .pl2_max_power = 40000,
+ .pl4_power = 47000
+ },
+ {
+ .mch_id = PCI_DID_INTEL_MTL_P_ID_5,
+ .cpu_tdp = 15,
+ .pl1_min_power = 10000,
+ .pl1_max_power = 15000,
+ .pl2_min_power = 40000,
+ .pl2_max_power = 40000,
+ .pl4_power = 47000
+ },
+};
+
+void variant_devtree_update(void)
+{
+ const struct cpu_tdp_power_limits *limits = variant_perf_efficient_limits;
+ size_t limits_size = ARRAY_SIZE(variant_perf_efficient_limits);
+
+ /*
+ * If battery is not present or battery level is at or below critical threshold
+ * to boot a platform with the performance efficient configuration, boot with
+ * the power optimized configuration.
+ */
+ if (CONFIG(EC_GOOGLE_CHROMEEC)) {
+ if (!google_chromeec_is_battery_present_and_above_critical_threshold()) {
+ limits = variant_power_efficient_limits;
+ limits_size = ARRAY_SIZE(variant_power_efficient_limits);
+ }
+ }
+
+ variant_update_cpu_power_limits(limits, limits_size);
+}