cmake: add SDP developer mode

Add SDP developer mode in which newly generated ASM file is always
included in build instead of the previous one.
The previous behaviour is now a user mode with one difference:
if AMS files are different, build error will occur instead of
a warning.

Signed-off-by: Magdalena Pastula <[email protected]>

Author: magp-nordic

cmake/sdp.cmake

@@ -20,7 +20,12 @@ function(sdp_assembly_install target hrt_srcs)
   sdp_assembly_prepare_install(${CONFIG_SOC} "${hrt_srcs}")
   sdp_assembly_target_sources(${CONFIG_SOC} ${target} "${hrt_srcs}")
 
-  add_dependencies(${target} asm_check)
+  if(CONFIG_SDP_DEVELOPER_MODE)
+    add_dependencies(asm_install asm_check)
+    add_dependencies(${target} asm_install)
+  else()
+    add_dependencies(${target} asm_check)
+  endif()
 endfunction()
 
 function(sdp_assembly_target_sources soc target hrt_srcs)
@@ -87,10 +92,16 @@ function(sdp_assembly_check soc hrt_srcs)
     )
   endif()
 
+  if(CONFIG_SDP_DEVELOPER_MODE)
+    set(dev_mode TRUE)
+  else()
+    set(dev_mode FALSE)
+  endif()
+
   string(REPLACE ";" "$<SEMICOLON>" hrt_srcs_semi "${hrt_srcs}")
 
   add_custom_target(asm_check
-    COMMAND ${CMAKE_COMMAND} -D hrt_srcs="${hrt_srcs_semi}" -D soc="${soc}" -P ${ZEPHYR_NRF_MODULE_DIR}/cmake/sdp_asm_check.cmake
+    COMMAND ${CMAKE_COMMAND} -D hrt_srcs="${hrt_srcs_semi}" -D soc="${soc}" -D dev_mode=${dev_mode} -P ${ZEPHYR_NRF_MODULE_DIR}/cmake/sdp_asm_check.cmake
     COMMENT ${asm_check_msg}
   )
 

cmake/sdp_asm_check.cmake

@@ -9,11 +9,16 @@
 # Arguments:
 # hrt_srcs - path to the .c file(s) to verify
 # soc      - name of the SoC the code is being built for
+# dev_mode - variable indicating if developer mode is enabled
 function(asm_check)
   if(NOT DEFINED soc)
     message(FATAL_ERROR "asm_check missing soc argument.")
   endif()
 
+  if(NOT DEFINED dev_mode)
+    set(dev_mode FALSE)
+  endif()
+
   foreach(hrt_src ${hrt_srcs})
     get_filename_component(asm_filename ${hrt_src} NAME_WE)  # filename without extension
     get_filename_component(src_dir ${hrt_src} DIRECTORY)
@@ -25,9 +30,16 @@ function(asm_check)
     if( compare_result EQUAL 0)
       message("File ${asm_filename}-${soc}.s has not changed.")
     elseif( compare_result EQUAL 1)
-      message(WARNING "${asm_filename}-${soc}.s ASM file content has changed.\
-              If you want to include the new ASM in build, \
-              please run `ninja asm_install` in FLPR build directory and build again")
+      if(dev_mode)
+        message(WARNING "${asm_filename}-${soc}.s ASM file content has changed.\
+                It will be updated and included in build.")
+      else()
+        message(FATAL_ERROR "${asm_filename}-${soc}.s ASM file content has changed.\
+                If you want to include the new ASM in build, \
+                please run `ninja asm_install` in FLPR build directory and build again. \
+                If you want to disable this error and include new ASM in build every time,\
+                enable SB_CONFIG_DEVELOPER_MODE option.")
+      endif()
     else()
       message("Something went wrong when comparing ${asm_filename}-${soc}.s and ${asm_filename}-${soc}-temp.s")
     endif()

cmake/sysbuild/sdp.cmake

@@ -30,6 +30,9 @@ if(SB_CONFIG_SDP)
       sdp_apply_flpr_fault_timer_params(${SB_CONFIG_SDP_FLPR_APP_FAULT_TIMEOUT})
     endif()
   endif()
+  if(SB_CONFIG_SDP_DEVELOPER_MODE)
+    set_config_bool(sdp CONFIG_SDP_DEVELOPER_MODE y)
+  endif()
 
   sdp_apply_snippets(${snippet_name})
   set(snippet_name)