From 5efcce9b0806fc8408ee4cbfbbe84342086a1dda Mon Sep 17 00:00:00 2001 From: Jordan Yates Date: Sat, 22 Feb 2025 21:48:57 +1000 Subject: [PATCH] secure_storage: swap `select` to `depends on` Mixing `select` and `depends on` is a common source of Kconfig dependency loops and should be avoided. Both `ZMS` and `SETTINGS` are more commonly used with `depends on` rather than `select`. The usage here also contradicts the Zephyr best practices guide for `select`: * Avoid selecting symbols with prompts or dependencies. Prefer depends on. Signed-off-by: Jordan Yates --- subsys/secure_storage/Kconfig.its_store | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/subsys/secure_storage/Kconfig.its_store b/subsys/secure_storage/Kconfig.its_store index 3370acdf6f4e..5cf1512caf68 100644 --- a/subsys/secure_storage/Kconfig.its_store +++ b/subsys/secure_storage/Kconfig.its_store @@ -14,7 +14,7 @@ config SECURE_STORAGE_ITS_STORE_IMPLEMENTATION_ZMS depends on FLASH_HAS_DRIVER_ENABLED \ && $(dt_path_enabled,$(DT_ITS_PARTITION)) \ && $(dt_node_has_compat,$(dt_node_parent,$(DT_ITS_PARTITION)),fixed-partitions) - select ZMS + depends on ZMS help This implementation of the ITS store module makes direct use of ZMS for storage. It needs a `secure_storage_its_partition` devicetree chosen property that points @@ -31,7 +31,7 @@ config SECURE_STORAGE_ITS_STORE_IMPLEMENTATION_SETTINGS && $(dt_node_has_compat,$(dt_node_parent,$(DT_SETTINGS_PARTITIION)),fixed-partitions))\ || ($(dt_path_enabled,$(DT_STORAGE_PARTITION)) \ && $(dt_node_has_compat,$(dt_node_parent,$(DT_STORAGE_PARTITION)),fixed-partitions))) - select SETTINGS + depends on SETTINGS config SECURE_STORAGE_ITS_STORE_IMPLEMENTATION_NONE bool "No ITS store module implementation"