Re: [PATCH] staging: Remove the mt29f_spinand driver

[Miquel Raynal <miquel.raynal@xxxxxxxxxxx>]

Hi Greg,

Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote on Mon, 5 Nov
2018 14:29:49 +0100:

> On Mon, Nov 05, 2018 at 11:12:27AM +0100, Miquel Raynal wrote:
> > Hi Greg,
> > 
> > Boris Brezillon <boris.brezillon@xxxxxxxxxxx> wrote on Mon, 22 Oct 2018
> > 22:10:59 +0200:
> >   
> > > A new SPI NAND subsystem has been added in drivers/mtd/nand/spi/ and
> > > Micron's MT29F devices are now supported in
> > > drivers/mtd/nand/spi/micron.c.
> > > 
> > > Remove the old driver.
> > > 
> > > Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxx>
> > > ---
> > > Hello,
> > > 
> > > If anything is missing in drivers/mtd/nand/spi/micron.c to properly
> > > support the devices supported by the mt29f_spinand driver, please let
> > > me know.
> > > I might accept to delay removal of this driver if I have some guarantees
> > > that existing users will actually switch to the new driver at some
> > > point.
> > > 
> > > Regards,
> > > 
> > > Boris
> > > ---  
> > 
> > I plan to apply this patch but I would like your approval first.
> > 
> > As a summary, the mt29f_spinand driver is a Micron SPI NAND chip
> > driver interfacing with the raw NAND API (which is 'wrong').
> > 
> > Boris has recently contributed a SPI NAND framework supporting SPI
> > NAND chips from several vendors, including Micron, that is supposed to
> > take over this driver.
> > 
> > Do you see anything that should prevent us to remove it now?  
> 
> Not at all, I was going to add this patch to my tree right now, as I
> couldn't do anything until after 4.20-rc1 was out.  Any objection from
> me just taking it that way and getting it into 4.20-final?

I'm fine with you taking it but I have changes in the pipe that are
impacted by this removal so it would be great if this could happen
pretty early in the 4.20 release cycle (4.20-rc2?), so I will still be
able to base nand/next on top of it.


Thanks,
Miquèl

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