mb/starlabs/starbook/kbl: Use function for getting power profile
All other variants use a function and definitions to get the power
profile. Make this board to the same.
Change-Id: I07ce71e20bd71229bb0cd3438ab59140cd0d8b42
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81412
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/src/mainboard/starlabs/starbook/variants/kbl/devtree.c b/src/mainboard/starlabs/starbook/variants/kbl/devtree.c
index 898519e..64f638b 100644
--- a/src/mainboard/starlabs/starbook/variants/kbl/devtree.c
+++ b/src/mainboard/starlabs/starbook/variants/kbl/devtree.c
@@ -17,19 +17,19 @@
struct device *nic_dev = pcidev_on_root(0x1c, 5);
/* Update PL1 & PL2 based on CMOS settings */
- switch (get_uint_option("power_profile", 0)) {
- case 1:
- soc_conf->tdp_pl1_override = 17;
- soc_conf->tdp_pl2_override = 20;
- break;
- case 2:
- soc_conf->tdp_pl1_override = 20;
- soc_conf->tdp_pl2_override = 25;
- break;
- default:
+ switch (get_power_profile(PP_POWER_SAVER)) {
+ case PP_POWER_SAVER:
disable_turbo();
- soc_conf->tdp_pl1_override = 15;
- soc_conf->tdp_pl2_override = 15;
+ soc_conf->tdp_pl1_override = 15;
+ soc_conf->tdp_pl2_override = 15;
+ break;
+ case PP_BALANCED:
+ soc_conf->tdp_pl1_override = 17;
+ soc_conf->tdp_pl2_override = 20;
+ break;
+ case PP_PERFORMANCE:
+ soc_conf->tdp_pl1_override = 20;
+ soc_conf->tdp_pl2_override = 25;
break;
}