[coreboot-gerrit] New patch to review for coreboot: ad2a165 northbridge/amd/agesa/family12/dimmSpd.c: Fix indent

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Sat Jan 3 09:20:25 CET 2015


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8050

-gerrit

commit ad2a1655ba9bc6ad11bc10222f626609a55be618
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat Jan 3 19:19:25 2015 +1100

    northbridge/amd/agesa/family12/dimmSpd.c: Fix indent
    
    This is automatically done by 'clang-format' as a demo.
    
    Change-Id: Ie723e01173f770ae040bd527d952578aeae87427
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/northbridge/amd/agesa/family12/dimmSpd.c | 56 ++++++++++++++--------------
 1 file changed, 27 insertions(+), 29 deletions(-)

diff --git a/src/northbridge/amd/agesa/family12/dimmSpd.c b/src/northbridge/amd/agesa/family12/dimmSpd.c
index e8ef02a..084fec7 100644
--- a/src/northbridge/amd/agesa/family12/dimmSpd.c
+++ b/src/northbridge/amd/agesa/family12/dimmSpd.c
@@ -14,7 +14,8 @@
  *       its contributors may be used to endorse or promote products derived
  *       from this software without specific prior written permission.
  *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ *AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
  * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
  * DISCLAIMED. IN NO EVENT SHALL ADVANCED MICRO DEVICES, INC. BE LIABLE FOR ANY
@@ -34,45 +35,42 @@
 
 #include <northbridge/amd/agesa/dimmSpd.h>
 
-typedef struct _DIMM_INFO_SMBUS{
-  UINT8   SocketId;
-  UINT8   MemChannelId;
-  UINT8   DimmId;
-  UINT8   SmbusAddress;
+typedef struct _DIMM_INFO_SMBUS
+{
+	UINT8 SocketId;
+	UINT8 MemChannelId;
+	UINT8 DimmId;
+	UINT8 SmbusAddress;
 } DIMM_INFO_SMBUS;
 /*
 * SPD address table - porting required
 */
-STATIC CONST DIMM_INFO_SMBUS SpdAddrLookup [] =
-{
-  /* Socket, Channel, Dimm, Smbus */
-  {0, 0, 0, 0xA0},
-  {0, 1, 0, 0xA2}
-};
+STATIC CONST DIMM_INFO_SMBUS SpdAddrLookup[] = {
+    /* Socket, Channel, Dimm, Smbus */
+    {0, 0, 0, 0xA0},
+    {0, 1, 0, 0xA2}};
 
 AGESA_STATUS
-AmdMemoryReadSPD (
-  IN UINT32 Func,
-  IN UINT32 Data,
-  IN OUT AGESA_READ_SPD_PARAMS *SpdData
-  )
+AmdMemoryReadSPD(IN UINT32 Func, IN UINT32 Data,
+		 IN OUT AGESA_READ_SPD_PARAMS *SpdData)
 {
-   UINT8  SmBusAddress = 0;
-   UINTN  Index;
-   UINTN  MaxSocket = ARRAY_SIZE(SpdAddrLookup);
-   for (Index = 0; Index < MaxSocket; Index ++){
-     if ((SpdData->SocketId     == SpdAddrLookup[Index].SocketId)     &&
-         (SpdData->MemChannelId == SpdAddrLookup[Index].MemChannelId) &&
-         (SpdData->DimmId       == SpdAddrLookup[Index].DimmId)) {
-        SmBusAddress = SpdAddrLookup[Index].SmbusAddress;
-        break;
-      }
-   }
+	UINT8 SmBusAddress = 0;
+	UINTN Index;
+	UINTN MaxSocket = ARRAY_SIZE(SpdAddrLookup);
+	for (Index = 0; Index < MaxSocket; Index++) {
+		if ((SpdData->SocketId == SpdAddrLookup[Index].SocketId) &&
+		    (SpdData->MemChannelId ==
+		     SpdAddrLookup[Index].MemChannelId) &&
+		    (SpdData->DimmId == SpdAddrLookup[Index].DimmId)) {
+			SmBusAddress = SpdAddrLookup[Index].SmbusAddress;
+			break;
+		}
+	}
 
 	if (SmBusAddress == 0)
 		return AGESA_ERROR;
 
-	int err = smbus_readSpd(SmBusAddress, (char *) SpdData->Buffer, 128);
+	int err = smbus_readSpd(SmBusAddress, (char *)SpdData->Buffer, 128);
 	if (err)
 		return AGESA_ERROR;
 	return AGESA_SUCCESS;



More information about the coreboot-gerrit mailing list