michaelh/zephyr
1
0
Fork 0
Code Releases Activity

modules: Kconfig.mcux: Remove implicit selection of HAS_CMSIS_CORE

Browse source 
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 <laurentiu.mihalcea@nxp.com>
This commit is contained in:
Laurentiu Mihalcea 2023-10-05 10:51:02 +03:00 • committed by Johan Hedberg
commit 3563347b10
1 changed files with 0 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
modules/Kconfig.mcux
View file

@ -5,7 +5,6 @@
config HAS_MCUX
bool
select HAS_CMSIS_CORE if !SOC_FAMILY_NXP_ADSP
depends on SOC_FAMILY_KINETIS || SOC_FAMILY_IMX || SOC_FAMILY_LPC || \
SOC_FAMILY_NXP_ADSP || SOC_FAMILY_NXP_S32