2022-11-23 16:36:01

by Conor Dooley

[permalink] [raw]
Subject: [PATCH] clk: microchip: enable the MPFS clk driver by default if SOC_MICROCHIP_POLARFIRE

From: Conor Dooley <[email protected]>

With the intent of removing driver selects from Kconfig.socs in
arch/riscv, essential drivers that were being selected there could
instead by enabled by defaulting them to the value of the SoC's Kconfig
symbol.

Do so here & drop the depend on RISC-V - the SOC_ symbols are only
defined there anyway.

Signed-off-by: Conor Dooley <[email protected]>
---
I thought I sent this with the SiFive stuff a few weeks ago - but
obviously not!

CC: Claudiu Beznea <[email protected]>
CC: Conor Dooley <[email protected]>
CC: Daire McNamara <[email protected]>
CC: Michael Turquette <[email protected]>
CC: Stephen Boyd <[email protected]>
CC: [email protected]
CC: [email protected]
CC: [email protected]
---
drivers/clk/microchip/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
index b46e864b3bd8..e33e51978938 100644
--- a/drivers/clk/microchip/Kconfig
+++ b/drivers/clk/microchip/Kconfig
@@ -5,7 +5,8 @@ config COMMON_CLK_PIC32

config MCHP_CLK_MPFS
bool "Clk driver for PolarFire SoC"
- depends on (RISCV && SOC_MICROCHIP_POLARFIRE) || COMPILE_TEST
+ depends on SOC_MICROCHIP_POLARFIRE || COMPILE_TEST
+ default SOC_MICROCHIP_POLARFIRE
select AUXILIARY_BUS
help
Supports Clock Configuration for PolarFire SoC
--
2.38.1


2022-12-09 01:23:52

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] clk: microchip: enable the MPFS clk driver by default if SOC_MICROCHIP_POLARFIRE

Quoting Conor Dooley (2022-11-23 08:19:22)
> From: Conor Dooley <[email protected]>
>
> With the intent of removing driver selects from Kconfig.socs in
> arch/riscv, essential drivers that were being selected there could
> instead by enabled by defaulting them to the value of the SoC's Kconfig
> symbol.
>
> Do so here & drop the depend on RISC-V - the SOC_ symbols are only
> defined there anyway.
>
> Signed-off-by: Conor Dooley <[email protected]>
> ---

Applied to clk-next