* [PATCH v2] remoteproc: dead code cleanup in Kconfig for STM32_RPROC
@ 2026-04-17 22:13 Julian Braha
2026-04-22 14:24 ` Mathieu Poirier
0 siblings, 1 reply; 2+ messages in thread
From: Julian Braha @ 2026-04-17 22:13 UTC (permalink / raw)
To: mathieu.poirier, andersson, fabien.dessenne
Cc: ludovic.barre, loic.pallardy, arnaud.pouliquen, linux-kernel,
linux-remoteproc, Julian Braha
There is already an 'if REMOTEPROC' condition wrapping this config option,
making the 'depends on REMOTEPROC' statement a duplicate dependency
(dead code).
I propose leaving the outer 'if REMOTEPROC...endif' and removing the
individual 'depends on REMOTEPROC' statement.
This dead code was found by kconfirm, a static analysis tool for Kconfig.
Signed-off-by: Julian Braha <julianbraha@gmail.com>
---
v2: fix typo in commit message
Link to v1: https://lore.kernel.org/all/20260330224545.29769-1-julianbraha@gmail.com/
---
drivers/remoteproc/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig
index ee54436fea5a..c78e431b7b2d 100644
--- a/drivers/remoteproc/Kconfig
+++ b/drivers/remoteproc/Kconfig
@@ -316,7 +316,6 @@ config ST_SLIM_REMOTEPROC
config STM32_RPROC
tristate "STM32 remoteproc support"
depends on ARCH_STM32 || COMPILE_TEST
- depends on REMOTEPROC
select MAILBOX
help
Say y here to support STM32 MCU processors via the
--
2.53.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] remoteproc: dead code cleanup in Kconfig for STM32_RPROC
2026-04-17 22:13 [PATCH v2] remoteproc: dead code cleanup in Kconfig for STM32_RPROC Julian Braha
@ 2026-04-22 14:24 ` Mathieu Poirier
0 siblings, 0 replies; 2+ messages in thread
From: Mathieu Poirier @ 2026-04-22 14:24 UTC (permalink / raw)
To: Julian Braha
Cc: andersson, fabien.dessenne, ludovic.barre, loic.pallardy,
arnaud.pouliquen, linux-kernel, linux-remoteproc
On Fri, Apr 17, 2026 at 11:13:37PM +0100, Julian Braha wrote:
> There is already an 'if REMOTEPROC' condition wrapping this config option,
> making the 'depends on REMOTEPROC' statement a duplicate dependency
> (dead code).
>
> I propose leaving the outer 'if REMOTEPROC...endif' and removing the
> individual 'depends on REMOTEPROC' statement.
>
> This dead code was found by kconfirm, a static analysis tool for Kconfig.
>
> Signed-off-by: Julian Braha <julianbraha@gmail.com>
> ---
> v2: fix typo in commit message
> Link to v1: https://lore.kernel.org/all/20260330224545.29769-1-julianbraha@gmail.com/
> ---
> drivers/remoteproc/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
I have picked-up this patch. It will show up in rproc-next when 7.1-rc1 is
released next week.
Thanks,
Mathieu
> diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig
> index ee54436fea5a..c78e431b7b2d 100644
> --- a/drivers/remoteproc/Kconfig
> +++ b/drivers/remoteproc/Kconfig
> @@ -316,7 +316,6 @@ config ST_SLIM_REMOTEPROC
> config STM32_RPROC
> tristate "STM32 remoteproc support"
> depends on ARCH_STM32 || COMPILE_TEST
> - depends on REMOTEPROC
> select MAILBOX
> help
> Say y here to support STM32 MCU processors via the
> --
> 2.53.0
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-04-22 14:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-17 22:13 [PATCH v2] remoteproc: dead code cleanup in Kconfig for STM32_RPROC Julian Braha
2026-04-22 14:24 ` Mathieu Poirier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox