Re: [PATCH] ARM: OMAP2+: Fix onenand initialization to avoid filesystem corruption

From: Tony Lindgren
Date: Wed Feb 10 2016 - 19:12:28 EST


* Aaro Koskinen <aaro.koskinen@xxxxxx> [160210 13:13]:
> Hi,
>
> On Mon, Feb 08, 2016 at 11:36:45AM -0800, Tony Lindgren wrote:
> > * Ivaylo Dimitrov <ivo.g.dimitrov.75@xxxxxxxxx> [160205 06:38]:
> > > Commit <e7b11dc7b77bfce0a351230a5feeadc1d0bba997> ("ARM: OMAP2+: Fix
> > > onenand rate detection to avoid filesystem corruption") partially fixed
> > > onenand configuration when GPMC module is reset. Finish the job by also
> > > providing the correct values in ONENAND_REG_SYS_CFG1 register.
> >
> > OK. So probably the INT or RDY polarity made the ECC not work.
> >
> > Aaro, care to dump out also the nolo configured CFG1 value from
> > n8x0 and n9(50)?
> >
> > You can do it by adding something like this to the beginning
> > of set_onenand_cfg():
> >
> > pr_info("CFG1: 0x%04x\n", readw(onenand_base + ONENAND_REG_SYS_CFG1));
> >
> > And presumably the values are the same as on n900. If not, we
> > should do the legacy file system flash test on all of them
> > before committing this fix.
>
> I got these (with CONFIG_OMAP_GPMC_DEBUG also enabled):
>
> N800:
> [ 2.803100] CFG1: 0x46c0
> [ 3.150115] CFG1: 0x06c0
>
> N810:
> [ 2.802368] CFG1: 0x46c0
> [ 3.148620] CFG1: 0x06c0
>
> N900:
> [ 4.965576] CFG1: 0x46c0
> [ 5.333740] CFG1: 0x06c0

Thanks, so these all should be fine with Ivaylo's patch.

> N950:
> [ 4.217193] CFG1: 0x66c4
> [ 4.583221] CFG1: 0x06c0
>
> N9:
> [ 4.212677] CFG1: 0x66c4
> [ 4.578613] CFG1: 0x06c0

Aaro, care to do one more test on n9(50) like Ivaylo posted
earlier:

1. Flash original maemo rootfs

2. Boot mainline kernel with Ivaylo's patch and mount onenand

3. Boot original maemo kernel and check dmesg there's no
file corruption

Also.. There's a chance somebody has created a onenand file system
with recent mainline kernels that did the reset and disabled ECC.
So with Ivaylo's patch fixing that, those may not mount properly
any longer. Most likely people just keep their maemo rootfs there
though with the MMC being available.

Regards,

Tony