On 1/3/2022 8:56 AM, Miquel Raynal wrote:
Hi Florian,
f.fainelli@xxxxxxxxx wrote on Wed, 22 Dec 2021 16:22:20 -0800:
In order to initialize a given chip select object for use by the
brcmnand driver, move all of the Device Tree specific routines outside
of brcmnand_init_cs() in order to make it usable in a platform data
configuration which will be necessary for supporting BCMA chips.
TBH I'm note a big fan of the idea. I'm not sure going back to
supporting platform data this way really is a good idea... There are so
much things that are well described with DT that we now rely upon that
I am not entirely convinced by these changes :-/ The move is generally
in the other direction: getting rid of the legacy platform data.
In the cover letter there is an explanation as to why we need to
introduce platform data/device support here: the platforms on which this
NAND controller shim is used do not have Device Tree support, and won't
have it in the future either. They are old platforms (first SoC
supported by bcm47xx is maybe 15 years old now) but they are still in
active and wide use by the OpenWrt, dd-wrt communities.
--
Florian
