On Mon, Sep 16, 2024 at 6:45 AM William Zhang <william.zh...@broadcom.com> wrote:
> Yes we have it working. And your patches works too after fixing two > Issues that I will comment on the specific patch. Yay! > You will need to > enable the NAND related configurations and the nand dts node. Aha maybe I simply missed the 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. I was planning to used the strap, which is ... what we should have been doing all the time. Oh well, no turning back time I guess. Maybe we can fix a lot of DTS:es to use the strap though? > What SoC do you use? This is on BCM6846. Thanks William! Linus