[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cannonlake,icelake}: Do not define PCH_DEV_PMC in ramstage

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34120 )

Change subject: soc/intel/{cannonlake,icelake}: Do not define PCH_DEV_PMC in ramstage
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34120/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34120/2//COMMIT_MSG@10 
PS2, Line 10: This device gets hidden from PCI bus
            : in FSP-S
> Uh... I  found CB:22556  […]

What's the CL you were meaning? 22556 points to an em100 change. If devices are disappearing one needs to hang the fixed resources off of something. There's no 'platform device' that just floats currently, but we could add something like that.  In either case, the resources need to be accounted for so the allocator will work.