Change in coreboot[master]: mb/google/brya: Disable c1-state auto demotion for Brya & Brask
Attention is currently required from: Subrata Banik, Vinay Kumar, Tim Wawrzynczak, Sridhar Siricilla. Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62630 )
Change subject: mb/google/brya: Disable c1-state auto demotion for Brya & Brask ......................................................................
Patch Set 7:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62630/comment/2d0996c4_336d66e4 PS5, Line 9: C state
Done […]
Done
https://review.coreboot.org/c/coreboot/+/62630/comment/4f51a94f_e3d1c5cc PS5, Line 13: disabling this feature results in 110 mW power savings
Yes, no performance impact was seen with disabling autodemotion. […]
Yes Subrata, as per my understanding it would not have impact in case of Balanced power/performance policy. Since coreboot uses balanced policy, it's working in our case. @Vinay, Can you help to clarify if my understanding is incorrect?
Commit Message:
https://review.coreboot.org/c/coreboot/+/62630/comment/272dfc0d_649db983 PS6, Line 7: C
c1
Done
https://review.coreboot.org/c/coreboot/+/62630/comment/a6df7419_e494af32 PS6, Line 9: c-state
c1 state
Done
-
Maulik V Vaghela (Code Review)