summaryrefslogtreecommitdiff
path: root/src/southbridge
diff options
context:
space:
mode:
authorJacob Garber <jgarber1@ualberta.ca>2019-03-12 22:27:52 -0600
committerPatrick Georgi <pgeorgi@google.com>2019-03-19 21:43:57 +0000
commit14e826f3f8891e29fb8838fba3beb4ab62ee2d27 (patch)
tree1ab9a6e0bb97669eef85ca6d649f536b9e72b34d /src/southbridge
parent8d0f59935d340287b15e21f19e2f2c28fcb45cce (diff)
downloadcoreboot-14e826f3f8891e29fb8838fba3beb4ab62ee2d27.tar.xz
src/southbridge/intel/i82801gx/pcie.c: Correct NULL check
Check if `pcie_dev` is NULL instead of `dev`. This was flagged as REVERSE_INULL during a Coverity scan, but is a simple typo. Signed-off-by: Jacob Garber <jgarber1@ualberta.ca> Change-Id: Idc40574b9341d1b10cb2136cbc1a865efa3ab3ee Reviewed-on: https://review.coreboot.org/c/coreboot/+/31866 Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/southbridge')
-rw-r--r--src/southbridge/intel/i82801gx/pcie.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/southbridge/intel/i82801gx/pcie.c b/src/southbridge/intel/i82801gx/pcie.c
index 4679ee58f8..9446527c7b 100644
--- a/src/southbridge/intel/i82801gx/pcie.c
+++ b/src/southbridge/intel/i82801gx/pcie.c
@@ -195,7 +195,7 @@ static void root_port_commit_config(struct device *dev)
pcie_dev = rpc.ports[i];
- if (dev == NULL) {
+ if (pcie_dev == NULL) {
printk(BIOS_ERR, "Root Port %d device is NULL?\n",
i + 1);
continue;