public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 00/21] SPI-NAND support
Date: Fri, 13 Jul 2018 14:26:29 +0200	[thread overview]
Message-ID: <20180713142629.38a43996@xps13> (raw)
In-Reply-To: <20180712132506.26359-1-miquel.raynal@bootlin.com>

Hi Miquel,

Miquel Raynal <miquel.raynal@bootlin.com> wrote on Thu, 12 Jul 2018
15:24:45 +0200:

> 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:
> - A few patches from Linux to resynchronize some areas of the MTD layer.
> - 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.
> 
> Any comments on the code, the organization and the respect of U-Boot
> driver model will be welcome.
> 
> Thanks,
> Miquèl
> 

Since this version, I tried to use UBI on top of this work and it
failed because of the following:
- mtdpart driver was not actually compiled because a Kconfig symbol was
  missing
- mtdpart.c could not compile anyway because of an error when
  backporting the patch from Linux (did not see it because of the above
  mistake).
- SPI-NAND devices were not supported in mtdparts.
- The mtd command did not create MTD devices for each desired partition.

This is now fixed/done in the next version that I am about to send.

One can try it with:

> setenv mtdparts mtdparts=spi-nand0:1m(foo),-(bar)
> setenv mtdids spi-nand0=spi-nand0
> ubi part bar

Thanks,
Miquèl

      parent reply	other threads:[~2018-07-13 12:26 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-12 13:24 [U-Boot] [PATCH v3 00/21] SPI-NAND support Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 01/21] mtd: Fallback to ->_read/write_oob() when ->_read/write() is missing Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 02/21] mtd: Uninline mtd_write_oob and move it to mtdcore.c Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 03/21] mtd: Add sanity checks in mtd_write/read_oob() Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 04/21] mtd: Fallback to ->_read/write() when ->_read/write_oob() is missing Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 05/21] mtd: add get/set of_node/flash_node helpers Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 06/21] mtd: fix build issue with includes Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 07/21] mtd: move definitions to enlarge their range Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 08/21] mtd: move all flash categories inside MTD submenu Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 09/21] mtd: move NAND files into a raw/ subdirectory Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 10/21] mtd: rename nand into rawnand in Kconfig prompt Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 11/21] mtd: nand: Add core infrastructure to deal with NAND devices Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 12/21] mtd: nand: Pass mode information to nand_page_io_req Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 13/21] spi: Extend the core to ease integration of SPI memory controllers Miquel Raynal
2018-07-12 13:24 ` [U-Boot] [PATCH v3 14/21] mtd: nand: Add core infrastructure to support SPI NANDs Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 15/21] mtd: spinand: Add initial support for Micron MT29F2G01ABAGD Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 16/21] mtd: spinand: Add initial support for Winbond W25M02GV Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 17/21] mtd: spinand: Add initial support for the MX35LF1GE4AB chip Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 18/21] mtd: spinand: Add initial support for the MX35LF2GE4AB chip Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 19/21] mtd: uclass: add probe function Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 20/21] cmd: mtd: add 'mtd' command Miquel Raynal
2018-07-12 13:25 ` [U-Boot] [PATCH v3 21/21] dt-bindings: Add bindings for SPI NAND devices Miquel Raynal
2018-07-13 12:26 ` Miquel Raynal [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180713142629.38a43996@xps13 \
    --to=miquel.raynal@bootlin.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox