Commit 380b4b40c60d ("AmpereAltraPkg: Update Ampere specific platform PCIe core", 2023-03-29) removed the DEBUG macro call from Ac01PcieCoreUpdateLink() that was the only reader of "Val". Since then, Val has only been set, but never read. This triggers "-Werror=unused-but-set-variable".
Found when trying to build "Jade.dsc". Keep the MmioRead32() call, so that the PCIe config space sees the same behavior as before, but explicitly throw away the result. Cc: Chuong Tran <chu...@os.amperecomputing.com> Cc: Leif Lindholm <quic_llind...@quicinc.com> Cc: Nhi Pham <n...@os.amperecomputing.com> Cc: Rebecca Cran <rebe...@os.amperecomputing.com> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=4564 Signed-off-by: Laszlo Ersek <ler...@redhat.com> --- Silicon/Ampere/AmpereAltraPkg/Library/Ac01PcieLib/PcieCore.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/Ac01PcieLib/PcieCore.c b/Silicon/Ampere/AmpereAltraPkg/Library/Ac01PcieLib/PcieCore.c index 3c1fe5e35ca2..fa00c1e36999 100644 --- a/Silicon/Ampere/AmpereAltraPkg/Library/Ac01PcieLib/PcieCore.c +++ b/Silicon/Ampere/AmpereAltraPkg/Library/Ac01PcieLib/PcieCore.c @@ -1747,7 +1747,6 @@ Ac01PcieCoreUpdateLink ( PHYSICAL_ADDRESS CfgBase; UINT8 PcieIndex; UINT32 Index; - UINT32 Val; *IsNextRoundNeeded = FALSE; *FailedPcieCount = 0; @@ -1767,7 +1766,7 @@ Ac01PcieCoreUpdateLink ( if (Pcie->Active && !Pcie->LinkUp) { if (PcieLinkUpCheck (Pcie)) { Pcie->LinkUp = TRUE; - Val = MmioRead32 (CfgBase + PCIE_CAPABILITY_BASE + LINK_CONTROL_LINK_STATUS_REG); + (VOID)MmioRead32 (CfgBase + PCIE_CAPABILITY_BASE + LINK_CONTROL_LINK_STATUS_REG); // Doing link checking and recovery if needed Ac01PcieCoreQoSLinkCheckRecovery (RootComplex, PcieIndex); -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109551): https://edk2.groups.io/g/devel/message/109551 Mute This Topic: https://groups.io/mt/101914660/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-