Re: [PATCH 0/2] staging: mtd: Support for GigaDevice SPI NAND flash

[Greg KH <gregkh@linuxfoundation.org>]

On Thu, Nov 06, 2014 at 05:32:43PM +0000, Ionela Voinescu wrote:
> On 11/06/14 16:05, Greg KH wrote:
> >On Thu, Nov 06, 2014 at 03:51:17PM +0000, Ionela Voinescu wrote:
> >>Hi,
> >>
> >>These patches add support for GigaDevice 4Gb SPI NAND flash.
> >>This is an initial version, adding it to staging, as there is no existing
> >>framework for SPI NAND as there is for SPI NOR in MTD.
> >>
> >>Therefore, I want to start an early discussion about this in order to find
> >>the best way to integrate it. I am at the beginning regarding both upstreaming
> >>and understanding all of the MTD layer, so your comments and ideas are
> >>welcomed.
> >>
> >>I've structured this driver similar to the existing Micron SPI NAND
> >>(mt29f_spinand), in order to make it easier to understand the common code.
> >>I know there is an atempt to add support for a generic SPI NAND based
> >>framework from Sourav Poddar but that never got merged, and it would be
> >>a valid option to consider.
> >>
> >>http://lists.infradead.org/pipermail/linux-mtd/2013-July/047373.html
> >>
> >>This initial version does not implement all the functionality and I will
> >>add to it once we decide on a structure. This is based on linux-next.
> >>
> >>Thank you and I welcome all comments,
> >If you want to add code to the staging tree, you need a TODO file that
> >lists what is needed to be done to the driver in order to get it out of
> >the staging tree.  Otherwise I don't know why the code is in staging at
> >all, and not in the "real" part of the kernel.
> >
> >thanks,
> >
> >greg k-h
> Hi Greg,
> 
> For now I am submitting this for staging, but I am interested in having
> a discussion about the best way to integrate it and hopefully merge
> this in drivers/mtd/spinand/.

Then why not just go directly for drivers/mtd/?  Why even stop at
drivers/staging/ at all?  I need an ACK from the MTD maintainer to get
this into staging as well.

thanks,

greg k-h