Attention is currently required from: Dengwu Yu, Karthik Ramasubramanian, Paul Menzel, Subrata Banik, wen zhang.
Jian Tong has posted comments on this change by Jian Tong. ( https://review.coreboot.org/c/coreboot/+/84202?usp=email )
Change subject: mb/google/brox/var/lotso: Update cpu power limits ......................................................................
Patch Set 8:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/84202/comment/6ef6f1ef_1748450a?usp... : PS6, Line 9: persent
present
Done
https://review.coreboot.org/c/coreboot/+/84202/comment/f3fc94f0_52ac8688?usp... : PS6, Line 9: limit PL4 to 40
… increase limit from 9 to 40
Done
https://review.coreboot.org/c/coreboot/+/84202/comment/5c79f591_b16a17cc?usp... : PS6, Line 10: Base on: https://review.coreboot.org/c/coreboot/+/83752
Please summarize the change-set, and once it’s commit, also add the git commit hash and summary.
Done
https://review.coreboot.org/c/coreboot/+/84202/comment/d92412cd_321b2e4a?usp... : PS6, Line 10: Base
Base*d*
Done
https://review.coreboot.org/c/coreboot/+/84202/comment/4c2ec7bb_11e9e9c8?usp... : PS6, Line 21:
What is the source for this data?
Done
https://review.coreboot.org/c/coreboot/+/84202/comment/bf49ac5a_b3d24111?usp... : PS6, Line 23: TEST=emerge-brox sys-boot/coreboot sys-boot/chromeos-bootimage
Should runtime tests be done?
Done