Re: [PATCH] mtd: denali: Disable sub-page writes in Denali NAND driver

From: Ricard Wanderlof
Date: Fri Feb 06 2015 - 03:29:49 EST



On Thu, 5 Feb 2015, Graham Moore wrote:

> Actually, we made this change to make UBIFS work. So, yes, the driver
> never worked for UBI. Worked fine for JFFS2, raw data.
>
> A customer reported an issue with ECC errors when using UBIFS on NAND
> flash with Altera SoC.
>
> We debugged it and found the ECC errors occur because the UBI subsystem
> is trying to write sub-pages in the NAND, but neither the NAND chip
> itself nor the Denali NAND controller support sub-page writes.

Just a bit curious.

It is not uncommon for controllers or chips not to support sub-page
writes. In that case however, the partition(s) used by UBI should be
formatted accordingly, i.e. using the appropriate --sub-page-size argument
to ubiformat (when formatting partitions on the system itself), or the
corresponding argument to ubinize (when preparing images offline).

If that is done correctly, then the lack of subpage write capability is
not a problem per se (of course, the UBI EC and VID headers then take up
more space so less space is available for user data; on a flash with 2k
pages it is only 2k bytes per LEB that is lost however).

/Ricard
--
Ricard Wolf Wanderlöf ricardw(at)axis.com
Axis Communications AB, Lund, Sweden www.axis.com
Phone +46 46 272 2016 Fax +46 46 13 61 30
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/