[coreboot-gerrit] Change in coreboot[master]: device/pci_device.c: Return if the scan parameter is invalid

[Arthur Heymans (Code Review)]

Attention is currently required from: Arthur Heymans.
Hello Arthur Heymans,

I'd like you to do a code review. Please visit

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

to review the following change.

Change subject: device/pci_device.c: Return if the scan parameter is invalid
......................................................................

device/pci_device.c: Return if the scan parameter is invalid

Clang is unhappy about codepath of an invalid parameter because
variables remain unset.

Change-Id: I1ba392a48cf3f81a29d9645e5cf220b122d588af
Signed-off-by: Arthur Heymans 
---
M src/device/pci_device.c
1 file changed, 2 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/63038/1

diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 22ccf59..90b26c1 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -1518,6 +1518,8 @@
    	primary = parent->secondary;
    	secondary = link->secondary;
    	subordinate = link->subordinate;
+	} else {
+		return;
    }

if (state == PCI_ROUTE_SCAN) {