[coreboot-gerrit] New patch to review for coreboot: 265edf4 AMD AGESA: Fix comment for `PCIE_DDI_DATA_INITIALIZER`

Bruce Griffith (Bruce.Griffith@se-eng.com) gerrit at coreboot.org
Thu Aug 1 14:56:27 CEST 2013


Bruce Griffith (Bruce.Griffith at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3840

-gerrit

commit 265edf4b2345ef47c44d17ed7e96474f0a1e3870
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Tue Jul 30 04:17:37 2013 -0600

    AMD AGESA: Fix comment for `PCIE_DDI_DATA_INITIALIZER`
    
    Copied from a similar commit for Family 10h AGESA [1]
    
    Remove the fourth argument in the comments. Luckily the compiler,
    at least gcc, warns about a wrong number of arguments, and therefore
    no incorrect code resulted from the wrong documentation.
    
    [1] 07e0f1b AMD AGESA: Fix argument list for `PCIE_DDI_DATA_INITIALIZER` in comments
    [2] fc47bfa Revert "AMD f14 vendorcode: Fix warning"
    
    Change-Id: I3806e368a823e4a40d22e99b91bf3598d9ed2f15
    Signed-off-by: Bruce Griffith <bruce.griffith at se-eng.com>
---
 src/vendorcode/amd/agesa/f16kb/AGESA.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/vendorcode/amd/agesa/f16kb/AGESA.h b/src/vendorcode/amd/agesa/f16kb/AGESA.h
index 34dc3b6..d71250c 100644
--- a/src/vendorcode/amd/agesa/f16kb/AGESA.h
+++ b/src/vendorcode/amd/agesa/f16kb/AGESA.h
@@ -1305,13 +1305,13 @@ typedef struct {
                                                         *    {
                                                         *      0,   //Descriptor flags
                                                         *      PCIE_ENGINE_DATA_INITIALIZER (PcieDdiEngine, 24, 27),
-                                                        *      PCIE_DDI_DATA_INITIALIZER (ConnectorTypeDP, Aux1, Hdp1, 0)
+                                                        *      PCIE_DDI_DATA_INITIALIZER (ConnectorTypeDP, Aux1, Hdp1)
                                                         *    },
                                                         *    // Initialize Ddi descriptor (DDI interface Lanes 28:31, HDMI, ...)
                                                         *    {
                                                         *      DESCRIPTOR_TERMINATE_LIST, //Descriptor flags  !!!IMPORTANT!!! Terminate last element of array
                                                         *      PCIE_ENGINE_DATA_INITIALIZER (PcieDdiEngine, 28, 31),
-                                                        *      PCIE_DDI_DATA_INITIALIZER (ConnectorTypeHDMI, Aux2, Hdp2, 0)
+                                                        *      PCIE_DDI_DATA_INITIALIZER (ConnectorTypeHDMI, Aux2, Hdp2)
                                                         *    }
                                                         *  };
                                                         * PCIe_COMPLEX_DESCRIPTOR PlatformTopology = {



More information about the coreboot-gerrit mailing list