Maulik V Vaghela has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/62630 )
Change subject: mb/google/brya: Disable C state auto demotion for Brya ......................................................................
mb/google/brya: Disable C state auto demotion for Brya
C state auto demotion feature allows hardware to determine C state as per platform policy. Since Brya sets performance policy to balanced from hardware, auto demotion can be disabled without performance impact.
Also, disabling this feature results in 110 mW power savings during video playback.
BUG=b:221876248 BRANCH=firmware-brya-14505.B TEST=Code compiles and correct value of C state auto demotion is passed to FSP. Also power and performance impact has been measure by respective teams.
Change-Id: I41eea916cdfe4a86e4d263e3191f5cb40fa33a90 Signed-off-by: MAULIK V VAGHELA maulik.v.vaghela@intel.com --- M src/mainboard/google/brya/variants/baseboard/brya/devicetree.cb 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/62630/1
diff --git a/src/mainboard/google/brya/variants/baseboard/brya/devicetree.cb b/src/mainboard/google/brya/variants/baseboard/brya/devicetree.cb index 6ac796a..406556a 100644 --- a/src/mainboard/google/brya/variants/baseboard/brya/devicetree.cb +++ b/src/mainboard/google/brya/variants/baseboard/brya/devicetree.cb @@ -82,6 +82,9 @@ register "gpio_pm[COMM_4]" = "0" register "gpio_pm[COMM_5]" = "0"
+ # Disable C state auto-demotion for all brya baseboards + register "DisableC1StateAutoDemotion" = "1" + # Intel Common SoC Config #+-------------------+---------------------------+ #| Field | Value |