RE: [PATCH] Staging: MTD: Micron SPINAND Driver support

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

 



> From: linux-mtd  On Behalf Of Brian Norris
> > On Tue, Oct 01, 2013 at 03:03:58PM +0530, Kamlakant Patel wrote:
> > This patch adds support for Micron SPINAND via MTD.
> >
> > Signed-off-by: Mona Anonuevo <manonuevo@xxxxxxxxxx>
> > Signed-off-by: Kamlakant Patel <kamlakant.patel@xxxxxxxxxxxx>
> > ---
> > This patch has to be merged via staging tree.
> >
> > This is a driver from Micron for MT29F1G01ZACH4 SPI based NAND chips.
> This driver had
> > been posted multiple times to the mtd list.
> > 1. http://lists.infradead.org/pipermail/linux-mtd/2010-May/031975.html
> > 2. http://lists.infradead.org/pipermail/linux-mtd/2010-April/029523.html
> > 3. patchwork.ozlabs.org/patch/258697/
> > This has not been merged into the main kernel yet.
> >
> > I have cleaned and updated it for current kernel. Since there are many
> users for
> > this driver, it may be useful to add it to the staging tree, where further fixes
> and
> > cleanups can be done. Once it reaches to the standard will be moved to the
> mtd.
> >
> > This driver has been tested with Micron SPINAND MT29F1G01ZACH4 chip
> on kernel 3.12 on
> > the Netlogic XLP platforms.
> >
> > Mona Anonuevo, I have retained your sign-offs from the original patch.
> >
I see that major portion (especially the generic framework) is ported from
http://lists.infradead.org/pipermail/linux-mtd/2010-May/031975.html
So shouldn't you just maintain the original authorship ?
And the original headers ?

> 
> You can address these comments in follow up patches after it's included
> in staging, though. There's probably more review needed anyway
> eventually (I see that no one really has reviewed this on the MTD
> mailing list yet).
> 
The original version of this patch in 2010 added *generic spinand*
framework. And support for m29f (Micron devices) should have been
added using that generic framework. But due to lack of response
generic framework never went in.
Therefore I think this patch series adds a customized driver just for
'm29f' (Micron serial NAND) devices, removing the generalization.

There was a refreshed version of original patch (with generic framework)
but no response on that too. I don't want to hold this patch, but it would
be good, if you can review generic framework also ..
https://lkml.org/lkml/2013/6/26/88


with regards, pekon
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux