Hi Stefan, Breno,
On 01/01/2018 22:53, Stefan Agner wrote:
On 2017-11-28 00:07, Breno Lima wrote:
From: Breno Lima breno.lima@nxp.com
MX6UL contains features that MX6ULL doesn't support. Deselect CONFIG_MX6UL and select SYS_L2CACHE_OFF and ROM_UNIFIED_SECTIONS.
The motivation for doing this change is that MX6UL supports CAAM and MX6ULL does not.
While I agree with the idea of the change in general, it must be done more carefully! There are several places where we rely that CONFIG_MX6ULL selects CONFIG_MX6ULL, e.g. include/configs/mx6_common.h arch/arm/include/asm/arch-mx6/imx-regs.h arch/arm/include/asm/mach-imx/regs-lcdif.h
Ouch...
It is much more as the initial goal of the patch. This breaks MX6ULL.
The idea to add features instead of global SOC define is good, rather we suffer from previous development, when we bound bunches of features simply to the SOC type.
I see that this patch already got merged... So I guess we either fix all those places before the release, or we revert this patch...
Agree - I am waiting for Breno's answer, but if we have a generally agreement I tend to revert this until this change will be properly handled.
Best regards, Stefano
-- Stefan
Signed-off-by: Breno Lima breno.lima@nxp.com
arch/arm/mach-imx/mx6/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig index 4f8bb8f..279f088 100644 --- a/arch/arm/mach-imx/mx6/Kconfig +++ b/arch/arm/mach-imx/mx6/Kconfig @@ -66,8 +66,9 @@ config MX6UL_OPOS6UL select SUPPORT_SPL
config MX6ULL
- select SYS_L2CACHE_OFF
- select ROM_UNIFIED_SECTIONS bool
- select MX6UL
config MX6_DDRCAL bool "Include dynamic DDR calibration routines"