mbox series

[v4,0/2] mtd: spinand: toshiba: Support for new Kioxia Serial NAND

Message ID cover.1583834323.git.ytc-mb-yfuruyama7@kioxia.com
Headers show
Series mtd: spinand: toshiba: Support for new Kioxia Serial NAND | expand

Message

Yoshio Furuyama March 11, 2020, 1:47 a.m. UTC
First patch is to rename function name becase of add new device.
Second patch is to supprot for new device.

Yoshio Furuyama (2):
  mtd: spinand: toshiba: Rename function name to change suffix and
    prefix (8Gbit)
  mtd: spinand: toshiba: Support for new Kioxia Serial NAND

 drivers/mtd/nand/spi/toshiba.c | 173 +++++++++++++++++++++++++++++++----------
 1 file changed, 130 insertions(+), 43 deletions(-)

Comments

Miquel Raynal March 11, 2020, 3:50 p.m. UTC | #1
Hi Yoshio,

Yoshio Furuyama <ytc-mb-yfuruyama7@kioxia.com> wrote on Wed, 11 Mar
2020 10:47:04 +0900:

> First patch is to rename function name becase of add new device.
> Second patch is to supprot for new device.
> 
> Yoshio Furuyama (2):
>   mtd: spinand: toshiba: Rename function name to change suffix and
>     prefix (8Gbit)
>   mtd: spinand: toshiba: Support for new Kioxia Serial NAND
> 
>  drivers/mtd/nand/spi/toshiba.c | 173 +++++++++++++++++++++++++++++++----------
>  1 file changed, 130 insertions(+), 43 deletions(-)
> 

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 Toshiba's driver).

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")

Thanks,
Miquèl
Yoshio Furuyama March 18, 2020, 5:40 a.m. UTC | #2
On 2020/03/12 0:50, Miquel Raynal wrote:
> Hi Yoshio,
>
> Yoshio Furuyama <ytc-mb-yfuruyama7@kioxia.com> wrote on Wed, 11 Mar
> 2020 10:47:04 +0900:
>
>> First patch is to rename function name becase of add new device.
>> Second patch is to supprot for new device.
>>
>> Yoshio Furuyama (2):
>>    mtd: spinand: toshiba: Rename function name to change suffix and
>>      prefix (8Gbit)
>>    mtd: spinand: toshiba: Support for new Kioxia Serial NAND
>>
>>   drivers/mtd/nand/spi/toshiba.c | 173 +++++++++++++++++++++++++++++++----------
>>   1 file changed, 130 insertions(+), 43 deletions(-)
>>
> 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 Toshiba's driver).
>
> Can you please rebase again on top of the current nand/next? I am very
> sorry for this extra work, this is my mistake.

Thanks comment.         I will revise rev (V5) next week.

BR

> 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")
>
> Thanks,
> Miquèl
>
>
>
Miquel Raynal March 18, 2020, 8:35 a.m. UTC | #3
Hi Yoshio,

Yoshio Furuyama <ytc-mb-yfuruyama7@kioxia.com> wrote on Wed, 18 Mar
2020 14:40:47 +0900:

> On 2020/03/12 0:50, Miquel Raynal wrote:
> > Hi Yoshio,
> >
> > Yoshio Furuyama <ytc-mb-yfuruyama7@kioxia.com> wrote on Wed, 11 Mar
> > 2020 10:47:04 +0900:
> >  
> >> First patch is to rename function name becase of add new device.
> >> Second patch is to supprot for new device.
> >>
> >> Yoshio Furuyama (2):
> >>    mtd: spinand: toshiba: Rename function name to change suffix and
> >>      prefix (8Gbit)
> >>    mtd: spinand: toshiba: Support for new Kioxia Serial NAND
> >>
> >>   drivers/mtd/nand/spi/toshiba.c | 173 +++++++++++++++++++++++++++++++----------
> >>   1 file changed, 130 insertions(+), 43 deletions(-)
> >>  
> > 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 Toshiba's driver).
> >
> > Can you please rebase again on top of the current nand/next? I am very
> > sorry for this extra work, this is my mistake.  
> 
> Thanks comment.         I will revise rev (V5) next week.

We are at -rc6 already, it would be good if you could send it before
the end of the week.

Thanks,
Miquèl