Deepika Punyamurtula has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46676 )
Change subject: mb/google/volteer/variants/delbin: Update DPTF parameters for delbin ......................................................................
Patch Set 13:
(4 comments)
https://review.coreboot.org/c/coreboot/+/46676/12//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/46676/12//COMMIT_MSG@7 PS12, Line 7: UPSTREAM:
Remove here; this will get added when the patch is downstreamed into the chromium repo
Done
https://review.coreboot.org/c/coreboot/+/46676/12/src/mainboard/google/volte... File src/mainboard/google/volteer/variants/delbin/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/46676/12/src/mainboard/google/volte... PS12, Line 26: # Default DPTF Policy for all Volteer boards if not overridden
This comment isn't true here
Done, deleted this
https://review.coreboot.org/c/coreboot/+/46676/12/src/mainboard/google/volte... PS12, Line 31: TEMP_PCT(57, 90), : TEMP_PCT(47, 80), : TEMP_PCT(40, 70), : TEMP_PCT(36, 60), : TEMP_PCT(34, 50), : TEMP_PCT(30, 40),}}}"
Won't the fan be running constantly at these thresholds?
Hi Tim, sensor 3 is the ambient sensor near the fan. These settings can help provide more granular fan control. The sensor temp is seen to vary between ~30-50 for different workloads
https://review.coreboot.org/c/coreboot/+/46676/12/src/mainboard/google/volte... PS12, Line 74:
Did you intend to leave out the setting for the 500mA charging rate?, i.e. […]
Good catch :) I have added it for now. @Pegatron, please comment on this