[XL] Change in coreboot[main]: soc/intel/ptl: Add SoC ACPI directory for Panther Lake

[Cliff Huang (Code Review)]

Attention is currently required from: Anil Kumar K, Bora Guvendik, Hannah Williams, Jamie Ryu, Jérémy Compostella, Kapil Porwal, Paul Menzel, Pranava Y N, Ravishankar Sarawadi, Saurabh Mishra, Subrata Banik, Wonkyu Kim.

Cliff Huang has posted comments on this change by Ravishankar Sarawadi. ( https://review.coreboot.org/c/coreboot/+/83772?usp=email )

Change subject: soc/intel/ptl: Add SoC ACPI directory for Panther Lake
......................................................................

Patch Set 98:

(2 comments)

File src/soc/intel/pantherlake/acpi/southbridge.asl:

https://review.coreboot.org/c/coreboot/+/83772/comment/3b540807_711b5e10?usp... :
PS81, Line 11: #if CONFIG(SOC_INTEL_COMMON_BLOCK_IOE_P2SB)
> can we have a PTL SoC that can decide to drop PCD P2SB support if not then I would default select io […]

I am not familiar with this but start looking into this.

File src/soc/intel/pantherlake/acpi/tcss_pcierp.asl:

https://review.coreboot.org/c/coreboot/+/83772/comment/c2db7088_217bcf46?usp... :
PS23, Line 56: 0xBAC
> > Subrata, should we have soc/tcss. […]

Subrata, I browsed across the ASL files and check how the Field under OperationRegion are written, the Offset always uses with direct value instead of macros. I assume you mean all the hard-coded offset value and not just this particular one.  Unless the same offset has been used in multiple places, I'd rather leave it as it for consistency reason. In addition, the ASL will look closer to its converted .dsl file from the platform for debug/comparison purpose.  Will it be okay?