Re: [EXT] Re: [PATCH v6 0/6] Add new series Micron SPI NAND devices

From: Boris Brezillon
Date: Wed Mar 11 2020 - 13:35:24 EST


On Wed, 11 Mar 2020 17:33:41 +0000
"Shivamurthy Shastri (sshivamurthy)" <sshivamurthy@xxxxxxxxxx> wrote:

> Hi Miquel,
>
> >
> > Hi Shiva,
> >
> > shiva.linuxworks@xxxxxxxxx wrote on Mon, 9 Mar 2020 12:52:24 +0100:
> >
> > > From: Shivamurthy Shastri <sshivamurthy@xxxxxxxxxx>
> > >
> > > This patchset is for the new series of Micron SPI NAND devices, and the
> > > following links are their datasheets.
> > >
> > > M78A:
> > > [1] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m78a_1gb_3v_nand_spi.pdf
> > > [2] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m78a_1gb_1_8v_nand_spi.pdf
> > >
> > > M79A:
> > > [3] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m79a_2gb_1_8v_nand_spi.pdf
> > > [4] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m79a_ddp_4gb_3v_nand_spi.pdf
> > >
> > > M70A:
> > > [5] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m70a_4gb_3v_nand_spi.pdf
> > > [6] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m70a_4gb_1_8v_nand_spi.pdf
> > > [7] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m70a_ddp_8gb_3v_nand_spi.pdf
> > > [8] https://www.micron.com/~/media/documents/products/data-
> > sheet/nand-flash/70-series/m70a_ddp_8gb_1_8v_nand_spi.pdf
> > >
> > > Changes since v5:
> > > -----------------
> > >
> > > 1. Rebased series to v5.6-rc1.
> >
> > I am very sorry but actually I had issues applying all your patches not
> > because they were not based on v5.6-rc1, but because since then I
> > applied a patch changing the detection that changed the content of a
> > lot of structures (including in Micron's patches).
> >
> > Can you please rebase again on top of the current nand/next? I am very
> > sorry for this extra work, this is my mistake.
> >
> > Head should be:
> >
> > a5d53ad26a8b ("mtd: rawnand: brcmnand: Add support for flash-edu
> > for dma transfers")
> >
> > And the culprit commit is:
> >
> > f1541773af49 ("mtd: spinand: rework detect procedure for different
> > READ_ID operation")
>
>
> I will rebase and send the patches.
> Meanwhile, there will be small code change because of the READ_ID patch.
>
> Do I need to drop Reviewed-by from Boris?

Nope, you can keep it.