[U-Boot] [PATCH] mtd: nand: spi: Add Gigadevice SPI NAND support

Stefan Roese
2018-10-04T08:44:47Z

Hi Miquel,

On 04.10.2018 10:41, Miquel Raynal wrote:
> Hi Stefan,
>
> Stefan Roese <sr@denx.de> wrote on Thu, 4 Oct 2018 10:03:54 +0200:
>
>> Hi Jagan,
>>
>> On 17.08.2018 11:10, Stefan Roese wrote:
>>> Hi Miquel,
>>>> On 17.08.2018 10:45, Miquel Raynal wrote:
>>>> Stefan Roese <sr@denx.de> wrote on Thu, 16 Aug 2018 18:05:08 +0200:
>>>>
>>>>> This patch adds support for Gigadevices SPI NAND device to the new SPI
>>>>> NAND infrastructure in U-Boot. Currently only the 128MiB GD5F1GQ4UC
>>>>> device is supported.
>>>>>
>>>>> Signed-off-by: Stefan Roese <sr@denx.de>
>>>>> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
>>>>> Cc: Boris Brezillon <boris.brezillon@bootlin.com>
>>>>> Cc: Jagan Teki <jagan@openedev.com>
>>>>> ---
>>>>> Miquel, I somehow always forgot to send this Gigadevice support
>>>>> patch in my last series versions. This is the chip I'm currently
>>>>> testing with. Either please fold it into your series, if you
>>>>> re-send it at a later time. Or we should add it on top. Of course
>>>>> only if nobody has any objections or change requests.
>>>>
>>>> No problem, as I don't wan't to resend another series of 30+ patches I
>>>> just asked Jagan/Tom if they could take them (up to patch 23 included),
>>>> so this one could be applied on top of it, alone.
>>>>
>>>> Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
>>>> Thanks.
>>
>> Could you please add to your upstream queue as well?
>
> Indeed, I completely forgot about this patch, I should have warned
> Jagan about it, sorry for forgetting.
>
> BTW, someone did raise a bug with a raw nand device and the mtd
> command, did you have a chance to test it with such device?

No, not yet, sorry. My current platforms only have SPI NOR and
SPI NAND.

> For now I
> just have a SPI-NAND and a parallel NOR to play with. I'll build
> another setup to test also parallel NANDs, but while on it...

Once I have such a platform on my desk (again), I'll definitely
will test it with the mtd command.

Thanks,
Stefan