From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Date: Sat, 4 Aug 2018 10:34:55 +0200 Subject: [U-Boot] [PATCH v6 00/27] SPI-NAND support In-Reply-To: <20180804092305.6fdc0ed0@xps13> References: <20180801081848.19398-1-miquel.raynal@bootlin.com> <20180803102711.35fabac2@xps13> <20180804092305.6fdc0ed0@xps13> Message-ID: <20180804103455.0c2e906c@bbrezillon> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de On Sat, 4 Aug 2018 09:23:05 +0200 Miquel Raynal wrote: > Hi Jagan, >=20 > Jagan Teki wrote on Fri, 3 Aug 2018 > 17:38:33 +0530: >=20 > > On Fri, Aug 3, 2018 at 3:20 PM, Jagan Teki = wrote: =20 > > > On Fri, Aug 3, 2018 at 1:57 PM, Miquel Raynal wrote: =20 > > >> Hi Jagan, Tom, > > >> > > >> Miquel Raynal wrote on Wed, 1 Aug 2018 > > >> 10:18:21 +0200: > > >> =20 > > >>> 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. T= hen, > > >>> he contributed a generic SPI-NAND driver, making use of this NAND c= ore, > > >>> 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 h= aving > > >>> 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 l= ayer. > > >>> - 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. > > >>> - Support for spi-nand devices in mtdparts. > > >>> > > >>> To test your SPI-NAND device with U-Boot simply follow these lines: > > >>> =20 > > >>> > setenv mtdparts mtdparts=3Dspi-nand0:1m(foo),-(bar) > > >>> > setenv mtdids spi-nand0=3Dspi-nand0 > > >>> > mtdparts # show the spi-nand device partitions > > >>> > ubi part bar # create a static UBI volume in the bar part= ition =20 > > >>> > > >>> Thanks, > > >>> Miqu=C3=A8l > > >>> > > >>> Changes since v5: > > >>> ----------------- > > >>> * Included Boris fixup about the build issues. > > >>> * Added Rb/Ab tags from Jagan on patchs 20/21. =20 > > >> > > >> I can't see a pull request flow on U-Boot ML, I suppose you use a > > >> different mean for that purpose. > > >> > > >> Jagan, is this version OK? Is it part of your PR? =20 > > > > > > Travis is going on [1], will send PR once all fine. > > > > > > [1] https://travis-ci.org/openedev/u-boot-amarula/builds/411596788 = =20 > >=20 > > There are some build issues, not quite sure whether it relates. please > > look into it. =20 >=20 > Thanks for the reports. >=20 > >=20 > > [2] https://travis-ci.org/openedev/u-boot-amarula/jobs/411596814 > > [3] https://travis-ci.org/openedev/u-boot-amarula/jobs/411596815 =20 >=20 > Boris supposedly fixed all the build issues related to my changes (I > don't think the "missing interrupt parent" in a DTS file is related), > please pull his branch [1]. >=20 > The fixes are under the form of fixups if you wanna check them. > Otherwise you can just 'rebase -i --autosquash' to automatically squash > them with the commit introducing the regression. One more thing, if you want to make things bisectable patch 6 ("mtd: fix build issue with includes") and it's fixup should be squashed in patch 5 ("mtd: add get/set of_node/flash_node helpers"), because the build failure you're fixing in patch 6 is introduced by patch 5. >=20 > [1] https://travis-ci.org/openedev/u-boot-amarula/jobs/411596814 You mean [1]https://github.com/bbrezillon/u-boot/commits/wip-spi