Skip to content

Commit 3563347

Browse files
LaurentiuM1234jhedberg
authored andcommitted
modules: Kconfig.mcux: Remove implicit selection of HAS_CMSIS_CORE
CONFIG_HAS_MCUX is also selected by ARM64 and XTENSA-based cores which don't have any CMSIS files. As such, it's wrong to implicitly select CONFIG_HAS_CMSIS_CORE when CONFIG_HAS_MCUX is selected. Since the ARM32-based cores implicitly select CONFIG_HAS_CMSIS_CORE there's no point in CONFIG_HAS_MCUX also selecting it too. The old approach caused config-related warnings in external projects such as SOF after the introduction of Commit '8d5ed53' ("modules: remove uncessary source for external cmsis kconfig"). The warnings were in the form of: "HAS_CMSIS_CORE (defined at modules/cmsis/Kconfig:7) has direct dependencies 0 with value n ..." Signed-off-by: Laurentiu Mihalcea <[email protected]>
1 parent f4a592f commit 3563347

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

modules/Kconfig.mcux

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
config HAS_MCUX
77
bool
8-
select HAS_CMSIS_CORE if !SOC_FAMILY_NXP_ADSP
98
depends on SOC_FAMILY_KINETIS || SOC_FAMILY_IMX || SOC_FAMILY_LPC || \
109
SOC_FAMILY_NXP_ADSP || SOC_FAMILY_NXP_S32
1110

0 commit comments

Comments
 (0)