style: fix 'if (' usage in cmake files

Replace with 'if(' and 'else(' per the cmake style guidelines.

Signed-off-by: Anas Nashif <[email protected]>

Author: nashif

CMakeLists.txt

@@ -409,7 +409,7 @@ if(CONFIG_COMPILER_FREESTANDING)
   zephyr_compile_options($<$<COMPILE_LANGUAGE:ASM>:$<TARGET_PROPERTY:compiler,freestanding>>)
 endif()
 
-if (CONFIG_PICOLIBC AND NOT CONFIG_PICOLIBC_IO_FLOAT)
+if(CONFIG_PICOLIBC AND NOT CONFIG_PICOLIBC_IO_FLOAT)
   # @Intent: Set compiler specific flag to disable printf-related optimizations
   zephyr_compile_options($<$<COMPILE_LANGUAGE:C>:$<TARGET_PROPERTY:compiler,no_printf_return_value>>)
 endif()
@@ -1157,7 +1157,7 @@ endif()
 
 get_property(OUTPUT_FORMAT        GLOBAL PROPERTY PROPERTY_OUTPUT_FORMAT)
 
-if (CONFIG_CODE_DATA_RELOCATION)
+if(CONFIG_CODE_DATA_RELOCATION)
   set(CODE_RELOCATION_DEP code_relocation_source_lib)
 endif() # CONFIG_CODE_DATA_RELOCATION
 
@@ -2134,7 +2134,7 @@ if(CONFIG_BUILD_OUTPUT_INFO_HEADER)
     )
 endif()
 
-if (CONFIG_LLEXT AND CONFIG_LLEXT_EXPORT_BUILTINS_BY_SLID)
+if(CONFIG_LLEXT AND CONFIG_LLEXT_EXPORT_BUILTINS_BY_SLID)
   #slidgen must be the first post-build command to be executed
   #on the Zephyr ELF to ensure that all other commands, such as
   #binary file generation, are operating on a preparated ELF.
@@ -2200,7 +2200,7 @@ if(signing_script)
 endif()
 
 # Generate USB-C VIF policies in XML format
-if (CONFIG_BUILD_OUTPUT_VIF)
+if(CONFIG_BUILD_OUTPUT_VIF)
   include(${CMAKE_CURRENT_LIST_DIR}/cmake/vif.cmake)
 endif()
 
@@ -2251,7 +2251,7 @@ if(LOG_DICT_DB_NAME_ARG)
       --build-header ${PROJECT_BINARY_DIR}/include/generated/zephyr/version.h
   )
 
-  if (NOT CONFIG_LOG_DICTIONARY_DB_TARGET)
+  if(NOT CONFIG_LOG_DICTIONARY_DB_TARGET)
     # If not using a separate target for generating logging dictionary
     # database, add the generation to post build command to make sure
     # the database is actually being generated.

arch/arm/core/CMakeLists.txt

@@ -27,7 +27,7 @@ add_subdirectory_ifdef(CONFIG_ARM_AARCH32_MMU mmu)
 add_subdirectory_ifdef(CONFIG_CPU_AARCH32_CORTEX_R cortex_a_r)
 add_subdirectory_ifdef(CONFIG_CPU_AARCH32_CORTEX_A cortex_a_r)
 
-if (CONFIG_ARM_ZIMAGE_HEADER)
+if(CONFIG_ARM_ZIMAGE_HEADER)
 zephyr_linker_sources(ROM_START SORT_KEY 0x0vectors zimage_header.ld)
 zephyr_linker_sources(ROM_START SORT_KEY 0x1vectors vector_table.ld)
 zephyr_linker_sources(ROM_START SORT_KEY 0x2vectors cortex_m/vector_table_pad.ld)

arch/arm/core/cortex_m/CMakeLists.txt

@@ -49,13 +49,13 @@ if(CONFIG_NULL_POINTER_EXCEPTION_DETECTION_DWT)
 endif()
 
 if(CONFIG_CORTEX_M_DWT)
-	if (CONFIG_TIMING_FUNCTIONS)
+	if(CONFIG_TIMING_FUNCTIONS)
 		zephyr_library_sources(timing.c)
 	endif()
 endif()
 
-if (CONFIG_SW_VECTOR_RELAY)
-if (CONFIG_CPU_CORTEX_M_HAS_VTOR)
+if(CONFIG_SW_VECTOR_RELAY)
+if(CONFIG_CPU_CORTEX_M_HAS_VTOR)
   set(relay_vector_table_sort_key relay_vectors)
 else()
 # Using 0x0 prefix will result in placing the relay vector table section
