[PATCH] nvmem: cleanup dead code in Kconfig

[PATCH] nvmem: cleanup dead code in Kconfig

[Julian Braha]

There is already an 'if NVMEM' condition wrapping NVMEM_RCAR_EFUSE,
making the 'depends on' statement a duplicate dependency (dead code).

I propose leaving the outer 'if NVMEM...endif' and removing the
individual 'depends on' statement.

This dead code was found by kconfirm, a static analysis tool for Kconfig.

Signed-off-by: Julian Braha <julianbraha@gmail.com>
---
 drivers/nvmem/Kconfig | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/nvmem/Kconfig b/drivers/nvmem/Kconfig
index 74ddbd0f79b0..a6f422a5a9a5 100644
--- a/drivers/nvmem/Kconfig
+++ b/drivers/nvmem/Kconfig
@@ -303,7 +303,6 @@ config NVMEM_RAVE_SP_EEPROM
 config NVMEM_RCAR_EFUSE
 	tristate "Renesas R-Car Gen4 E-FUSE support"
 	depends on (ARCH_RENESAS && ARM64) || COMPILE_TEST
-	depends on NVMEM
 	help
 	  Enable support for reading the fuses in the E-FUSE or OTP
 	  non-volatile memory block on Renesas R-Car Gen4 SoCs.
@@ -483,4 +482,4 @@ config NVMEM_QORIQ_EFUSE
 	  This driver can also be built as a module. If so, the module
 	  will be called nvmem_qoriq_efuse.
 
-endif
+endif # NVMEM
-- 
2.51.2

Re: [PATCH] nvmem: cleanup dead code in Kconfig

[Srinivas Kandagatla]

On Tue, 31 Mar 2026 13:04:59 +0100, Julian Braha wrote:
> There is already an 'if NVMEM' condition wrapping NVMEM_RCAR_EFUSE,
> making the 'depends on' statement a duplicate dependency (dead code).
> 
> I propose leaving the outer 'if NVMEM...endif' and removing the
> individual 'depends on' statement.
> 
> This dead code was found by kconfirm, a static analysis tool for Kconfig.
> 
> [...]

Applied, thanks!

[1/1] nvmem: cleanup dead code in Kconfig
      commit: 3a99fdfe8380f5d7c5f9e652187681c471fa4a27

Best regards,
-- 
Srinivas Kandagatla <srini@kernel.org>