Change in coreboot[master]: soc/intel/common: Implements ACPI CPPCv3 package to support hybrid core

[Subrata Banik (Code Review)]

Attention is currently required from: Nico Huber, Tim Wawrzynczak, Sridhar Siricilla, Michael Niewöhner, Patrick Rudolph.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59359 )

Change subject: soc/intel/common: Implements ACPI CPPCv3 package to support hybrid core
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/common/block/acpi/acpi/globalnvs.asl:

https://review.coreboot.org/c/coreboot/+/59359/comment/37aefbc5_14b05484 
PS1, Line 28: 	SFBC,	16,	// 0x48 - 0x49 Indicates Scaling factor for Big core
            : 	SFSC,	16,	// 0x50 - 0x51 Indicates Scaling Factor for Small Core
            : 	NMFQ,	16,	// 0x52 - 0x53 Indicates Nominal Frequency
            : 	CORE,	32,	// 0x54 - 0x57 Each marked bit indicates Big Core corresponding to core index
            : 	INFS,	8,	// 0x58 - Nominal Frequency is supported
            :
> where are these used? do these values have to be in gnvs in contrast to generating and adding them t […]

Typically, as we all know the purpose of GNVS is to make those data collected from 'C' files and put into GNVS so that we can make those data available as part of  ASL code to ACPI OS.

Now your question is valid that, can this data exist in ASL itself if not borrowed these values from 'C' code as mentioned above.