Skip to content

Commit 455f4fb

Browse files
Vge0rgetomi-font
authored andcommitted
nordic_nrf: Fix UART and FLASH build issues
The nordic platforms enable SECURE_UART1 by default, enabling this without TFM_SPM_LOG_RAW_ENABLED doesn't make sense since it will configure a UART as secure and then it will not use it. This also causes a building issue since the code expects both these two options to be enabled. Also fixes a building issue related to flash. The cmsis flash driver is used by both PLATFORM_DEFAULT_OTP and byITS but it was only included when ITS was build. So it was failing for builds with PLATFORM_DEFAULT_OTP and without ITS Signed-off-by: Georgios Vasilakis <[email protected]> Change-Id: I4f8f75ec2a2817634c190ebb844b9be9cc7f1a45 (cherry picked from commit c4f46d4)
1 parent 5e3e071 commit 455f4fb

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

platform/ext/target/nordic_nrf/common/core/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,11 @@ target_include_directories(platform_s
7878
services/include
7979
)
8080

81+
if(PLATFORM_DEFAULT_OTP OR TFM_PARTITION_INTERNAL_TRUSTED_STORAGE)
8182
set(nvm_sources
82-
$<$<BOOL:${TFM_PARTITION_INTERNAL_TRUSTED_STORAGE}>:${CMAKE_CURRENT_SOURCE_DIR}/cmsis_drivers/Driver_Flash.c>
83+
${CMAKE_CURRENT_SOURCE_DIR}/cmsis_drivers/Driver_Flash.c
8384
)
85+
endif()
8486

8587
if(HAS_RRAMC)
8688
list(APPEND nvm_sources

platform/ext/target/nordic_nrf/common/core/config.cmake

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,3 +45,4 @@ set(CONFIG_TFM_USE_TRUSTZONE ON)
4545
set(TFM_MULTI_CORE_TOPOLOGY OFF)
4646

4747
set(NRF_SECURE_UART_INSTANCE 1 CACHE STRING "The UART instance number to use for secure UART")
48+
set(TFM_SPM_LOG_RAW_ENABLED ON CACHE BOOL "Enable LOG raw")

0 commit comments

Comments
 (0)