Re: [PATCH v2] mtd: rawnand: nandsim: Add support to disable subpage writes.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+Miquel

Kenneth, please Cc the NAND maintainers next time.

On Fri, 4 Jan 2019 09:53:02 +0100
Boris Brezillon <bbrezillon@xxxxxxxxxx> wrote:

> Hi Richard,
> 
> On Thu, 03 Jan 2019 21:43:13 +0100
> Richard Weinberger <richard@xxxxxx> wrote:
> 
> > > It's true that you could change the vid offset with ubiattach but that 
> > > require you know that the LEB missmatch error you get is fixed by a 
> > > different vid offset and you need to understand what that value should be.
> > > This way you only need to match the basic mtd parameters and the rest 
> > > will just work automatically.    
> > 
> > I agree that adding a parameter to nandsim to control subpage behavior
> > is nice.
> > But please keep in mind that nandsim will never perfectly match your hardware.
> > 
> > Boris, unless you have objections I'm fine to control NAND_NO_SUBPAGE_WRITE
> > in nandsim using a module parameter.  
> 
> Hm, I'd really like to have nandsim replaced by something more generic
> at some point (mtdsim or MTD/NAND emulation in qemu using a virtio
> iface), so adding new features/option to this driver is something I'd
> like to avoid. This being said, this new MTD/NAND emulation
> infrastructure is not there yet, and I don't think I'll have time to
> work on it (I started working on mtdsim a while back, and someone took
> over this work, but it stopped at some point).
> 
> To sum-up, if you think this is absolutely necessary (given the
> discussion I'm not convinced it is) I'm okay to add this module param,
> but I'd prefer if people were working on a solution to replace all
> those emulation layers we have right now (mtdram, nandsim, ...).
> 
> Regards,
> 
> Boris


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/



[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux