[coreboot-gerrit] [L] Change in coreboot[master]: soc/amd/block/ivrs: Generalize IVRS table generation

[Naresh Solanki (Code Review)]

Attention is currently required from: Arthur Heymans, Felix Held, Felix Singer, Fred Reitberger, Jason Glenesk, Marvin Drees, Matt DeVillier, Raul Rangel.

Naresh Solanki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75451?usp=email )

Change subject: soc/amd/block/ivrs: Generalize IVRS table generation
......................................................................

Patch Set 7:

(1 comment)

File src/soc/amd/common/block/acpi/ivrs.c:

https://review.coreboot.org/c/coreboot/+/75451/comment/ef2cd2a0_a2effe68 :
PS7, Line 262: /* Find EMMC */
             : 	if (!nb_dev->bus)
             : 		dev = pcidev_path_behind(nb_dev->bus, PCI_DEVFN(0x13, 1));
> I guess you want this one unconditionally as it does not add a real PCI device?

Previously, the entry was added without any conditions.
It seems that the assumption was made that it is always present.
However, based on my understanding, this entry should only be relevant if the device is present.
I'm uncertain if adding it without any conditions would be appropriate.
Please let me know your thoughts. Will align it accordingly.
Thanks.