From 8321d0b17f536deab074a08c9744707a53eb5294 Mon Sep 17 00:00:00 2001 From: Aleksandr Khromykh Date: Fri, 29 Nov 2024 09:25:38 +0100 Subject: [PATCH] secure_storage: change dependencies on settings and flash The commit changes dependencies on SETTINGS and FLASH to avoid circular dependencies if security storage subsystem users select them too. Signed-off-by: Aleksandr Khromykh --- samples/psa/its/overlay-secure_storage.conf | 4 ++++ samples/psa/persistent_key/overlay-secure_storage.conf | 4 ++++ subsys/secure_storage/Kconfig | 1 - subsys/secure_storage/Kconfig.its_store | 4 +--- .../secure_storage/psa/crypto/overlay-secure_storage.conf | 4 ++++ .../subsys/secure_storage/psa/its/overlay-secure_storage.conf | 4 ++++ 6 files changed, 17 insertions(+), 4 deletions(-) diff --git a/samples/psa/its/overlay-secure_storage.conf b/samples/psa/its/overlay-secure_storage.conf index 3473ae389101ab5..112986420a8a749 100644 --- a/samples/psa/its/overlay-secure_storage.conf +++ b/samples/psa/its/overlay-secure_storage.conf @@ -7,4 +7,8 @@ CONFIG_MBEDTLS_PSA_CRYPTO_C=y # On top of that, the ITS implementation uses the stack for buffers. CONFIG_MAIN_STACK_SIZE=3072 +CONFIG_FLASH=y +CONFIG_FLASH_MAP=y +CONFIG_NVS=y +CONFIG_SETTINGS=y CONFIG_SECURE_STORAGE=y diff --git a/samples/psa/persistent_key/overlay-secure_storage.conf b/samples/psa/persistent_key/overlay-secure_storage.conf index 3473ae389101ab5..112986420a8a749 100644 --- a/samples/psa/persistent_key/overlay-secure_storage.conf +++ b/samples/psa/persistent_key/overlay-secure_storage.conf @@ -7,4 +7,8 @@ CONFIG_MBEDTLS_PSA_CRYPTO_C=y # On top of that, the ITS implementation uses the stack for buffers. CONFIG_MAIN_STACK_SIZE=3072 +CONFIG_FLASH=y +CONFIG_FLASH_MAP=y +CONFIG_NVS=y +CONFIG_SETTINGS=y CONFIG_SECURE_STORAGE=y diff --git a/subsys/secure_storage/Kconfig b/subsys/secure_storage/Kconfig index 11e78a7dbd2ce88..86c589e81a4ffb8 100644 --- a/subsys/secure_storage/Kconfig +++ b/subsys/secure_storage/Kconfig @@ -65,7 +65,6 @@ endif menuconfig SECURE_STORAGE_ITS_STORE_MODULE bool "ITS store module" - imply FLASH # for FLASH_HAS_DRIVER_ENABLED help The module that handles the storage/retrieval of the ITS data to/from NVM. Zephyr's ITS implementation calls into it. diff --git a/subsys/secure_storage/Kconfig.its_store b/subsys/secure_storage/Kconfig.its_store index 9e4d9b650205931..da9f2999ec195a1 100644 --- a/subsys/secure_storage/Kconfig.its_store +++ b/subsys/secure_storage/Kconfig.its_store @@ -10,9 +10,7 @@ config SECURE_STORAGE_ITS_STORE_IMPLEMENTATION_SETTINGS depends on FLASH_HAS_DRIVER_ENABLED \ && $(dt_path_enabled,$(DT_STORAGE_PARTITION)) \ && $(dt_node_has_compat,$(dt_node_parent,$(DT_STORAGE_PARTITION)),fixed-partitions) - imply FLASH_MAP - imply NVS - select SETTINGS + depends on SETTINGS config SECURE_STORAGE_ITS_STORE_IMPLEMENTATION_NONE bool "No ITS store module implementation" diff --git a/tests/subsys/secure_storage/psa/crypto/overlay-secure_storage.conf b/tests/subsys/secure_storage/psa/crypto/overlay-secure_storage.conf index e076008277b6641..b9320c450cd565f 100644 --- a/tests/subsys/secure_storage/psa/crypto/overlay-secure_storage.conf +++ b/tests/subsys/secure_storage/psa/crypto/overlay-secure_storage.conf @@ -8,6 +8,10 @@ CONFIG_TIMER_RANDOM_GENERATOR=y CONFIG_COMMON_LIBC_MALLOC_ARENA_SIZE=2048 CONFIG_MBEDTLS_PSA_CRYPTO_C=y +CONFIG_FLASH=y +CONFIG_FLASH_MAP=y +CONFIG_NVS=y +CONFIG_SETTINGS=y CONFIG_SECURE_STORAGE=y # For testing isolation between the different callers of the ITS. CONFIG_SECURE_STORAGE_PS_IMPLEMENTATION_ITS=y diff --git a/tests/subsys/secure_storage/psa/its/overlay-secure_storage.conf b/tests/subsys/secure_storage/psa/its/overlay-secure_storage.conf index 3ca9fdabcf25eb1..db6285b525ac366 100644 --- a/tests/subsys/secure_storage/psa/its/overlay-secure_storage.conf +++ b/tests/subsys/secure_storage/psa/its/overlay-secure_storage.conf @@ -1,4 +1,8 @@ CONFIG_ZTEST_STACK_SIZE=3072 CONFIG_MAIN_STACK_SIZE=2048 +CONFIG_FLASH=y +CONFIG_FLASH_MAP=y +CONFIG_NVS=y +CONFIG_SETTINGS=y CONFIG_SECURE_STORAGE=y