On Mon, Aug 15, 2022 at 7:15 PM Florian Fainelli <f.fainelli@xxxxxxxxx> wrote: > On 7/13/22 15:05, Florian Fainelli wrote: > > On 7/13/22 02:17, Geert Uytterhoeven wrote: > > > > The itch that I wanted to calm was that ARCH_BRCMSTB was after the other > > Broadcom platforms separated by ARCH_BERLIN. if you prefer a pair of > > KConfig comments to delineate them and flatten the platform selection, > > that works for me, too. > > There are 2 Marvell-based platforms (Berlin and mvebu) as well as two > NXP-based platforms (MXC and S32), would it be better to also group them > under an ARCH_MARVELL and ARCH_NXP menuconfig symbol the same way this > patch does it for Broadcom-based SoCs? Berlin is now Synaptics after Marvell sold them the business unit, so that one should probably stay separate. For NXP, there is also Layerscape. I agree it would be nice to group the three NXP together. Arnd