Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig

[Philip Oberfichtner <pro@denx.de>]

Hi,

following the whole discussion I figured using 'select SYS_l2_PL310 if
!SYS_L2CACHE_OFF' is the preferred solution.

Now the thing is, if I'd put this line under the ARCH_XXX Kconfig
entries, I would change behavior for many boards. Take, for example,
ARCH_MVEBU:

grep -lr ARCH_MVEBU configs | ./tools/moveconfig.py -d-
CONFIG_SYS_L2_PL310

Tells me that there are 30 ARCH_MVEBU defconfigs, of which 19 use
CONFIG_SYS_L2_PL310 and 11 don't. Those 11 boards not using it also do
not define CONFIG_SYS_L2CACHE_OFF. So using a 'select SYS_L2_PL310'
under ARCH_MVEBU would change behavior for those 11 boards.

There is a similar picture for other architectures, like SOCFPGA or
OMAP2. The only place where selecting based on CONFIG_ARCH does not
change behavior is ARCH_MX6, when excluding CONFIG_MX6UL(L).

If I didn't miss something here, there's no easy way out. Maybe to
define SYS_L2CACHE_OFF for respective boards would be an option, but I
don't know if there would be side effects.

Any other suggestions?

Best regards,
Philip