[coreboot-gerrit] [S] Change in coreboot[master]: soc/amd/common/block/lpc/lpc: don't report PCI config IO ports in LPC

[Felix Held (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Fred Reitberger.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74840 )

Change subject: soc/amd/common/block/lpc/lpc: don't report PCI config IO ports in LPC
......................................................................

Patch Set 2:

(1 comment)

File src/soc/amd/common/block/lpc/lpc.c:

https://review.coreboot.org/c/coreboot/+/74840/comment/77f5512c_ca905076 
PS2, Line 117: IORESOURCE_FIXED
i'm not sure about the IORESOURCE_FIXED flag. when adding subtractive domain MMIO resources, the IORESOURCE_FIXED flag mustn't be set. if IORESOURCE_FIXED is set in the subtractive domain resources to tel lthe resource allocator where it may put non-fixed mmio resources, we get this error

DOMAIN: 0000: Resource ranges:
   * EMPTY!!

and after that also some NULL dereferences