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

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

Hello Piotr Król, Angel Pons, build bot (Jenkins), Michał Żygowski,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/31026

to look at the new patch set (#2).

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

AGESA/binaryPI: Add NULL pointers check

Fix regression after commits
  4ad7f5b AGESA: Use pcidev_on_root()
  33ff44c binaryPI: Use pcidev_on_root()

Previously used call dev_find_slot() returned
pointers to PCI device nodes that were actually
not present in the hardware at all. Register
reads would come back with invalid (0xff) values
and writes would be ignored.

After change to pcidev_on_root(), attempting to
do register operations with non-present PCI
hardware immediately halts with error
  get_pbus: dev is NULL!

Change-Id: I785350c171a642207c5fab884a953d45a3bfe592
Signed-off-by: Kyösti Mälkki 
---
M src/northbridge/amd/agesa/family16kb/northbridge.c
M src/northbridge/amd/pi/00660F01/northbridge.c
M src/northbridge/amd/pi/00730F01/northbridge.c
3 files changed, 15 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/31026/2