[XL] Change in coreboot[main]: soc/intel/ptl: Do initial Panther Lake SoC commit till romstage

[Saurabh Mishra (Code Review)]

Attention is currently required from: Bora Guvendik, Cliff Huang, Jérémy Compostella, Kapil Porwal, Pranava Y N, Ravishankar Sarawadi, Subrata Banik.

Saurabh Mishra has posted comments on this change by Saurabh Mishra. ( https://review.coreboot.org/c/coreboot/+/83635?usp=email )

Change subject: soc/intel/ptl: Do initial Panther Lake SoC commit till romstage
......................................................................

Patch Set 74:

(6 comments)

File src/soc/intel/pantherlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/83635/comment/d81332bd_3f141ee0?usp... :
PS74, Line 49: PTL-U (15W) & PTL-H 12Xe (25W) SoC.
> wondering on a client device, how is it possible to have two SoC integrated together ? […]

Sure, updated the note line.

https://review.coreboot.org/c/coreboot/+/83635/comment/34bfeddb_b8bb5036?usp... :
PS74, Line 55: 	select SOC_INTEL_PANTHERLAKE_BASE
>

Ack, added.

https://review.coreboot.org/c/coreboot/+/83635/comment/9c9833b0_ae0653c6?usp... :
PS74, Line 116: if SOC_INTEL_PANTHERLAKE_U_H
> you can drop this

Acknowledged

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

https://review.coreboot.org/c/coreboot/+/83635/comment/393dac06_b89ecb3d?usp... :
PS74, Line 60: 1 << 0,
> do you wish to use BIT(0) here to maintain the parity with other macros ?

Ack, using BIT(x) throughout.

https://review.coreboot.org/c/coreboot/+/83635/comment/b154633d_0bd51073?usp... :
PS74, Line 60: //
> also ```/* ... […]

Acknowledged

https://review.coreboot.org/c/coreboot/+/83635/comment/14b08d0c_8e57fb02?usp... :
PS74, Line 162: 	enum {
              : 		IGD_SM_0MB = 0x00,
              : 		IGD_SM_32MB = 0x01,
              : 		IGD_SM_64MB = 0x02,
              : 		IGD_SM_96MB = 0x03,
              : 		IGD_SM_128MB = 0x04,
              : 		IGD_SM_160MB = 0x05,
              : 		IGD_SM_4MB = 0xF0,
              : 		IGD_SM_8MB = 0xF1,
              : 		IGD_SM_12MB = 0xF2,
              : 		IGD_SM_16MB = 0xF3,
              : 		IGD_SM_20MB = 0xF4,
              : 		IGD_SM_24MB = 0xF5,
              : 		IGD_SM_28MB = 0xF6,
              : 		IGD_SM_36MB = 0xF8,
              : 		IGD_SM_40MB = 0xF9,
              : 		IGD_SM_44MB = 0xFA,
              : 		IGD_SM_48MB = 0xFB,
              : 		IGD_SM_52MB = 0xFC,
              : 		IGD_SM_56MB = 0xFD,
              : 		IGD_SM_60MB = 0xFE,
              : 	} IgdDvmt50PreAlloc;
> please confirm if this macro is relevant for PTL FSP or not ?

Hi Subrata, we are using these macro to set IGD stolen size in romstage. Will be used using FSP UPD.