[coreboot] New patch to review for coreboot: ac98151 AMD f14 vendorcode: Fix warning

Martin Roth (martin.roth@se-eng.com) gerrit at coreboot.org
Sun Feb 24 00:42:29 CET 2013

Martin Roth (martin.roth at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2495


commit ac9815103552250a979d88348916842d8937daf2
Author: Martin Roth <martin.roth at se-eng.com>
Date:   Sat Feb 23 16:29:25 2013 -0700

    AMD f14 vendorcode: Fix warning
    Add brackets around initializer in #define for
    PCIE_DDI_DATA_INITIALIZER to fix the warning:
      PlatformGnbPcie.c:89, GNU Compiler 4 (gcc), Priority: Normal
      missing braces around initializer [-Wmissing-braces]
    This warning happens for Inagua and South Station
    Change-Id: I7d8f742dd8335b704b0493aa6e9eaebc3cc50b1e
    Signed-off-by: Martin Roth <martin.roth at se-eng.com>
 src/vendorcode/amd/agesa/f14/AGESA.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vendorcode/amd/agesa/f14/AGESA.h b/src/vendorcode/amd/agesa/f14/AGESA.h
index 5eaa638..3333976 100644
--- a/src/vendorcode/amd/agesa/f14/AGESA.h
+++ b/src/vendorcode/amd/agesa/f14/AGESA.h
@@ -813,7 +813,7 @@ typedef enum {
 #define  PCIE_PORT_DATA_INITIALIZER(mPortPresent, mChannelType, mDevAddress, mHotplug, mMaxLinkSpeed, mMaxLinkCap, mAspm, mResetId) \
 {mPortPresent, mChannelType, mDevAddress, 0, mMaxLinkSpeed, mAspm, mHotplug, mResetId, {0, mMaxLinkCap} }
 #define  PCIE_DDI_DATA_INITIALIZER(mConnectorType, mAuxIndex, mHpdIndex ) \
-{mConnectorType, mAuxIndex, mHpdIndex, {0, 0}}
+{mConnectorType, mAuxIndex, mHpdIndex, {{0}, {0}}}
 #define  PCIE_DDI_DATA_INITIALIZER_V1(mConnectorType, mAuxIndex, mHpdIndex, mMapping0, mMapping1) \
 {mConnectorType, mAuxIndex, mHpdIndex, {mMapping0, mMapping1}}

More information about the coreboot mailing list