Build: Simplify GNUARM toolchain
- Removed obsolete version checks
- Extracted MCPU feature control to a separate file for reuse
- Stop using -march for MCU feature selection
- Added -mcpu='' options explicitly to both compile and link
- Removed C++ support from the secure-side toolchain
- Consolidated binary file generation into a single step
- Propagated changes to NS toolchain
- Added stack sealing to Musca-B1
- Remove entry point from toolchain being defined in the linker script
Signed-off-by: Anton Komlev <anton.komlev@arm.com>
Change-Id: Iadfff03ab327e6d75ec898cff6c04c4f473f15f5
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 2560dd1..ecbae1d 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,7 +1,5 @@
#-------------------------------------------------------------------------------
# SPDX-FileCopyrightText: Copyright The TrustedFirmware-M Contributors
-# Copyright (c) 2022 Cypress Semiconductor Corporation (an Infineon company)
-# or an affiliate of Cypress Semiconductor Corporation. All rights reserved.
#
# SPDX-License-Identifier: BSD-3-Clause
#
@@ -37,9 +35,6 @@
include(${TFM_TOOLCHAIN_FILE})
-set(CMAKE_C_COMPILER_FORCED true)
-set(CMAKE_CXX_COMPILER_FORCED true)
-
project("Trusted Firmware M" VERSION ${TFM_VERSION} LANGUAGES C CXX ASM)
add_subdirectory(lib/backtrace)
@@ -77,4 +72,4 @@
################################################################################
-include(cmake/install.cmake)
+include(install)
diff --git a/bl1/bl1_2/CMakeLists.txt b/bl1/bl1_2/CMakeLists.txt
index 23fe370..cb10f91 100644
--- a/bl1/bl1_2/CMakeLists.txt
+++ b/bl1/bl1_2/CMakeLists.txt
@@ -112,7 +112,6 @@
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/scripts/create_bl2_img.py
DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/config/bl2_image_config.pickle
DEPENDS bl2_bin
- DEPENDS ${CMAKE_BINARY_DIR}/bin/bl2.bin
COMMAND ${CMAKE_COMMAND} -E env PYTHONPATH=${CMAKE_SOURCE_DIR}/tools/modules
${Python3_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/scripts/create_bl2_img.py
--bl2_image_config ${CMAKE_CURRENT_BINARY_DIR}/config/bl2_image_config.pickle
diff --git a/cmake/install.cmake b/cmake/install.cmake
index dc60e2b..d59b579 100644
--- a/cmake/install.cmake
+++ b/cmake/install.cmake
@@ -1,7 +1,5 @@
#-------------------------------------------------------------------------------
-# Copyright (c) 2020-2024, Arm Limited. All rights reserved.
-# Copyright (c) 2022-2023 Cypress Semiconductor Corporation (an Infineon company)
-# or an affiliate of Cypress Semiconductor Corporation. All rights reserved.
+# SPDX-FileCopyrightText: Copyright The TrustedFirmware-M Contributors
#
# SPDX-License-Identifier: BSD-3-Clause
#
@@ -307,5 +305,7 @@
${INSTALL_CMAKE_DIR}/spe_config.cmake @ONLY)
# Toolchain utils
-install(FILES cmake/set_extensions.cmake
+install(FILES
+ cmake/set_extensions.cmake
+ cmake/mcpu_features.cmake
DESTINATION ${INSTALL_CMAKE_DIR})
diff --git a/cmake/mcpu_features.cmake b/cmake/mcpu_features.cmake
new file mode 100644
index 0000000..3da9735
--- /dev/null
+++ b/cmake/mcpu_features.cmake
@@ -0,0 +1,49 @@
+#-------------------------------------------------------------------------------
+# SPDX-FileCopyrightText: Copyright The TrustedFirmware-M Contributors
+#
+# SPDX-License-Identifier: BSD-3-Clause
+#
+#-------------------------------------------------------------------------------
+
+# Check if the MCPU supports security extension (CMSE)
+set(CMSE_SUPPORTED_MCUS "cortex-m23" "cortex-m33" "cortex-m35p" "cortex-m55" "cortex-m52" "cortex-m85")
+list(FIND CMSE_SUPPORTED_MCUS ${TFM_SYSTEM_PROCESSOR} ITEM_INDEX)
+if(NOT ITEM_INDEX EQUAL -1)
+ set(COMPILER_CMSE_FLAG "-mcmse")
+endif()
+
+# ===================== Set toolchain CPU and Arch =============================
+set(TFM_SYSTEM_PROCESSOR_FEATURED ${TFM_SYSTEM_PROCESSOR})
+
+set(FEATURE_SUPPORTED_MCUS "cortex-m33" "cortex-m35p" "cortex-m55" "cortex-m52" "cortex-m85")
+list(FIND FEATURE_SUPPORTED_MCUS ${TFM_SYSTEM_PROCESSOR} ITEM_INDEX)
+if(NOT ITEM_INDEX EQUAL -1)
+ # Hardware DSP is currently not supported in TF-M
+ string(APPEND TFM_SYSTEM_PROCESSOR_FEATURED "+nodsp")
+
+ if(NOT CONFIG_TFM_ENABLE_FP)
+ string(APPEND TFM_SYSTEM_PROCESSOR_FEATURED "+nofp")
+ endif()
+
+ if(TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main")
+ # For GNU Arm Embedded Toolchain doesn't emit __ARM_ARCH_8_1M_MAIN__, adding this macro manually.
+ add_compile_definitions(__ARM_ARCH_8_1M_MAIN__=1)
+
+ if(NOT CONFIG_TFM_ENABLE_MVE)
+ string(APPEND TFM_SYSTEM_PROCESSOR_FEATURED "+nomve")
+ endif()
+
+ if(NOT CONFIG_TFM_ENABLE_MVE_FP)
+ string(APPEND TFM_SYSTEM_PROCESSOR_FEATURED "+nomve.fp")
+ endif()
+ endif()
+else()
+ set(NOFP_SUPPORTED_MCUS "cortex-m7" "cortex-m4")
+ list(FIND NOFP_SUPPORTED_MCUS ${TFM_SYSTEM_PROCESSOR} ITEM_INDEX)
+ if(NOT ITEM_INDEX EQUAL -1)
+ if(NOT CONFIG_TFM_ENABLE_FP)
+ string(APPEND TFM_SYSTEM_PROCESSOR_FEATURED "+nofp")
+ endif()
+ endif()
+endif()
+add_compile_options(-mcpu=${TFM_SYSTEM_PROCESSOR_FEATURED})
diff --git a/cmake/spe-CMakeLists.cmake b/cmake/spe-CMakeLists.cmake
index 6cbf30b..412f479 100644
--- a/cmake/spe-CMakeLists.cmake
+++ b/cmake/spe-CMakeLists.cmake
@@ -168,7 +168,6 @@
)
add_custom_command(OUTPUT ${CMAKE_BINARY_DIR}/bin/${NS_TARGET_NAME}_signed.bin
DEPENDS ${NS_TARGET_NAME}_bin
- DEPENDS $<TARGET_FILE_DIR:${NS_TARGET_NAME}>/${NS_TARGET_NAME}.bin
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/image_signing/layout_files/signing_layout_ns.o
WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/image_signing/scripts
@@ -215,7 +214,7 @@
)
add_custom_command(OUTPUT ${CMAKE_BINARY_DIR}/bin/tfm_s_ns.bin
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/bin/tfm_s.bin
- DEPENDS ${NS_TARGET_NAME}_bin $<TARGET_FILE_DIR:${NS_TARGET_NAME}>/${NS_TARGET_NAME}.bin
+ DEPENDS ${NS_TARGET_NAME}_bin
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/image_signing/layout_files/signing_layout_s_ns.o
WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/image_signing/scripts
diff --git a/platform/ext/target/arm/musca_b1/Device/Source/gcc/musca_bl2.ld b/platform/ext/target/arm/musca_b1/Device/Source/gcc/musca_bl2.ld
index c0878d7..9e02964 100644
--- a/platform/ext/target/arm/musca_b1/Device/Source/gcc/musca_bl2.ld
+++ b/platform/ext/target/arm/musca_b1/Device/Source/gcc/musca_bl2.ld
@@ -191,13 +191,19 @@
__bss_end__ = .;
} > RAM
- .msp_stack : ALIGN(32)
+ .msp_stack (NOLOAD) : ALIGN(32)
{
. += __msp_stack_size__;
} > RAM
Image$$ARM_LIB_STACK$$ZI$$Base = ADDR(.msp_stack);
Image$$ARM_LIB_STACK$$ZI$$Limit = ADDR(.msp_stack) + SIZEOF(.msp_stack);
+ .msp_stack_seal_res :
+ {
+ . += 0x8;
+ } > RAM
+ __StackSeal = ADDR(.msp_stack_seal_res);
+
.heap : ALIGN(8)
{
. = ALIGN(8);
diff --git a/platform/ext/target/arm/rse/common/ns/common.cmake b/platform/ext/target/arm/rse/common/ns/common.cmake
index 15370c5..4fa63aa 100644
--- a/platform/ext/target/arm/rse/common/ns/common.cmake
+++ b/platform/ext/target/arm/rse/common/ns/common.cmake
@@ -76,7 +76,6 @@
add_custom_command(OUTPUT ${NS_TARGET_NAME}_sic_tables.bin
OUTPUT ${NS_TARGET_NAME}_encrypted.bin
- DEPENDS $<TARGET_FILE_DIR:${NS_TARGET_NAME}>/${NS_TARGET_NAME}.bin
DEPENDS ${NS_TARGET_NAME}_bin
COMMAND ${Python3_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/../image_signing/scripts/create_xip_tables.py
--input_image $<TARGET_FILE_DIR:${NS_TARGET_NAME}>/${NS_TARGET_NAME}.bin
diff --git a/platform/ext/target/cypress/psoc64/ns/cpuarch_ns.cmake b/platform/ext/target/cypress/psoc64/ns/cpuarch_ns.cmake
index ccdfad2..5bffc8d 100644
--- a/platform/ext/target/cypress/psoc64/ns/cpuarch_ns.cmake
+++ b/platform/ext/target/cypress/psoc64/ns/cpuarch_ns.cmake
@@ -12,4 +12,4 @@
# Set architecture and CPU. Cypress PSoC64 runs the NS code on its Cortex-M4
set(TFM_SYSTEM_PROCESSOR cortex-m4)
-set(TFM_SYSTEM_ARCHITECTURE armv7-m)
+set(TFM_SYSTEM_ARCHITECTURE armv7e-m)
diff --git a/platform/ns/toolchain_ns_GNUARM.cmake b/platform/ns/toolchain_ns_GNUARM.cmake
index 64ee5b3..b87f43f 100644
--- a/platform/ns/toolchain_ns_GNUARM.cmake
+++ b/platform/ns/toolchain_ns_GNUARM.cmake
@@ -7,28 +7,19 @@
set(CMAKE_SYSTEM_NAME Generic)
-set(CMAKE_C_COMPILER_FORCED TRUE)
-set(CMAKE_CXX_COMPILER_FORCED TRUE)
-
if(NOT DEFINED CROSS_COMPILE)
- set(CROSS_COMPILE arm-none-eabi CACHE STRING "Cross-compiler prefix")
+ set(CROSS_COMPILE "arm-none-eabi" CACHE STRING "Cross-compiler prefix")
endif()
-find_program(CMAKE_C_COMPILER ${CROSS_COMPILE}-gcc)
-find_program(CMAKE_CXX_COMPILER ${CROSS_COMPILE}-g++)
-
-if(CMAKE_C_COMPILER STREQUAL "CMAKE_C_COMPILER-NOTFOUND")
- message(FATAL_ERROR "Could not find compiler: '${CROSS_COMPILE}-gcc'")
-endif()
-
-if(CMAKE_CXX_COMPILER STREQUAL "CMAKE_CXX_COMPILER-NOTFOUND")
- message(FATAL_ERROR "Could not find compiler: '${CROSS_COMPILE}-g++'")
-endif()
+set(CMAKE_C_COMPILER ${CROSS_COMPILE}-gcc)
+set(CMAKE_C_COMPILER_FORCED TRUE)
+set(CMAKE_C_STANDARD 99)
set(CMAKE_ASM_COMPILER ${CMAKE_C_COMPILER})
-# Set compiler ID explicitly as it's not detected at this moment
-set(CMAKE_C_COMPILER_ID GNU)
+set(CMAKE_CXX_COMPILER ${CROSS_COMPILE}-g++)
+set(CMAKE_CXX_COMPILER_FORCED TRUE)
+set(CMAKE_CXX_STANDARD 11)
# This variable name is a bit of a misnomer. The file it is set to is included
# at a particular step in the compiler initialisation. It is used here to
@@ -37,91 +28,24 @@
set(CMAKE_USER_MAKE_RULES_OVERRIDE ${CMAKE_CURRENT_LIST_DIR}/set_extensions.cmake)
# CMAKE_C_COMPILER_VERSION is not guaranteed to be defined.
-EXECUTE_PROCESS( COMMAND ${CMAKE_C_COMPILER} -dumpversion OUTPUT_VARIABLE GCC_VERSION )
+EXECUTE_PROCESS( COMMAND ${CMAKE_C_COMPILER} -dumpversion OUTPUT_VARIABLE CMAKE_C_COMPILER_VERSION)
-# ===================== Set toolchain CPU and Arch =============================
+if (${CMAKE_C_COMPILER_VERSION} VERSION_LESS 10.3.1)
+ message(FATAL_ERROR "Please use GNU Arm toolchain version 10.3.1 or later")
+endif()
-if (DEFINED TFM_SYSTEM_PROCESSOR)
- if(TFM_SYSTEM_PROCESSOR MATCHES "cortex-m85" AND GCC_VERSION VERSION_LESS "13.0.0")
- # GNUARM until version 13 does not support the -mcpu=cortex-m85 flag
- message(WARNING "Cortex-m85 is only supported from GCC13. "
- "Falling back to -march usage for earlier versions.")
- else()
- set(CMAKE_SYSTEM_PROCESSOR ${TFM_SYSTEM_PROCESSOR})
-
- if (DEFINED TFM_SYSTEM_DSP)
- if (NOT TFM_SYSTEM_DSP)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nodsp")
- endif()
- endif()
- # GCC specifies that '+nofp' is available on following M-profile cpus: 'cortex-m4',
- # 'cortex-m7', 'cortex-m33', 'cortex-m35p', 'cortex-m55' and 'cortex-m85'.
- # Build fails if other M-profile cpu, such as 'cortex-m23', is added with '+nofp'.
- # Explicitly list those cpu to align with GCC description.
- if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- if(NOT CONFIG_TFM_ENABLE_FP AND
- (TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m4"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m7"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m33"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m35p"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m55"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m85"))
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nofp")
- endif()
- endif()
-
- if(TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main")
- if(NOT CONFIG_TFM_ENABLE_MVE)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nomve")
- endif()
- if(NOT CONFIG_TFM_ENABLE_MVE_FP)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nomve.fp")
- endif()
- endif()
+function(min_toolchain_version mcpu gnu-version)
+ if (${TFM_SYSTEM_PROCESSOR} MATCHES ${mcpu} AND
+ ${CMAKE_C_COMPILER_VERSION} VERSION_LESS ${gnu-version})
+ message(FATAL_ERROR "-mcpu=${mcpu} is supported in GNU Arm version ${gnu-version} and later.\n"
+ "Please upgrade your toolchain to a supported version from: "
+ "https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads")
endif()
+endfunction()
-endif()
-
-# CMAKE_SYSTEM_ARCH variable is not a built-in CMAKE variable. It is used to
-# set the compile and link flags when TFM_SYSTEM_PROCESSOR is not specified.
-# The variable name is choosen to align with the ARMCLANG toolchain file.
-set(CMAKE_SYSTEM_ARCH ${TFM_SYSTEM_ARCHITECTURE})
-
-if(TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main")
- if(CONFIG_TFM_ENABLE_MVE)
- string(APPEND CMAKE_SYSTEM_ARCH "+mve")
- endif()
- if(CONFIG_TFM_ENABLE_MVE_FP)
- string(APPEND CMAKE_SYSTEM_ARCH "+mve.fp")
- endif()
-endif()
-
-if (DEFINED TFM_SYSTEM_DSP)
- # +nodsp modifier is only supported from GCC version 8.
- if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- # armv8.1-m.main arch does not have +nodsp option
- if ((NOT TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main") AND
- NOT TFM_SYSTEM_DSP)
- string(APPEND CMAKE_SYSTEM_ARCH "+nodsp")
- endif()
- endif()
-endif()
-
-if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- if(CONFIG_TFM_ENABLE_FP)
- string(APPEND CMAKE_SYSTEM_ARCH "+fp")
- endif()
-endif()
-
-if (GCC_VERSION VERSION_LESS 7.3.1)
- message(FATAL_ERROR "Please use newer GNU Arm compiler version starting from 7.3.1.")
-endif()
-
-if (GCC_VERSION VERSION_EQUAL 10.2.1)
- message(FATAL_ERROR "GNU Arm compiler version 10-2020-q4-major has an issue in CMSE support."
- " Select other GNU Arm compiler versions instead."
- " See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99157 for the issue detail.")
-endif()
+# the lowest supported GCC version for a specific cpu
+min_toolchain_version("cortex-m85" "13.0.0")
+min_toolchain_version("cortex-m52" "14.2.0")
# GNU Arm compiler version greater equal than *11.3.Rel1*
# has a linker issue that required system calls are missing,
@@ -131,56 +55,28 @@
# READONLY linker script attribute is not supported in older
# GNU Arm compilers. For these version the preprocessor will
# remove the READONLY string from the linker scripts.
-if (GCC_VERSION VERSION_GREATER_EQUAL 11.3.1)
+if (CMAKE_C_COMPILER_VERSION VERSION_GREATER_EQUAL 11.3.1)
set(CONFIG_GNU_SYSCALL_STUB_ENABLED TRUE)
set(CONFIG_GNU_LINKER_READONLY_ATTRIBUTE TRUE)
endif()
-if (CMAKE_SYSTEM_PROCESSOR)
- set(CMAKE_C_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_CXX_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_ASM_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_C_LINK_FLAGS "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_ASM_LINK_FLAGS "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
-else()
- set(CMAKE_C_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_CXX_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_ASM_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_C_LINK_FLAGS "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_ASM_LINK_FLAGS "-march=${CMAKE_SYSTEM_ARCH}")
-endif()
-
-set(CMAKE_C_FLAGS ${CMAKE_C_FLAGS_INIT})
-set(CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS_INIT})
-set(CMAKE_ASM_FLAGS ${CMAKE_ASM_FLAGS_INIT})
-
-set(COMPILER_CP_FLAG "-mfloat-abi=soft")
-set(LINKER_CP_OPTION "-mfloat-abi=soft")
-
if(CONFIG_TFM_FLOAT_ABI STREQUAL "hard")
- set(COMPILER_CP_FLAG "-mfloat-abi=hard")
- set(LINKER_CP_OPTION "-mfloat-abi=hard")
+ add_compile_options(-mfloat-abi=hard)
+ add_link_options(-mfloat-abi=hard)
if(CONFIG_TFM_ENABLE_FP OR CONFIG_TFM_ENABLE_MVE_FP)
- string(APPEND COMPILER_CP_FLAG " " "-mfpu=${CONFIG_TFM_FP_ARCH}")
- string(APPEND LINKER_CP_OPTION " " "-mfpu=${CONFIG_TFM_FP_ARCH}")
+ add_compile_options(-mfpu=${CONFIG_TFM_FP_ARCH})
+ add_link_options(-mfpu=${CONFIG_TFM_FP_ARCH})
endif()
+else()
+ add_compile_options(-mfloat-abi=soft)
+ add_link_options(-mfloat-abi=soft)
endif()
-string(APPEND CMAKE_C_FLAGS " " ${COMPILER_CP_FLAG})
-string(APPEND CMAKE_CXX_FLAGS " " ${COMPILER_CP_FLAG})
-string(APPEND CMAKE_ASM_FLAGS " " ${COMPILER_CP_FLAG})
-string(APPEND CMAKE_C_LINK_FLAGS " " ${LINKER_CP_OPTION})
-string(APPEND CMAKE_CXX_LINK_FLAGS " " ${LINKER_CP_OPTION})
-string(APPEND CMAKE_ASM_LINK_FLAGS " " ${LINKER_CP_OPTION})
-
-# For GNU Arm Embedded Toolchain doesn't emit __ARM_ARCH_8_1M_MAIN__, adding this macro manually.
-add_compile_definitions($<$<STREQUAL:${TFM_SYSTEM_ARCHITECTURE},armv8.1-m.main>:__ARM_ARCH_8_1M_MAIN__=1>)
+include(mcpu_features)
file(REAL_PATH "${CMAKE_SOURCE_DIR}/../" TOP_LEVEL_PROJECT_DIR)
add_compile_options(
- -specs=nano.specs
- -specs=nosys.specs
-Wall
-Wno-format
-Wno-unused-but-set-variable
@@ -196,16 +92,13 @@
# Strip /workspace/trusted-firmware-m
-fmacro-prefix-map=${CMAKE_SOURCE_DIR}/=
-mthumb
- $<$<COMPILE_LANGUAGE:C>:-std=c99>
- $<$<COMPILE_LANGUAGE:CXX>:-std=c++11>
- $<$<AND:$<COMPILE_LANGUAGE:C>,$<BOOL:${TFM_DEBUG_SYMBOLS}>>:-g>
- $<$<AND:$<COMPILE_LANGUAGE:CXX>,$<BOOL:${TFM_DEBUG_SYMBOLS}>>:-g>
+ $<$<AND:$<COMPILE_LANGUAGE:C,CXX>,$<BOOL:${TFM_DEBUG_SYMBOLS}>>:-g>
$<$<AND:$<COMPILE_LANGUAGE:C,CXX>,$<BOOL:${TFM_DEBUG_OPTIMISATION}>,$<CONFIG:Debug>>:-Og>
$<$<AND:$<COMPILE_LANGUAGE:C,CXX>,$<BOOL:${CONFIG_TFM_WARNINGS_ARE_ERRORS}>>:-Werror>
)
add_link_options(
- --entry=Reset_Handler
+ -mcpu=${TFM_SYSTEM_PROCESSOR}
-specs=nano.specs
-specs=nosys.specs
LINKER:-check-sections
@@ -287,43 +180,13 @@
endmacro()
+# Macro for converting the output *.axf file to finary files: bin, elf, hex
macro(add_convert_to_bin_target target)
get_target_property(bin_dir ${target} RUNTIME_OUTPUT_DIRECTORY)
-
add_custom_target(${target}_bin
- SOURCES ${bin_dir}/${target}.bin
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.bin
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O binary $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.bin
- )
-
- add_custom_target(${target}_elf
- SOURCES ${bin_dir}/${target}.elf
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.elf
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O elf32-littlearm $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.elf
- )
-
- add_custom_target(${target}_hex
- SOURCES ${bin_dir}/${target}.hex
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.hex
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O ihex $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.hex
- )
-
- add_custom_target(${target}_binaries
- ALL
- DEPENDS ${target}_bin
- DEPENDS ${target}_elf
- DEPENDS ${target}_hex
+ ALL DEPENDS ${target}
+ COMMAND ${CMAKE_OBJCOPY} -O binary $<TARGET_FILE:${target}> ${bin_dir}/${target}.bin
+ COMMAND ${CMAKE_OBJCOPY} -O elf32-littlearm $<TARGET_FILE:${target}> ${bin_dir}/${target}.elf
+ COMMAND ${CMAKE_OBJCOPY} -O ihex $<TARGET_FILE:${target}> ${bin_dir}/${target}.hex
)
endmacro()
diff --git a/toolchain_GNUARM.cmake b/toolchain_GNUARM.cmake
index e1a1c1d..cec82f9 100644
--- a/toolchain_GNUARM.cmake
+++ b/toolchain_GNUARM.cmake
@@ -7,21 +7,16 @@
set(CMAKE_SYSTEM_NAME Generic)
-find_program(CMAKE_C_COMPILER ${CROSS_COMPILE}-gcc)
-find_program(CMAKE_CXX_COMPILER ${CROSS_COMPILE}-g++)
-
-if(CMAKE_C_COMPILER STREQUAL "CMAKE_C_COMPILER-NOTFOUND")
- message(FATAL_ERROR "Could not find compiler: '${CROSS_COMPILE}-gcc'")
-endif()
-
-if(CMAKE_CXX_COMPILER STREQUAL "CMAKE_CXX_COMPILER-NOTFOUND")
- message(FATAL_ERROR "Could not find compiler: '${CROSS_COMPILE}-g++'")
-endif()
+set(CMAKE_C_COMPILER ${CROSS_COMPILE}-gcc)
+set(CMAKE_C_COMPILER_FORCED TRUE)
+set(CMAKE_C_STANDARD 99)
set(CMAKE_ASM_COMPILER ${CMAKE_C_COMPILER})
-set(LINKER_VENEER_OUTPUT_FLAG -Wl,--cmse-implib,--out-implib=)
-set(COMPILER_CMSE_FLAG -mcmse)
+# C++ support is not quaranted. This settings is to compile with RPi Pico SDK.
+set(CMAKE_CXX_COMPILER ${CROSS_COMPILE}-g++)
+set(CMAKE_CXX_COMPILER_FORCED TRUE)
+set(CMAKE_CXX_STANDARD 11)
# This variable name is a bit of a misnomer. The file it is set to is included
# at a particular step in the compiler initialisation. It is used here to
@@ -29,88 +24,32 @@
# with the Ninja generator.
set(CMAKE_USER_MAKE_RULES_OVERRIDE ${CMAKE_CURRENT_LIST_DIR}/cmake/set_extensions.cmake)
-# CMAKE_C_COMPILER_VERSION is not guaranteed to be defined.
-EXECUTE_PROCESS( COMMAND ${CMAKE_C_COMPILER} -dumpversion OUTPUT_VARIABLE GCC_VERSION )
+# CMAKE_C_COMPILER_VERSION is not initialised at this moment so do it manually
+EXECUTE_PROCESS(COMMAND ${CMAKE_C_COMPILER} -dumpversion OUTPUT_VARIABLE CMAKE_C_COMPILER_VERSION)
-# ===================== SEt toolchain CPU and Arch =============================
-
-if (DEFINED TFM_SYSTEM_PROCESSOR)
- if(TFM_SYSTEM_PROCESSOR MATCHES "cortex-m85" AND GCC_VERSION VERSION_LESS "13.0.0")
- # GNUARM until version 13 does not support the -mcpu=cortex-m85 flag
- message(WARNING "Cortex-m85 is only supported from GCC13. "
- "Falling back to -march usage for earlier versions.")
- else()
- set(CMAKE_SYSTEM_PROCESSOR ${TFM_SYSTEM_PROCESSOR})
-
- if (DEFINED TFM_SYSTEM_DSP)
- if (NOT TFM_SYSTEM_DSP)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nodsp")
- endif()
- endif()
- # GCC specifies that '+nofp' is available on following M-profile cpus: 'cortex-m4',
- # 'cortex-m7', 'cortex-m33', 'cortex-m35p', 'cortex-m55' and 'cortex-m85'.
- # Build fails if other M-profile cpu, such as 'cortex-m23', is added with '+nofp'.
- # Explicitly list those cpu to align with GCC description.
- if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- if(NOT CONFIG_TFM_ENABLE_FP AND
- (TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m4"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m7"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m33"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m35p"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m55"
- OR TFM_SYSTEM_PROCESSOR STREQUAL "cortex-m85"))
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nofp")
- endif()
- endif()
-
- if(TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main")
- if(NOT CONFIG_TFM_ENABLE_MVE)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nomve")
- endif()
- if(NOT CONFIG_TFM_ENABLE_MVE_FP)
- string(APPEND CMAKE_SYSTEM_PROCESSOR "+nomve.fp")
- endif()
- endif()
- endif()
-
+if (${CMAKE_C_COMPILER_VERSION} VERSION_LESS 10.3.1)
+ message(FATAL_ERROR "Please use GNU Arm toolchain version 10.3.1 or later")
endif()
-# CMAKE_SYSTEM_ARCH variable is not a built-in CMAKE variable. It is used to
-# set the compile and link flags when TFM_SYSTEM_PROCESSOR is not specified.
-# The variable name is choosen to align with the ARMCLANG toolchain file.
-set(CMAKE_SYSTEM_ARCH ${TFM_SYSTEM_ARCHITECTURE})
+function(min_toolchain_version mcpu gnu-version)
+ if (${TFM_SYSTEM_PROCESSOR} MATCHES ${mcpu} AND
+ ${CMAKE_C_COMPILER_VERSION} VERSION_LESS ${gnu-version})
+ message(FATAL_ERROR "-mcpu=${mcpu} is supported in GNU Arm version ${gnu-version} and later.\n"
+ "Please upgrade your toolchain to a supported version from: "
+ "https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads")
+ endif()
+endfunction()
-if(TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main")
- if(CONFIG_TFM_ENABLE_MVE)
- string(APPEND CMAKE_SYSTEM_ARCH "+mve")
- endif()
- if(CONFIG_TFM_ENABLE_MVE_FP)
- string(APPEND CMAKE_SYSTEM_ARCH "+mve.fp")
- endif()
-endif()
+# the lowest supported GCC version for a specific cpu
+min_toolchain_version("cortex-m85" "13.0.0")
+min_toolchain_version("cortex-m52" "14.2.0")
-if (DEFINED TFM_SYSTEM_DSP)
- # +nodsp modifier is only supported from GCC version 8.
- if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- # armv8.1-m.main arch does not have +nodsp option
- if ((NOT TFM_SYSTEM_ARCHITECTURE STREQUAL "armv8.1-m.main") AND
- NOT TFM_SYSTEM_DSP)
- string(APPEND CMAKE_SYSTEM_ARCH "+nodsp")
- endif()
- endif()
-endif()
-
-if(GCC_VERSION VERSION_GREATER_EQUAL "8.0.0")
- if(CONFIG_TFM_ENABLE_FP)
- string(APPEND CMAKE_SYSTEM_ARCH "+fp")
- endif()
-endif()
+include(mcpu_features)
file(REAL_PATH "${CMAKE_SOURCE_DIR}/../" TOP_LEVEL_PROJECT_DIR)
add_compile_options(
- -specs=nano.specs
- -specs=nosys.specs
+ -mfix-cmse-cve-2021-35465
-Wall
-Wno-format
-Wno-unused-but-set-variable
@@ -126,17 +65,13 @@
# Strip /workspace/trusted-firmware-m
-fmacro-prefix-map=${CMAKE_SOURCE_DIR}/=
-mthumb
- $<$<COMPILE_LANGUAGE:C>:-std=c99>
- $<$<COMPILE_LANGUAGE:CXX>:-std=c++11>
$<$<OR:$<BOOL:${TFM_DEBUG_SYMBOLS}>,$<BOOL:${TFM_CODE_COVERAGE}>>:-g>
- $<$<AND:$<COMPILE_LANGUAGE:C,CXX>,$<BOOL:${TFM_DEBUG_OPTIMISATION}>,$<CONFIG:Debug>>:-Og>
- $<$<AND:$<COMPILE_LANGUAGE:C,CXX>,$<BOOL:${CONFIG_TFM_WARNINGS_ARE_ERRORS}>>:-Werror>
+ $<$<AND:$<COMPILE_LANGUAGE:C>,$<BOOL:${TFM_DEBUG_OPTIMISATION}>,$<CONFIG:Debug>>:-Og>
+ $<$<AND:$<COMPILE_LANGUAGE:C>,$<BOOL:${CONFIG_TFM_WARNINGS_ARE_ERRORS}>>:-Werror>
)
-#
# Pointer Authentication Code and Branch Target Identification (PACBTI) Options
# Not currently supported for GNUARM.
-#
if(NOT ${CONFIG_TFM_BRANCH_PROTECTION_FEAT} STREQUAL BRANCH_PROTECTION_DISABLED)
message(FATAL_ERROR "BRANCH_PROTECTION NOT supported for GNU-ARM")
endif()
@@ -144,13 +79,11 @@
# Workaround to add diagnostics color while using Ninja generator.
# For reference: https://github.com/ninja-build/ninja/issues/174
if (CMAKE_GENERATOR STREQUAL "Ninja")
- add_compile_options(
- -fdiagnostics-color=always
- )
+ add_compile_options(-fdiagnostics-color=always)
endif()
add_link_options(
- --entry=Reset_Handler
+ -mcpu=${TFM_SYSTEM_PROCESSOR}
-specs=nano.specs
-specs=nosys.specs
LINKER:-check-sections
@@ -159,20 +92,12 @@
LINKER:--no-wchar-size-warning
)
+set(LINKER_VENEER_OUTPUT_FLAG -Wl,--cmse-implib,--out-implib=)
+
if(NOT CONFIG_TFM_MEMORY_USAGE_QUIET)
add_link_options(LINKER:--print-memory-usage)
endif()
-if (GCC_VERSION VERSION_LESS 7.3.1)
- message(FATAL_ERROR "Please use newer GNU Arm compiler version starting from 7.3.1.")
-endif()
-
-if (GCC_VERSION VERSION_EQUAL 10.2.1)
- message(FATAL_ERROR "GNU Arm compiler version 10-2020-q4-major has an issue in CMSE support."
- " Select other GNU Arm compiler versions instead."
- " See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99157 for the issue detail.")
-endif()
-
# GNU Arm compiler version greater equal than *11.3.Rel1*
# has a linker issue that required system calls are missing,
# such as _read and _write. Add stub functions of required
@@ -181,29 +106,11 @@
# READONLY linker script attribute is not supported in older
# GNU Arm compilers. For these version the preprocessor will
# remove the READONLY string from the linker scripts.
-if (GCC_VERSION VERSION_GREATER_EQUAL 11.3.1)
+if (CMAKE_C_COMPILER_VERSION VERSION_GREATER_EQUAL 11.3.1)
set(CONFIG_GNU_SYSCALL_STUB_ENABLED TRUE)
set(CONFIG_GNU_LINKER_READONLY_ATTRIBUTE TRUE)
endif()
-if (CMAKE_SYSTEM_PROCESSOR)
- set(CMAKE_C_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_CXX_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_ASM_FLAGS_INIT "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_C_LINK_FLAGS "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
- set(CMAKE_ASM_LINK_FLAGS "-mcpu=${CMAKE_SYSTEM_PROCESSOR}")
-else()
- set(CMAKE_C_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_CXX_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_ASM_FLAGS_INIT "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_C_LINK_FLAGS "-march=${CMAKE_SYSTEM_ARCH}")
- set(CMAKE_ASM_LINK_FLAGS "-march=${CMAKE_SYSTEM_ARCH}")
-endif()
-
-set(CMAKE_C_FLAGS ${CMAKE_C_FLAGS_INIT})
-set(CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS_INIT})
-set(CMAKE_ASM_FLAGS ${CMAKE_ASM_FLAGS_INIT})
-
set(BL2_COMPILER_CP_FLAG -mfloat-abi=soft)
set(BL2_LINKER_CP_OPTION -mfloat-abi=soft)
@@ -222,9 +129,6 @@
set(LINKER_CP_OPTION -mfloat-abi=soft)
endif()
-# For GNU Arm Embedded Toolchain doesn't emit __ARM_ARCH_8_1M_MAIN__, adding this macro manually.
-add_compile_definitions($<$<STREQUAL:${TFM_SYSTEM_ARCHITECTURE},armv8.1-m.main>:__ARM_ARCH_8_1M_MAIN__=1>)
-
macro(target_add_scatter_file target)
target_link_options(${target}
PRIVATE
@@ -274,44 +178,14 @@
)
endmacro()
+# Macro for converting the output *.axf file to finary files: bin, elf, hex
macro(add_convert_to_bin_target target)
get_target_property(bin_dir ${target} RUNTIME_OUTPUT_DIRECTORY)
-
add_custom_target(${target}_bin
- SOURCES ${bin_dir}/${target}.bin
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.bin
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O binary $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.bin
- )
-
- add_custom_target(${target}_elf
- SOURCES ${bin_dir}/${target}.elf
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.elf
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O elf32-littlearm $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.elf
- )
-
- add_custom_target(${target}_hex
- SOURCES ${bin_dir}/${target}.hex
- )
- add_custom_command(OUTPUT ${bin_dir}/${target}.hex
- DEPENDS ${target}
- COMMAND ${CMAKE_OBJCOPY}
- -O ihex $<TARGET_FILE:${target}>
- ${bin_dir}/${target}.hex
- )
-
- add_custom_target(${target}_binaries
- ALL
- DEPENDS ${target}_bin
- DEPENDS ${target}_elf
- DEPENDS ${target}_hex
+ ALL DEPENDS ${target}
+ COMMAND ${CMAKE_OBJCOPY} -O binary $<TARGET_FILE:${target}> ${bin_dir}/${target}.bin
+ COMMAND ${CMAKE_OBJCOPY} -O elf32-littlearm $<TARGET_FILE:${target}> ${bin_dir}/${target}.elf
+ COMMAND ${CMAKE_OBJCOPY} -O ihex $<TARGET_FILE:${target}> ${bin_dir}/${target}.hex
)
endmacro()