[coreboot-gerrit] [M] Change in coreboot[master]: mb/google/nissa/var/craask: Modify DPTF settings

[Paul Menzel (Code Review)]

Attention is currently required from: David Wu, Tarun Tuli, Kangheui Won, Ren Kuo, Reka Norman, Vidya Gopalakrishnan, Sumeet R Pawnikar, Tyler Wang, Peter Ou.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71860 )

Change subject: mb/google/nissa/var/craask: Modify DPTF settings
......................................................................

Patch Set 5:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/71860/comment/cf442333_7e583977 
PS5, Line 9: Craasneto
Shouldn’t that be mentioned in the prefix?

https://review.coreboot.org/c/coreboot/+/71860/comment/cae7c824_b5edc1bb 
PS5, Line 9: Limit(PL)
Please add a space before the (.

https://review.coreboot.org/c/coreboot/+/71860/comment/c6c98859_61eaf879 
PS5, Line 9: Add ADL-N 15W CPU Power Limit(PL) settings for Craasneto
Where did you get the new information from?

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

https://review.coreboot.org/c/coreboot/+/71860/comment/16b13595_34db7951 
PS5, Line 25: 		option THERMAL_SOLUTION_15W	1
Where is that field defined or used? I am not finding it in the current source code of origin/master (commit b63eb4d172 (crossgcc: Upgrade LLVM version 15.0.6 to 15.0.7)).