[nrf fromtree] secure_fw: crypto: add -Wno-unused-const-variable

Flavio Ceolin · mbolivar-nordic · commit 150e7b43dbf6 · 2023-02-20T15:52:10.000-08:00
When using TF-M with upstream MbedTLS, the upstream project generates a warning about const variables being set but not unused. This warning causes CI to fail in some downstream consumers of TF-M (Zephyr in this case). Add `-Wno-unused-const-variable` avoids this warning. Author: Flavio Ceolin <flavio.ceolin@intel.com> Signed-off-by: Kevin Townsend <kevin.townsend@linaro.org> Change-Id: I106d0d8598a6d075e3824202038fc37e0c5a9861 Signed-off-by: Joakim Andersson <joakim.andersson@nordicsemi.no> (cherry picked from commit 1b7ee9d)
diff --git a/secure_fw/partitions/crypto/CMakeLists.txt b/secure_fw/partitions/crypto/CMakeLists.txt
@@ -185,7 +185,9 @@ target_sources(${MBEDTLS_TARGET_PREFIX}mbedcrypto
 
 target_compile_options(${MBEDTLS_TARGET_PREFIX}mbedcrypto
     PRIVATE
+        $<$<C_COMPILER_ID:GNU>:-Wno-unused-const-variable>
         $<$<C_COMPILER_ID:GNU>:-Wno-unused-parameter>
+        $<$<C_COMPILER_ID:ARMClang>:-Wno-unused-const-variable>
         $<$<C_COMPILER_ID:ARMClang>:-Wno-unused-parameter>
 )
 

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,9 @@ target_sources(${MBEDTLS_TARGET_PREFIX}mbedcrypto`
`185`	`185`
`186`	`186`	`target_compile_options(${MBEDTLS_TARGET_PREFIX}mbedcrypto`
`187`	`187`	`PRIVATE`
	`188`	`+ $<$<C_COMPILER_ID:GNU>:-Wno-unused-const-variable>`
`188`	`189`	`$<$<C_COMPILER_ID:GNU>:-Wno-unused-parameter>`
	`190`	`+ $<$<C_COMPILER_ID:ARMClang>:-Wno-unused-const-variable>`
`189`	`191`	`$<$<C_COMPILER_ID:ARMClang>:-Wno-unused-parameter>`
`190`	`192`	`)`
`191`	`193`