[coreboot-gerrit] [S] Change in coreboot[main]: mb/google/brask/var/bujia: Hook up two missing sensors for wireless a...

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Derek Huang, Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Paul Menzel, Shon Wang, Subrata Banik.

Sumeet R Pawnikar has posted comments on this change by Shon Wang. ( https://review.coreboot.org/c/coreboot/+/83302?usp=email )

Change subject: mb/google/brask/var/bujia: Hook up two missing sensors for wireless and memory
......................................................................

Patch Set 4:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83302/comment/93074cd5_ba56cbda?usp... :
PS4, Line 12: 327549688
Please add me in the bug to go through the history on this. Thanks.

File src/mainboard/google/brya/variants/bujia/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/83302/comment/6d76ff4c_580ce1aa?usp... :
PS4, Line 86: CHARGER
Any reason for removing Charger control (throttling input charging current) in new code change?

https://review.coreboot.org/c/coreboot/+/83302/comment/7e03ee7b_b18ca870?usp... :
PS4, Line 66: CPU
Any reason for changing the sensor0 name from DRAM to CPU? We have DPTF_CPU available and used in active policy below at line #75.
Also, same question for sensor1 name change.