[coreboot-gerrit] [M] Change in coreboot[main]: mb/google/{brya,hades}: use soc index for variant_update_power_limits()

[Subrata Banik (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, SH Kim.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81436?usp=email )

Change subject: mb/google/{brya,hades}: use soc index for variant_update_power_limits()
......................................................................

Patch Set 5:

(1 comment)

File src/mainboard/google/brya/variants/baseboard/brya/ramstage.c:

https://review.coreboot.org/c/coreboot/+/81436/comment/c286e8ef_e185d018 :
PS5, Line 68: 			soc_config->tdp_pl4 = DIV_ROUND_UP(limits[i].pl4_power,
> How about writing 'soc_config->tdp_pl1_override' and 'soc_config->tdp_pl2_override' with the values in limits[i] here as well?
> After booting to ChromeOS, dptf_power_limits setting values will be replaced with DTT setting in OS.

sure, if we wish to override the PL1 and PL2 values (bt i haven't came across to such requirement yet)