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

[Jagan Teki]

On Fri, Jul 6, 2018 at 5:36 PM, Miquel Raynal <miquel.raynal at bootlin.com> wrote:
> Hi Jagan,
>
> Jagan Teki <jagan at 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 at 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.