[coreboot-gerrit] [M] Change in coreboot[master]: soc/intel/common: Support power limits update for variants

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Tarun Tuli, Subrata Banik, Paul Menzel, Kapil Porwal.

Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74620 )

Change subject: soc/intel/common: Support power limits update for variants
......................................................................

Patch Set 8:

(2 comments)

File src/soc/intel/common/block/power_limit/power_limit.c:

https://review.coreboot.org/c/coreboot/+/74620/comment/843582fd_59d5d019 
PS7, Line 237: #if CONFIG(SOC_INTEL_COMMON_BLOCK_VARIANT_POWER_LIMIT)
> > It's added to prevent building this new code base for older platforms.  […]

Here the intention behind adding this new guard is to prevent building this new function code base for older platforms. 
So, we might not need this at the caller. We need this guard here only.

https://review.coreboot.org/c/coreboot/+/74620/comment/b38cfc03_2cc83ae2 
PS7, Line 241: if (!num_entries)
             : 		return;
> did u missed to submit a CL bt marked ACK?

Yes. Submitted new patch set for review now.