Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Marshall Dawson: Looks good to me, approved Frans Hendriks: Looks good to me, approved
vc/amd/agesa: Remove unused assignments

'Status' is assigned a value three times before it is checked.
Remove the first two assignments.

Change-Id: Id7136d62b4dbd6dce877983467960373b3a7ac22
Signed-off-by: Joe Moore <awokd@danwin1210.me>
Found-by: Coverity CID 1241809
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36257
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
---
M src/vendorcode/amd/agesa/f14/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
M src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
M src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
3 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/src/vendorcode/amd/agesa/f14/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c b/src/vendorcode/amd/agesa/f14/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
index 70aa595..440c5ff 100644
--- a/src/vendorcode/amd/agesa/f14/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
+++ b/src/vendorcode/amd/agesa/f14/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
@@ -474,13 +474,11 @@
if (ExtDisplayDeviceInfo == NULL) {
//Run out of supported display device types
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}

if ((Gfx->Gnb3dStereoPinIndex != 0) && (ConnectorEnumInfo.Engine->Type.Ddi.DdiData.HdpIndex == (Gfx->Gnb3dStereoPinIndex - 1))) {
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}

diff --git a/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c b/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
index f3af5e9..2d42063 100644
--- a/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
+++ b/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
@@ -488,13 +488,11 @@
if (ExtDisplayDeviceInfo == NULL) {
//Run out of supported display device types
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}

if ((Gfx->Gnb3dStereoPinIndex != 0) && (ConnectorEnumInfo.Engine->Type.Ddi.DdiData.HdpIndex == (Gfx->Gnb3dStereoPinIndex - 1))) {
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}

diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
index c9c9166..2350122 100644
--- a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
+++ b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxInitLibV1/GfxEnumConnectors.c
@@ -488,13 +488,11 @@
if (ExtDisplayDeviceInfo == NULL) {
//Run out of supported display device types
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}

if ((Gfx->Gnb3dStereoPinIndex != 0) && (ConnectorEnumInfo.Engine->Type.Ddi.DdiData.HdpIndex == (Gfx->Gnb3dStereoPinIndex - 1))) {
AGESA_STATUS_UPDATE (AGESA_ERROR, ConnectorEnumInfo.Status);
- Status = AGESA_ERROR;
PcieConfigDisableEngine (ConnectorEnumInfo.Engine);
}


To view, visit change 36257. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id7136d62b4dbd6dce877983467960373b3a7ac22
Gerrit-Change-Number: 36257
Gerrit-PatchSet: 3
Gerrit-Owner: awokd@danwin1210.me
Gerrit-Reviewer: Frans Hendriks <fhendriks@eltan.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged