RE: [PATCH 0/7] mtd: nand: brcmnand: Backported fixes from Linux

[William Zhang <william.zhang@broadcom.com>]

[-- Attachment #1: Type: text/plain, Size: 1749 bytes --]

> -----Original Message-----
> From: Linus Walleij <linus.walleij@linaro.org>
> Sent: Thursday, September 12, 2024 1:16 AM
> To: William Zhang <william.zhang@broadcom.com>
> Cc: u-boot@lists.denx.de; Dario Binacchi
> <dario.binacchi@amarulasolutions.com>; Michael Trimarchi
> <michael@amarulasolutions.com>; Anand Gore
> <anand.gore@broadcom.com>; Kursad Oney
> <kursad.oney@broadcom.com>; Philippe Reynes
> <philippe.reynes@softathome.com>; Florian Fainelli
> <florian.fainelli@broadcom.com>; Miquel Raynal
> <miquel.raynal@bootlin.com>; Kamal Dasu <kamal.dasu@broadcom.com>;
> David Regan <dregan@broadcom.com>
> Subject: Re: [PATCH 0/7] mtd: nand: brcmnand: Backported fixes from
> Linux
>
> On Thu, Sep 12, 2024 at 3:32 AM William Zhang
> <william.zhang@broadcom.com> wrote:
>
> > Thanks for merging the patches to u-boot.   We actually had something
> > similar
> > here locally but just didn't get the chance to upstream yet.
> >
> > We will review and compare your patches to ours and get back to you.
>
> Ah, excellent, I guess you guys know the stuff better than me
> so I prefer if your patches are used.
>
> Do you even have U-Boot working on BCMBCA? I didn't get the
> NAND to detect properly yet despite all the backported patches. :/
>
Yes we have it working.  And your patches works too after fixing two
Issues that I will comment on the specific patch.  You will need to
enable the  NAND related configurations and the nand dts node.

You can use existing dts node to set the specific ecc level, oob size and
ecc step size.  Or use the new automatically detection from the strap
property brcm,nand-ecc-use-strap that I upstreamed to linux.

What SoC do you use?

> Yours,
> Linus Walleij

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4212 bytes --]