]> git.ipfire.org Git - people/ms/u-boot.git/commit - drivers/mtd/nand/davinci_nand.c
davinci_nand: cleanup II (CONFIG_SYS_DAVINCI_BROKEN_ECC)
authorDavid Brownell <dbrownell@users.sourceforge.net>
Tue, 28 Apr 2009 20:19:53 +0000 (13:19 -0700)
committerScott Wood <scottwood@freescale.com>
Tue, 7 Jul 2009 22:58:01 +0000 (17:58 -0500)
commit6e29ed8e576a6900c5d8dcde36b423ac576894dc
tree0fac6451d1e3ffc3a4cf4bacee2e387af6830041
parentfcb774777562bb7bcdc53c608d0e6bae906ce0f6
davinci_nand: cleanup II (CONFIG_SYS_DAVINCI_BROKEN_ECC)

Remove CONFIG_SYS_DAVINCI_BROKEN_ECC option.  It's not just nasty;
it's also unused by any current boards, and doesn't even match the
main U-Boot distributions from TI (which use soft ECC, or 4-bit ECC
on newer chips that support it).

DaVinci GIT kernels since 2.6.24, and mainline Linux since 2.6.30,
match non-BROKEN code paths for 1-bit HW ECC.  The BROKEN code paths
do seem to partially match what MontaVista/TI kernels (4.0/2.6.10,
and 5.0/2.6.18) do ... but only for small pages.  Large page support
is really broken (and it's unclear just what software it was trying
to match!), and the ECC layout was making three more bytes available
for use by filesystem (or whatever) code.

Since this option itself seems broken, remove it.  Add a comment
about the MV/TI compat issue, and the most straightforward way to
address it (should someone really need to solve it).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Scott Wood <scottwood@freescale.com>
doc/README.nand
drivers/mtd/nand/davinci_nand.c
include/asm-arm/arch-davinci/nand_defs.h