[PATCH 2/4] mtd: mxc_nand: limit the size of used oob
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Mon Apr 27 12:31:27 PDT 2015
Hello,
On Mon, Apr 27, 2015 at 02:43:41PM +0300, Baruch Siach wrote:
> On Mon, Apr 27, 2015 at 09:50:23AM +0200, Uwe Kleine-König wrote:
> > On Mon, Apr 27, 2015 at 10:20:57AM +0300, Baruch Siach wrote:
> > > On Mon, Apr 27, 2015 at 09:12:38AM +0200, Uwe Kleine-König wrote:
> > > > Hmm I rechecked the reference manual and found a register to specify the
> > > > size of the spare area (I didn't notice that one before). Did you try
> > > > what happens if you set this to 0x70 for 224 bytes oob?
> > >
> > > Which register is that?
> > Spare Area Size Register (SPAS) at offset 0x1e10 for the i.MX25 (that's
> > what you're using, don't you?).
>
> Yes, that's what I'm using.
>
> I tried setting the SPAS register to oobsize/2 (0x70 in my case), but I see no
> change in behaviour. Moreover, it turns out the previously Barebox set this
> register (apparently wrongly) to 0x20 for spare size of 64. Current Barebox
> master still do. For v3 Barebox limits CONFIG2_SPAS to 218 bytes spare size
I guess for v21 writing to SPAS is just what the controller assumes
after reset. I.e. that a 512-byte nand has 16 bytes spare and a 2k-nand
has 64.
> since Eric Bénard's commit 632c45795065 (nand_imx: update to support onfi & 4k
> flashs, 2012-07-05). As you can see, the kernel doesn't touch this register
> for v2 NFC.
right.
> I have no idea what is the effect of the SPAS in v2 (or any other) NFC.
I didn't try, but I'm surprised it doesn't make a difference. How did
you test? I'd do in barebox:
memset -w 0xbb000000 0x55 0x1200
set SPAS to 218/2
trigger page read
check how much of the 0x55 was overwritten
set SPAS to 224/2
trigger page read
compare with above
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-arm-kernel
mailing list