[U-Boot] [PATCH v7 2/5] nand: lpc32xx: add hardware ECC support

[LEMIEUX, SYLVAIN]

> -----Original Message-----
> From: Vladimir Zapolskiy [mailto:vz at mleia.com]
> Sent: 14-Aug-15 11:58 AM
> To: slemieux.tyco at gmail.com; u-boot at lists.denx.de
> Cc: scottwood at freescale.com; marex at denx.de; albert.u.boot at aribaud.net; LEMIEUX, SYLVAIN; Tom Rini
> Subject: Re: [PATCH v7 2/5] nand: lpc32xx: add hardware ECC support
>
> Hi Sylvain,
>
> On 13.08.2015 22:40, slemieux.tyco at gmail.com wrote:
> > From: Sylvain Lemieux <slemieux at tycoint.com>
> >
> > Incorporate NAND SLC hardware ECC support from legacy LPCLinux NXP BSP.
> > The code taken from the legacy patch is:
> > - lpc32xx SLC NAND driver (hardware ECC support)
> > - lpc3250 header file missing SLC NAND registers definition
> >
> > The legacy driver was updated and clean-up as part of the integration with the existing NAND SLC driver.
> >
> > Signed-off-by: Sylvain Lemieux <slemieux at tycoint.com>
>
> Looks good, tested on non-SPL build (SPL build produces a programmed
> #warning and the following compilation error due to missing DMA driver
> in SPL)

CONFIG_SPL_DMA_SUPPORT need to be define in "devkit3250.h"
when enabling the DMA for an SPL build.

 The warning will be display, as the DMA support in SPL image is not tested.

>
> Tested-by: Vladimir Zapolskiy <vz at mleia.com>
>
> With best wishes,
> Vladimir

________________________________

This e-mail contains privileged and confidential information intended for the use of the addressees named above. If you are not the intended recipient of this e-mail, you are hereby notified that you must not disseminate, copy or take any action in respect of any information contained in it. If you have received this e-mail in error, please notify the sender immediately by e-mail and immediately destroy this e-mail and its attachments.