From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Wed, 3 Jul 2013 17:12:56 -0500 Subject: [U-Boot] [PATCH v3 5/5] ARM: at91: atmel_nand: add code to check the ONFI parameter ECC requirement In-Reply-To: <1372821109-22007-6-git-send-email-josh.wu@atmel.com> (from josh.wu@atmel.com on Tue Jul 2 22:11:49 2013) References: <1372821109-22007-1-git-send-email-josh.wu@atmel.com> <1372821109-22007-6-git-send-email-josh.wu@atmel.com> Message-ID: <1372889576.8183.153@snotra> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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. -Scott