[U-Boot] [RFC PATCH 00/20] SPI-NAND support

[Tom Rini <trini@konsulko.com>]

On Fri, Jul 06, 2018 at 05:45:02PM +0530, Jagan Teki wrote:
> On Fri, Jul 6, 2018 at 5:36 PM, Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> > Hi Jagan,
> >
> > Jagan Teki <jagan@amarulasolutions.com> wrote on Fri, 6 Jul 2018
> > 17:13:21 +0530:
> >
> >> On Wed, Jun 6, 2018 at 9:00 PM, Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> >> > During the last months, Boris Brezillon shared his work to support
> >> > serial flashes within Linux. First, he delivered (and merged) a new
> >> > layer called spi-mem. He also initiated in Linux MTD subsystem the move
> >> > of all 'raw' NAND related code to a raw/ subdirectory, adding at the
> >> > same time a NAND core that would be shared with all NAND devices. Then,
> >> > he contributed a generic SPI-NAND driver, making use of this NAND core,
> >> > as well as some vendor code to drive a few chips.
> >> >
> >> > On top of this work, I added an 'mtd' U-Boot command to handle all sort
> >> > of MTD devices. This should become the default command instead of having
> >> > one per flash flavor ('sf', 'nand', 'spi-nand' ?).
> >> >
> >> > The series has been tested on an Ocelot board PCB123 (VSC7514),
> >> > featuring a Macronix SPI NAND chip.
> >> >
> >> > TL;DR: the series contains:
> >> > - Various fixes and re-organization of the MTD subsystem.
> >> > - The introduction of the SPI-mem interface.
> >> > - The addition of the generic SPI-NAND driver (and its bindings).
> >> > - Several SPI NAND chip drivers (Macronix, Micron, Winbond).
> >> > - A new 'mtd' command.
> >> > - DT changes to make use of a SPI NAND on the Ocelot board.
> >> >
> >> > Any comments on the code, the organization and the respect of U-Boot
> >> > driver model will be welcome.
> >> >
> >> > Thanks,
> >> > Miquèl
> >> >
> >> >
> >> > Boris Brezillon (6):
> >> >   mtd: Fallback to ->_read/write_oob() when ->_read/write() is missing
> >> >   mtd: nand: Add core infrastructure to deal with NAND devices
> >> >   mtd: nand: Pass mode information to nand_page_io_req
> >> >   spi: Extend the core to ease integration of SPI memory controllers
> >> >   mtd: spinand: Add initial support for the MX35LF1GE4AB chip
> >> >   dt-bindings: Add bindings for SPI NAND devices
> >> >
> >> > Brian Norris (1):
> >> >   mtd: add get/set of_node/flash_node helpers
> >> >
> >> > Frieder Schrempf (1):
> >> >   mtd: spinand: Add initial support for Winbond W25M02GV
> >> >
> >> > Miquel Raynal (10):
> >> >   mtd: fix build issue with includes
> >> >   mtd: move definitions to enlarge their range
> >> >   mtd: move all flash categories inside MTD submenu
> >> >   mtd: move NAND fiels into a raw/ subdirectory
> >> >   mtd: rename nand into rawnand in Kconfig prompt
> >> >   mtd: spinand: Add initial support for the MX35LF2GE4AB chip
> >> >   mtd: uclass: add probe function
> >> >   cmd: mtd: add 'mtd' command
> >> >   mips: dts: ocelot: describe SPI CS pins
> >> >   mips: dts: ocelot: add the SPI NAND node
> >> >
> >> > Peter Pan (2):
> >> >   mtd: nand: Add core infrastructure to support SPI NANDs
> >> >   mtd: spinand: Add initial support for Micron MT29F2G01ABAGD
> >>
> >> I have reviewed and commented most of the stuff that are in my
> >> control, except few spi-nand code, which I would like to give a try to
> >> other people in the list. Hope to seen the fine-tuning patches on next
> >> version.
> >
> > Thanks for taking the time to review all this; I'm on it.
> >
> > About spi-nand, reviews from other contributors are welcome, but it's
> > mostly a copy from Linux framework that I just applied to nand/next.
> > There have been 9 versions of it before that so I think it's pretty
> > well reviewed.
> 
> OK, that's fine then.

If everyone is agreeable I would like to see this in the v2018.09
release.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180706/8daf333b/attachment.sig>