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

[Saurabh Mishra (Code Review)]

Attention is currently required from: Jérémy Compostella, Kapil Porwal, Pranava Y N, Subrata Banik.

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

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

Patch Set 75:

(4 comments)

File src/soc/intel/pantherlake/acpi.c:

https://review.coreboot.org/c/coreboot/+/83798/comment/f2265b3d_320472fd?usp... :
PS75, Line 126: 	/* Early return if the map is already initialized */
> why we need this comment ? i mean what benefit one will get by looking into the comment as the code  […]

Acknowledged, removed.

https://review.coreboot.org/c/coreboot/+/83798/comment/8e0b658e_d7090e3a?usp... :
PS75, Line 127:  {
> single statement doesn't need a brace (https://doc.coreboot.org/contributing/coding_style. […]

Acknowledged

File src/soc/intel/pantherlake/include/soc/iomap.h:

https://review.coreboot.org/c/coreboot/+/83798/comment/14c919cd_bd52d82d?usp... :
PS75, Line 23: Dummy
> `dummy`

Acknowledged

File src/soc/intel/pantherlake/pmc.c:

https://review.coreboot.org/c/coreboot/+/83798/comment/0eb026f9_9746fda9?usp... :
PS75, Line 25: pmcbase
> ideally we need to check if `pmcbase` is not NULL

Implemented the NULL check.