From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Wu Date: Thu, 4 Jul 2013 15:28:57 +0800 Subject: [U-Boot] [PATCH v3 5/5] ARM: at91: atmel_nand: add code to check the ONFI parameter ECC requirement In-Reply-To: <1372889576.8183.153@snotra> References: <1372821109-22007-1-git-send-email-josh.wu@atmel.com> <1372821109-22007-6-git-send-email-josh.wu@atmel.com> <1372889576.8183.153@snotra> Message-ID: <51D52439.2050509@atmel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 7/4/2013 6:12 AM, Scott Wood wrote: > On 07/02/2013 10:11:49 PM, Josh Wu wrote: >> + if (!get_onfi_ecc_param(chip, cap, sector_size)) >> + MTDDEBUG(MTD_DEBUG_LEVEL1, "ONFI params, minimum >> required ECC: %d bits in %d bytes\n", >> + *cap, *sector_size); > > Use braces around multi-line if/loop bodies, even if it's a single > statement. > > Otherwise, ACK this patchset. thanks, I'll resend this patch with braces around. > > -Scott Best Regards, Josh Wu