patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 7 Packages Projects Releases Wiki Activity

modules: trusted-firmware-m: fix incorrect check of <option> args

Browse source 
'IPC' and 'REGRESSION' are passed to the <option>
argument of cmake_parse_arguments, hence they are always defined,
and 'if (DEFINED)' would always return TRUE.

Use 'if' directly instead of 'if DEFINED' to check if these
options are set or not.

Ref: NCSDK-7702

Signed-off-by: Håkon Øye Amundsen <haakon.amundsen@nordicsemi.no>
This commit is contained in:
Håkon Øye Amundsen 2021-02-17 10:26:44 +01:00 committed by Ioannis Glaropoulos
parent c097690f2a
commit f52debbd8e
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/trusted-firmware-m/CMakeLists.txt
View file

@ -64,7 +64,7 @@ function(trusted_firmware_build)
endif()
set(TFM_BL2_ARG "-DBL2=${TFM_BL2}")
if(DEFINED TFM_IPC)
if(TFM_IPC)
set(TFM_IPC_ARG -DTFM_PSA_API=ON)
# PSA API awareness for the Non-Secure application
target_compile_definitions(app PRIVATE "TFM_PSA_API")
@ -74,7 +74,7 @@ function(trusted_firmware_build)
set(TFM_ISOLATION_LEVEL_ARG -DTFM_ISOLATION_LEVEL=${TFM_ISOLATION_LEVEL})
endif()
if(DEFINED TFM_REGRESSION)
if(TFM_REGRESSION)
set(TFM_REGRESSION_ARG -DTEST_S=ON)
endif()