[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