diff options
author | Sean Rhodes <sean@starlabs.systems> | 2024-10-02 15:25:30 +0100 |
---|---|---|
committer | Sean Rhodes <sean@starlabs.systems> | 2024-10-03 09:35:39 +0000 |
commit | 68a51ec0d31055222d84909249a40fd275c38060 (patch) | |
tree | 1d8911613b54a62bb802f349c76915216bd7c17e /src/mainboard/starlabs/starbook | |
parent | 059a291f9e1d4bb963df6fbade2d9112a2fdfd84 (diff) |
mb/starlabs/*: Don't disable Turbo Boost in Power Saver profile
Tested on 24.04, disabling Turbo Boost increases power consumption
which doesn't align with the aim of the Power Saver profile.
Change-Id: I19e8189ee6c44d19bf222c921429284ed1e1aa2a
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84628
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/starlabs/starbook')
5 files changed, 0 insertions, 5 deletions
diff --git a/src/mainboard/starlabs/starbook/variants/adl/devtree.c b/src/mainboard/starlabs/starbook/variants/adl/devtree.c index 27ab6c90f2..fb961e20bd 100644 --- a/src/mainboard/starlabs/starbook/variants/adl/devtree.c +++ b/src/mainboard/starlabs/starbook/variants/adl/devtree.c @@ -24,7 +24,6 @@ void devtree_update(void) /* Update PL1 & PL2 based on CMOS settings */ switch (get_power_profile(PP_POWER_SAVER)) { case PP_POWER_SAVER: - disable_turbo(); soc_conf_10core->tdp_pl1_override = 15; soc_conf_12core->tdp_pl1_override = 15; soc_conf_10core->tdp_pl2_override = 15; diff --git a/src/mainboard/starlabs/starbook/variants/cml/devtree.c b/src/mainboard/starlabs/starbook/variants/cml/devtree.c index 95a5d44ea3..100a85c274 100644 --- a/src/mainboard/starlabs/starbook/variants/cml/devtree.c +++ b/src/mainboard/starlabs/starbook/variants/cml/devtree.c @@ -19,7 +19,6 @@ void devtree_update(void) /* Update PL1 & PL2 based on CMOS settings */ 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; cfg->tcc_offset = 20; diff --git a/src/mainboard/starlabs/starbook/variants/kbl/devtree.c b/src/mainboard/starlabs/starbook/variants/kbl/devtree.c index 38017dc2e9..5ee3a3ef8d 100644 --- a/src/mainboard/starlabs/starbook/variants/kbl/devtree.c +++ b/src/mainboard/starlabs/starbook/variants/kbl/devtree.c @@ -19,7 +19,6 @@ void devtree_update(void) /* Update PL1 & PL2 based on CMOS settings */ 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; cfg->tcc_offset = 20; diff --git a/src/mainboard/starlabs/starbook/variants/rpl/devtree.c b/src/mainboard/starlabs/starbook/variants/rpl/devtree.c index b190865975..a7b0157bb8 100644 --- a/src/mainboard/starlabs/starbook/variants/rpl/devtree.c +++ b/src/mainboard/starlabs/starbook/variants/rpl/devtree.c @@ -27,7 +27,6 @@ void devtree_update(void) /* Update PL1 & PL2 based on CMOS settings */ switch (get_power_profile(PP_POWER_SAVER)) { case PP_POWER_SAVER: - disable_turbo(); soc_conf_6core->tdp_pl1_override = 15; soc_conf_12core->tdp_pl1_override = 15; soc_conf_6core->tdp_pl2_override = 15; diff --git a/src/mainboard/starlabs/starbook/variants/tgl/devtree.c b/src/mainboard/starlabs/starbook/variants/tgl/devtree.c index 7c94430ee8..8d206fa38c 100644 --- a/src/mainboard/starlabs/starbook/variants/tgl/devtree.c +++ b/src/mainboard/starlabs/starbook/variants/tgl/devtree.c @@ -26,7 +26,6 @@ void devtree_update(void) /* Update PL1 & PL2 based on CMOS settings */ switch (get_power_profile(PP_POWER_SAVER)) { case PP_POWER_SAVER: - disable_turbo(); soc_conf_2core->tdp_pl1_override = 15; soc_conf_4core->tdp_pl1_override = 15; soc_conf_2core->tdp_pl2_override = 15; |