From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= Date: Fri, 16 Nov 2012 21:15:33 +0100 (CET) Subject: [U-Boot] [PATCH v2 13/13] mxc nand: Add support for i.MX5 In-Reply-To: <1353018123.7487.12@snotra> Message-ID: <822775189.1498148.1353096933515.JavaMail.root@advansee.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Scott, On Thursday, November 15, 2012 11:22:03 PM, Scott Wood wrote: > On 09/18/2012 01:11:21 PM, Tom Rini wrote: > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > On 09/17/12 17:36, Scott Wood wrote: > > > On Tue, Aug 21, 2012 at 11:04:14PM +0200, Beno?t Th?baudeau > > > wrote: > > >> Signed-off-by: Beno?t Th?baudeau > > >> Cc: Scott Wood > > >> Cc: Stefano Babic --- > > >> Changes for v2: - Fix warning for unused tmp variable in > > >> board_nand_init() for NFC V1. > > >> > > >> .../arch/arm/include/asm/arch-mx5/imx-regs.h | 9 + > > >> .../drivers/mtd/nand/mxc_nand.c | 219 > > >> +++++++++++++++----- .../include/fsl_nfc.h > > >> | 149 ++++++++----- .../nand_spl/nand_boot_fsl_nfc.c > > >> | 114 +++++++--- 4 files changed, 365 insertions(+), 126 > > >> deletions(-) > > > > > > Unless Tom or Wolfgang object, I'm inclined to drop the objection > > > to adding new hardware support to nand_spl in this case. I'd > > > rather see the support be merged rather than ignored because a > > > contributor has time for a small job but not a large one. Plus, > > > this code will likely be reused by the new SPL support, if this > > > platform is space constrained, so it's not dead-end effort. > > > > I'm OK with pulling this in while providing an on-the-record prod > > to > > please find some time to do a conversion or two in time for > > v2013.01. > > I'm going to leave this one until there's a config added that uses > it, > so that we at least get compile-testing. OK. I won't be able to add my i.MX51 board to mainline before at least a few months. It's hard to find the schematics of all the i.MX5 boards supported by mainline U-Boot. According to the links below, at least the Genesi EFIKA MX Smartbook and the Freescale MX53 ARD boards have embedded NAND. Matt, Fabio, is it possible to find the schematics of these boards somewhere? http://www.genesi-tech.com/products/smartbook https://community.freescale.com/thread/289468 The EFIKA MX Smartbook probably has MLC if we consider its NAND size. 4-kiB pages should be supported by nand_spl/nand_boot_fsl_nfc.c, but not yet by drivers/mtd/nand/mxc_nand.c. The latter currently has a CONFIG_SYS_NAND_LARGEPAGE to differentiate 512-B and 2-kiB pages. If we were to add support for 4-kiB pages to this driver, I think that this should be replaced with a CONFIG_SYS_NAND_PAGE_SIZE like in nand_boot_fsl_nfc.c. Do you agree? Best regards, Beno?t