@@ -72,14 +72,14 @@ zephyr_linker_sources(
   )
 endif()
 
-if (CONFIG_SW_VECTOR_RELAY OR CONFIG_SW_VECTOR_RELAY_CLIENT)
+if(CONFIG_SW_VECTOR_RELAY OR CONFIG_SW_VECTOR_RELAY_CLIENT)
 zephyr_linker_sources(
   RAM_SECTIONS
   vt_pointer_section.ld
   )
 endif()
 
-if (CONFIG_CPU_CORTEX_M_HAS_VTOR)
+if(CONFIG_CPU_CORTEX_M_HAS_VTOR)
 zephyr_linker_sources_ifdef(CONFIG_SRAM_VECTOR_TABLE
   RAM_SECTIONS
   # Maybe need to be changed in order to be placed at the beginning of RAM

arch/arm/core/mpu/CMakeLists.txt

@@ -11,10 +11,10 @@ if(CONFIG_CPU_CORTEX_M AND NOT CONFIG_CPU_HAS_CUSTOM_FIXED_SOC_MPU_REGIONS)
   zephyr_library_sources_ifdef(CONFIG_CPU_HAS_NXP_SYSMPU nxp_mpu_regions.c)
 endif()
 
-if (CONFIG_CPU_AARCH32_CORTEX_R)
+if(CONFIG_CPU_AARCH32_CORTEX_R)
 zephyr_library_include_directories(cortex_a_r)
-elseif (CONFIG_CPU_CORTEX_M)
+elseif(CONFIG_CPU_CORTEX_M)
 zephyr_library_include_directories(cortex_m)
-else ()
+else()
 message(FATAL_ERROR "CPU is not Cortex-A/R/M")
-endif ()
+endif()

arch/arm64/core/CMakeLists.txt

@@ -41,9 +41,9 @@ zephyr_library_sources_ifdef(CONFIG_HAS_ARM_SMCCC smccc-call.S)
 zephyr_library_sources_ifdef(CONFIG_AARCH64_IMAGE_HEADER header.S)
 zephyr_library_sources_ifdef(CONFIG_SEMIHOST semihost.c)
 zephyr_library_sources_ifdef(CONFIG_DEBUG_COREDUMP coredump.c)
-if ((CONFIG_MP_MAX_NUM_CPUS GREATER 1) OR (CONFIG_SMP))
+if((CONFIG_MP_MAX_NUM_CPUS GREATER 1) OR (CONFIG_SMP))
   zephyr_library_sources(smp.c)
-endif ()
+endif()
 
 zephyr_cc_option_ifdef(CONFIG_USERSPACE -mno-outline-atomics)
 zephyr_cc_option_ifdef(CONFIG_FRAME_POINTER -mno-omit-leaf-frame-pointer)

arch/common/CMakeLists.txt

@@ -86,7 +86,7 @@ zephyr_linker_sources_ifdef(CONFIG_NOCACHE_MEMORY
   nocache.ld
 )
 
-if (DEFINED CONFIG_ARCH_SUPPORTS_ROM_OFFSET)
+if(DEFINED CONFIG_ARCH_SUPPORTS_ROM_OFFSET)
   # Exclamation mark is printable character with lowest number in ASCII table.
   # We are sure that this file will be included as a first.
   zephyr_linker_sources(ROM_START SORT_KEY ! rom_start_address.ld)
@@ -101,7 +101,7 @@ endif()
 
 # isr_tables is a normal CMake library and not a zephyr_library because it
 # should not be --whole-archive'd
-if (CONFIG_GEN_ISR_TABLES)
+if(CONFIG_GEN_ISR_TABLES)
   add_library(isr_tables
     isr_tables.c
   )

arch/posix/CMakeLists.txt

@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: Apache-2.0
 
-if (NOT CMAKE_HOST_UNIX OR CMAKE_HOST_APPLE)
+if(NOT CMAKE_HOST_UNIX OR CMAKE_HOST_APPLE)
   message(FATAL_ERROR "The POSIX architecture only works on Linux. If on Windows or macOS "
           "consider using a virtual machine to run a Linux guest.")
 endif()
@@ -43,14 +43,14 @@ if(EXISTS ${CMAKE_CURRENT_LIST_DIR}/${CMAKE_HOST_SYSTEM_NAME}.${CMAKE_HOST_SYSTE
   # @Intent: Set necessary compiler & linker options for this specific host architecture & OS
   include(${CMAKE_HOST_SYSTEM_NAME}.${CMAKE_HOST_SYSTEM_PROCESSOR}.cmake)
 else() # Linux.x86_64
-  if (CONFIG_64BIT)
+  if(CONFIG_64BIT)
     # some gcc versions fail to build without -fPIC
     zephyr_compile_options(-m64 -fPIC)
     zephyr_link_libraries(-m64)
 
     target_link_options(native_simulator INTERFACE "-m64")
     target_compile_options(native_simulator INTERFACE "-m64")
-  else ()
+  else()
     zephyr_compile_options(-m32)
     zephyr_link_libraries(-m32)
 
@@ -64,7 +64,7 @@ else() # Linux.x86_64
     check_set_compiler_property(APPEND PROPERTY fpsse2 "SHELL:-msse2 -mfpmath=sse")
     zephyr_compile_options($<TARGET_PROPERTY:compiler,fpsse2>)
     target_compile_options(native_simulator INTERFACE "$<TARGET_PROPERTY:compiler,fpsse2>")
-  endif ()
+  endif()
 endif()
 
 zephyr_compile_options(
@@ -81,7 +81,7 @@ zephyr_ld_options(
   -Wl,--unresolved-symbols=ignore-all
 )
 
-if (NOT CONFIG_EXTERNAL_LIBC)
+if(NOT CONFIG_EXTERNAL_LIBC)
   # Get the *compiler* include path, that is where the *compiler* provided headers are (not the
   # default libC ones). This includes basic headers like stdint.h, stddef.h or float.h
   # We expect something like
@@ -106,7 +106,7 @@ if (NOT CONFIG_EXTERNAL_LIBC)
   )
 endif()
 
-if (CONFIG_COMPILER_WARNINGS_AS_ERRORS)
+if(CONFIG_COMPILER_WARNINGS_AS_ERRORS)
   target_compile_options(native_simulator INTERFACE $<TARGET_PROPERTY:compiler,warnings_as_errors>)
 endif()
 
@@ -126,7 +126,7 @@ if(CONFIG_COVERAGE)
   target_link_options(native_simulator INTERFACE $<TARGET_PROPERTY:linker,coverage>)
 endif()
 
-if (CONFIG_GPROF)
+if(CONFIG_GPROF)
   zephyr_compile_options($<TARGET_PROPERTY:compiler,gprof>)
   zephyr_link_libraries($<TARGET_PROPERTY:linker,gprof>)
 

arch/posix/Linux.aarch64.cmake

@@ -14,21 +14,21 @@ execute_process(
   OUTPUT_STRIP_TRAILING_WHITESPACE
 )
 
-if (CONFIG_64BIT)
-  if (${WORDSIZE} STREQUAL "32")
+if(CONFIG_64BIT)
+  if(${WORDSIZE} STREQUAL "32")
     message(FATAL_ERROR
       "CONFIG_64BIT=y but this Aarch64 machine has a 32-bit userspace.\n"
       "If you were targeting native_sim/native/64, target native_sim instead.\n"
       "Otherwise, be sure to define CONFIG_64BIT appropriately.\n"
     )
   endif()
   zephyr_compile_options(-fPIC)
-else ()
-  if (${WORDSIZE} STREQUAL "64")
+else()
+  if(${WORDSIZE} STREQUAL "64")
     message(FATAL_ERROR
       "CONFIG_64BIT=n but this Aarch64 machine has a 64-bit userspace.\n"
       "If you were targeting native_sim, target native_sim/native/64 instead.\n"
       "Otherwise, be sure to define CONFIG_64BIT appropriately.\n"
     )
   endif()
-endif ()
+endif()

arch/riscv/core/CMakeLists.txt

@@ -13,11 +13,11 @@ zephyr_library_sources(
   thread.c
 )
 
-if ((CONFIG_MP_MAX_NUM_CPUS GREATER 1) OR (CONFIG_SMP))
+if((CONFIG_MP_MAX_NUM_CPUS GREATER 1) OR (CONFIG_SMP))
   zephyr_library_sources(smp.c)
-endif ()
+endif()
 
-if (CONFIG_SMP)
+if(CONFIG_SMP)
   zephyr_library_sources(ipi.c)
 
   zephyr_library_sources_ifdef(CONFIG_RISCV_SMP_IPI_CLINT ipi_clint.c)

arch/x86/CMakeLists.txt

@@ -32,7 +32,7 @@ else()
 endif()
 
 # Always set for 64-bit (long mode requires page tables), optional for 32-bit
-if (CONFIG_MMU)
+if(CONFIG_MMU)
   set(GEN_MMU ${ZEPHYR_BASE}/arch/x86/gen_mmu.py)
 
   if(DEFINED X86_EXTRA_GEN_MMU_ARGUMENTS)