[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 6 Jun 2023 12:59:58 +0300 (EEST)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Geert Uytterhoeven <geert+renesas@...der.be>
cc: Philipp Zabel <p.zabel@...gutronix.de>,
Arnd Bergmann <arnd@...db.de>,
Jacky Huang <ychuang3@...oton.com>,
linux-arm-kernel@...ts.infradead.org,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] reset: RESET_NUVOTON_MA35D1 should depend on ARCH_MA35
On Tue, 6 Jun 2023, Geert Uytterhoeven wrote:
> The Nuvoton MA35D1 reset controller is only present on Nuvoton MA35
> SoCs. Hence add a dependency on ARCH_MA35, to prevent asking the user
> about this driver when configuring a kernel without MA35 SoC support.
> Also, do not enable the driver by default when merely compile-testing.
>
> While at it, fix a misspelling of "Nuvoton".
>
> Fixes: e4bb55d6ccf0f774 ("reset: Add Nuvoton ma35d1 reset driver support")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
> ---
> drivers/reset/Kconfig | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig
> index 0cdf9d64c2185c89..ccd59ddd76100a51 100644
> --- a/drivers/reset/Kconfig
> +++ b/drivers/reset/Kconfig
> @@ -144,8 +144,9 @@ config RESET_NPCM
> BMC SoCs.
>
> config RESET_NUVOTON_MA35D1
> - bool "Nuvton MA35D1 Reset Driver"
> - default ARCH_MA35 || COMPILE_TEST
> + bool "Nuvoton MA35D1 Reset Driver"
> + depends on ARCH_MA35 || COMPILE_TEST
> + default ARCH_MA35
> help
> This enables the reset controller driver for Nuvoton MA35D1 SoC.
All adjustments make sense.
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
--
i.
Powered by blists - more mailing lists