[coreboot-gerrit] Change in coreboot[master]: soc/intel/alderlake: set power limits dynamically for thermal

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Patrick Rudolph, EricR Lai, Karthik Ramasubramanian.
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57035 )

Change subject: soc/intel/alderlake: set power limits dynamically for thermal
......................................................................

Patch Set 8:

(4 comments)

File src/soc/intel/alderlake/chip.h:

https://review.coreboot.org/c/coreboot/+/57035/comment/e7b97bfb_e97c00e7 
PS7, Line 35: 	ADL_P_282_15W = 15,
> ex TDP_15W=15 then you can use in […]

Done

File src/soc/intel/alderlake/chip.h:

https://review.coreboot.org/c/coreboot/+/57035/comment/8a5368e7_99aba206 
PS6, Line 34: enum soc_intel_alderlake_cpu_tdps {
> if the value are the same, you can just declare 9W 15W 28W 45W. […]

Ack

File src/soc/intel/alderlake/systemagent.c:

https://review.coreboot.org/c/coreboot/+/57035/comment/fb3986d4_d3a4fcbf 
PS4, Line 80: 	switch (sa_pci_id) {
> That works 😊

Done

File src/soc/intel/alderlake/systemagent.c:

https://review.coreboot.org/c/coreboot/+/57035/comment/e381d982_92858872 
PS7, Line 81: 		if (sa_pci_id == cpuid_to_adlp[i].cpu_id) {
> you can use && here as well

Done