On Wed, Sep 02, 2015 at 09:57:01AM +0200, Boris Brezillon wrote: > Also drop the oob_required contidion, since ->oob_pio already contains > FFs when oob_required is false.
I was pointing out this issues not just because you didn't note it in the commit message, but because the previous behavior actually potentially has user-visible effects. Particularly, because you had the conditions backwards, you were programming 0xff to the spare area, no matter what the user asked for. Normally, this isn't a big issue, since most users aren't storing data in the OOB region. But there may be cases where your driver was subtly broken (marking bad blocks, perhaps?), and this patch is subtly fixing them. Especially since this will get backported to -stable, it'd be nice to have this noted completely. Did this driver pass mtd_oobtest previously? Brian -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
