Attention is currently required from: Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Sumeet R Pawnikar.
Subrata Banik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/83565?usp=email )
Change subject: mb/google/brya/var/trulo: Add Thermal descriptions ......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83565/comment/d0ac55ac_583ddd44?usp... : PS1, Line 12: BUG=b:351976770
Add me in bug to understand the background of this change. Thanks.
This bug is to make Trulo paper design ready to boot N/DOrisa SKUs and if you would like to start thermal design for Orisa then please file a bug for compal.
File src/mainboard/google/brya/variants/trulo/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/83565/comment/db32cf99_b5833f22?usp... : PS1, Line 198: register "policies.active" = "{
We need different setting values to control Fan speed for 6W and 15W design, if 6W system has Fan.
Note: this is base CL for N/DOrisa to operational and we can always optimise these numbers once we have board in hand.
Is 6W design Fan based the system?
Yes, we have fan in all the SKUs