[coreboot-gerrit] Change in ...coreboot[master]: AGESA/binaryPI: Add NULL pointers check

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31026 )

Change subject: AGESA/binaryPI: Add NULL pointers check
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31026/2/src/northbridge/amd/agesa/family16kb... 
File src/northbridge/amd/agesa/family16kb/northbridge.c:

https://review.coreboot.org/#/c/31026/2/src/northbridge/amd/agesa/family16kb... 
PS2, Line 576: 	dev = pcidev_on_root(1, 1);
> For each file, maybe reverse the check to print an error and get out? […]

It is not an error to have IGD 0:1.0 and associated HDA(?) 0:1.1 not present.