Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/52408 )
Change subject: mb/google/dedede/var/sasuke: Update DPTF parameters ......................................................................
mb/google/dedede/var/sasuke: Update DPTF parameters
Add control charging current from TSR0 and correct charger_perf table value.
BUG=b:179067801 BRANCH=dedede TEST=emerge-dedede coreboot
Change-Id: Ie0d969898defe76952e5c136fa93b7edffe51de3 Signed-off-by: Seunghwan Kim sh_.kim@samsung.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/52408 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Sumeet R Pawnikar sumeet.r.pawnikar@intel.com Reviewed-by: Edward Doan edoan@google.com --- M src/mainboard/google/dedede/variants/sasuke/overridetree.cb 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Sumeet R Pawnikar: Looks good to me, approved Edward Doan: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/dedede/variants/sasuke/overridetree.cb b/src/mainboard/google/dedede/variants/sasuke/overridetree.cb index 6bcde70..491de2f 100644 --- a/src/mainboard/google/dedede/variants/sasuke/overridetree.cb +++ b/src/mainboard/google/dedede/variants/sasuke/overridetree.cb @@ -86,7 +86,7 @@ register "options.tsr[1].desc" = ""Vcore"" register "policies.passive" = "{ [0] = DPTF_PASSIVE(CPU, CPU, 90, 10000), - [1] = DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 60, 60000), + [1] = DPTF_PASSIVE(CHARGER, TEMP_SENSOR_0, 60, 60000), [2] = DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 60, 15000), }" register "policies.critical" = "{ @@ -112,9 +112,9 @@ }" register "controls.charger_perf" = "{ [0] = { 255, 3000 }, - [1] = { 24, 2500 }, - [2] = { 16, 2000 }, - [3] = { 8, 1500 } + [1] = { 39, 2500 }, + [2] = { 31, 2000 }, + [3] = { 23, 1500 } }" device generic 0 on end end