[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/alderlake: Set PL1 Time to Intel default 56s for certain CPUs

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Michał Żygowski, Paul Menzel, Subrata Banik, Tarun Tuli.

Sumeet R Pawnikar has posted comments on this change by Michał Żygowski. ( https://review.coreboot.org/c/coreboot/+/69217?usp=email )

Change subject: soc/intel/alderlake: Set PL1 Time to Intel default 56s for certain CPUs
......................................................................

Patch Set 3:

(1 comment)

File src/soc/intel/common/block/include/intelblocks/power_limit.h:

https://review.coreboot.org/c/coreboot/+/69217/comment/18dff202_192213d1?usp... :
PS3, Line 16: MOBILE_SKU_PL1_TIME_SEC		
It might make sense for me to change this macro name as well which reflects the Time in Seconds, something like this "POWER_LIMIT1_TIME_28_SEC"