* 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html