On Tue, Dec 02, 2014 at 09:21:50AM +0100, Boris Brezillon wrote: > On Tue, 2 Dec 2014 08:47:43 +0800 Huang Shijie <[email protected]> wrote: > > On Mon, Dec 01, 2014 at 09:58:58AM +0100, Boris Brezillon wrote: > > > On Mon, 1 Dec 2014 00:47:09 -0800 Brian Norris > > > <[email protected]> wrote: > > > > Also, is it time to yank / fixup some of these comments from > > > > gpmi-nand.c? > > > > > > I was asking myself the same question... > > > > > > > > > > > ... > > > > * FIXME: The following paragraph is incorrect, now that there exist > > > > * ecc.read_oob_raw and ecc.write_oob_raw functions. > > > > * > > > > * Since MTD assumes the OOB is not covered by ECC, there is no pair of > > > > * ECC-based/raw functions for reading or or writing the OOB. The fact > > > > that the > > > > * caller wants an ECC-based or raw view of the page is not propagated > > > > down to > > > > * this driver. > > > > */ > > > > > > I guess we can remove them. > > > Huang can you confirm that the raw access functions introduced in this > > > series are covering what's described here ? > > I think we can remove these comments now. > > Brian, do you want me to send this patch or are you taking care of it ?
Go ahead and send a patch. Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
