From mboxrd@z Thu Jan  1 00:00:00 1970
From: Sergei Shtylyov <sshtylyov@mvista.com>
Date: Tue, 21 Dec 2010 16:36:25 +0300
Subject: [U-Boot] [PATCH v2 1/9] mpc8xxx: Display RDIMM if detected
In-Reply-To: <20101218222153.41A4C1557C2@gemini.denx.de>
References: <1292630381-23022-1-git-send-email-yorksun@freescale.com>
	<20101218222153.41A4C1557C2@gemini.denx.de>
Message-ID: <4D10AD59.7040002@mvista.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello.

On 19-12-2010 1:21, Wolfgang Denk wrote:

>> Print a message when a RDIMM is detected.
> ...
>> +	} else
>>   		printf("ERROR:  Mix of registered buffered and unbuffered "
>>   				"DIMMs detected!\n");
>> -	}

> Mutli-line statement needs braces.

    Well, not only that. CodingStyle requires that if some parts of the *if* 
statement used {}, then all parts need to use {}.

> Best regards,

> Wolfgang Denk

WBR, Sergei