[U-Boot] [PATCH] arm: omap: cm_t35: Fix: Re-add GPMC_NAND_ECC_LP_x8_LAYOUT

Gupta, Pekon pekon at ti.com
Thu Mar 13 12:49:22 CET 2014


>From: Stefan Roese [mailto:sr at denx.de]
>
>Patch a7e36fc9 (mtd: nand: omap: remove unused #defines from common
>omap_gpmc.h) removed some MTD related defines. Including
>GPMC_NAND_ECC_LP_x8_LAYOUT. But this define is also needed for the
>memory controller configuration (only the x8 defines are needed,
>the x16 defines are the default). Without it the NAND subsystem is
>not configured correctly and booting into U-Boot does not work.
>
The root cause of this problem is in  ..
	arch/arm/cpu/armv7/omap3/mem.c
#if defined(GPMC_NAND_ECC_SP_x8_LAYOUT) || defined(GPMC_NAND_ECC_LP_x8_LAYOUT)
where based on GPMC_NAND_ECC_LP_x8_LAYOUT, different
controller configuration is selected. Unless you need a urgent fix,

- I would suggest to use [1] instead. I'll try to get a cleaner version
of [1] incorporating other comments from Scott, soon.

- Alternatively, you can also look at [2], which is more robust but
was still under discussion.


[1] http://lists.denx.de/pipermail/u-boot/2013-December/168507.html

[2] 
http://lists.denx.de/pipermail/u-boot/2013-September/163882.html
http://lists.denx.de/pipermail/u-boot/2013-September/163879.html


with regards, pekon


More information about the U-Boot mailing list