[coreboot] New patch to review for coreboot: 3de5255 Persimmon: dimmSpd.c: Use spaces for alignment of if-predicate

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Thu Feb 21 14:37:18 CET 2013


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2471

-gerrit

commit 3de525502e8e92c4556d1dc9957b5e5f218aa97b
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Thu Feb 21 14:01:39 2013 +0100

    Persimmon: dimmSpd.c: Use spaces for alignment of if-predicate
    
    The relational operators in the if-predicate are aligned in all
    `dimmSpd.c` files so revert part of the change in
    
        commit 36abff1dc8e74beafa47ad83de17416681970916
        Author: Marc Jones <marcj303 at gmail.com>
        Date:   Mon Nov 7 23:26:14 2011 -0700
    
            Cleanup Persimmon mainboard whitespace.
    
            Reviewed-on: http://review.coreboot.org/427
    
    to remove the incorrectly introduced tabs and to unify that. It
    might contradict the current coding style but it is even used in
    the latest code as seen in the following file.
    
         src/northbridge/amd/agesa/family15tn/dimmSpd.c
    
    Change-Id: Ib611267f99090d0830bdc2319527389f193ea1eb
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/mainboard/amd/persimmon/dimmSpd.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mainboard/amd/persimmon/dimmSpd.c b/src/mainboard/amd/persimmon/dimmSpd.c
index 4aaa682..a021cc2 100644
--- a/src/mainboard/amd/persimmon/dimmSpd.c
+++ b/src/mainboard/amd/persimmon/dimmSpd.c
@@ -152,9 +152,9 @@ AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PA
 {
 	int spdAddress, ioBase;
 
-	if (info->SocketId	 >= DIMENSION (spdAddressLookup		)) return AGESA_ERROR;
-	if (info->MemChannelId >= DIMENSION (spdAddressLookup[0]	)) return AGESA_ERROR;
-	if (info->DimmId		>= DIMENSION (spdAddressLookup[0][0])) return AGESA_ERROR;
+	if (info->SocketId     >= DIMENSION (spdAddressLookup      )) return AGESA_ERROR;
+	if (info->MemChannelId >= DIMENSION (spdAddressLookup[0]   )) return AGESA_ERROR;
+	if (info->DimmId       >= DIMENSION (spdAddressLookup[0][0])) return AGESA_ERROR;
 
 	spdAddress = spdAddressLookup [info->SocketId] [info->MemChannelId] [info->DimmId];
 	if (spdAddress == 0) return AGESA_ERROR;



More information about the coreboot mailing list