[PATCH v10] MTD: LPC32xx MLC NAND driver

On Fri, Jun 29, 2012 at 10:04 AM, Artem Bityutskiy [off-list ref] wrote:

On Fri, 2012-06-29 at 09:56 -0300, Alexandre Pereira da Silva wrote:

quoted

quoted

@@ @@
+drivers/mtd/nand/lpc32xx_mlc.c: In function 'lpc32xx_nand_probe':
+drivers/mtd/nand/lpc32xx_mlc.c:679:10: warning: variable 'sr' set but not used [-Wunused-but-set-variable]

gcc 4.6, aiaiai uses W=1 when building.

quoted

quoted

+drivers/mtd/nand/lpc32xx_mlc.c:561:24: error: bad constant expression [sparse]

This is sparse, not gcc.

Does the oob area need to be ecc protected? If not I suggest to just
remove the read_oob and write_oob and let this to nand_base. I think
this is too much overhead to read 6 bytes.