[PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE

Conor Dooley posted 3 patches 2 months, 2 weeks ago
There is a newer version of this series
[PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE
Posted by Conor Dooley 2 months, 2 weeks ago
From: Conor Dooley <conor.dooley@microchip.com>

This driver is used by non-polarfire devices now, and the ARCH_MICROCHIP
symbol has been defined for some time on RISCV so drop it without any
functional change.

Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
---
 drivers/clk/microchip/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
index cab9a909893b..a0ef14310417 100644
--- a/drivers/clk/microchip/Kconfig
+++ b/drivers/clk/microchip/Kconfig
@@ -5,8 +5,8 @@ config COMMON_CLK_PIC32
 
 config MCHP_CLK_MPFS
 	bool "Clk driver for PolarFire SoC"
-	depends on ARCH_MICROCHIP_POLARFIRE || COMPILE_TEST
-	default ARCH_MICROCHIP_POLARFIRE
+	depends on ARCH_MICROCHIP || COMPILE_TEST
+	default y
 	depends on MFD_SYSCON
 	select AUXILIARY_BUS
 	select COMMON_CLK_DIVIDER_REGMAP
-- 
2.51.0
Re: [PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE
Posted by Claudiu Beznea 2 months ago

On 11/21/25 15:44, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
> 
> This driver is used by non-polarfire devices now, and the ARCH_MICROCHIP
> symbol has been defined for some time on RISCV so drop it without any
> functional change.
> 
> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
> ---
>  drivers/clk/microchip/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
> index cab9a909893b..a0ef14310417 100644
> --- a/drivers/clk/microchip/Kconfig
> +++ b/drivers/clk/microchip/Kconfig
> @@ -5,8 +5,8 @@ config COMMON_CLK_PIC32
>  
>  config MCHP_CLK_MPFS
>  	bool "Clk driver for PolarFire SoC"
> -	depends on ARCH_MICROCHIP_POLARFIRE || COMPILE_TEST
> -	default ARCH_MICROCHIP_POLARFIRE
> +	depends on ARCH_MICROCHIP || COMPILE_TEST
> +	default y
>  	depends on MFD_SYSCON
>  	select AUXILIARY_BUS
>  	select COMMON_CLK_DIVIDER_REGMAP

OK, I found v2 in my inbox. Same symptom here. It doesn't apply on top of
the current at91-next either.

Thank you,
Claudiu
Re: [PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE
Posted by Conor Dooley 2 months ago
On Sat, Dec 06, 2025 at 01:18:30PM +0200, Claudiu Beznea wrote:
> 
> 
> On 11/21/25 15:44, Conor Dooley wrote:
> > From: Conor Dooley <conor.dooley@microchip.com>
> > 
> > This driver is used by non-polarfire devices now, and the ARCH_MICROCHIP
> > symbol has been defined for some time on RISCV so drop it without any
> > functional change.
> > 
> > Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
> > ---
> >  drivers/clk/microchip/Kconfig | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
> > index cab9a909893b..a0ef14310417 100644
> > --- a/drivers/clk/microchip/Kconfig
> > +++ b/drivers/clk/microchip/Kconfig
> > @@ -5,8 +5,8 @@ config COMMON_CLK_PIC32
> >  
> >  config MCHP_CLK_MPFS
> >  	bool "Clk driver for PolarFire SoC"
> > -	depends on ARCH_MICROCHIP_POLARFIRE || COMPILE_TEST
> > -	default ARCH_MICROCHIP_POLARFIRE
> > +	depends on ARCH_MICROCHIP || COMPILE_TEST
> > +	default y
> >  	depends on MFD_SYSCON
> >  	select AUXILIARY_BUS
> >  	select COMMON_CLK_DIVIDER_REGMAP
> 
> OK, I found v2 in my inbox. Same symptom here. It doesn't apply on top of
> the current at91-next either.

I think this should sort itself out after -rc1, but I'll resend if it
doesn't.
Re: [PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE
Posted by claudiu beznea 1 month ago

On 12/8/25 20:02, Conor Dooley wrote:
> On Sat, Dec 06, 2025 at 01:18:30PM +0200, Claudiu Beznea wrote:
>>
>>
>> On 11/21/25 15:44, Conor Dooley wrote:
>>> From: Conor Dooley <conor.dooley@microchip.com>
>>>
>>> This driver is used by non-polarfire devices now, and the ARCH_MICROCHIP
>>> symbol has been defined for some time on RISCV so drop it without any
>>> functional change.
>>>
>>> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
>>> ---
>>>   drivers/clk/microchip/Kconfig | 4 ++--
>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
>>> index cab9a909893b..a0ef14310417 100644
>>> --- a/drivers/clk/microchip/Kconfig
>>> +++ b/drivers/clk/microchip/Kconfig
>>> @@ -5,8 +5,8 @@ config COMMON_CLK_PIC32
>>>   
>>>   config MCHP_CLK_MPFS
>>>   	bool "Clk driver for PolarFire SoC"
>>> -	depends on ARCH_MICROCHIP_POLARFIRE || COMPILE_TEST
>>> -	default ARCH_MICROCHIP_POLARFIRE
>>> +	depends on ARCH_MICROCHIP || COMPILE_TEST
>>> +	default y
>>>   	depends on MFD_SYSCON
>>>   	select AUXILIARY_BUS
>>>   	select COMMON_CLK_DIVIDER_REGMAP
>>
>> OK, I found v2 in my inbox. Same symptom here. It doesn't apply on top of
>> the current at91-next either.
> 
> I think this should sort itself out after -rc1, but I'll resend if it
> doesn't.

Still doesn't apply. It conflicts at least with
commit c6f2dddfa7f9 ("clk: microchip: mpfs: use regmap for clocks")

Thank you,
Claudiu
Re: [PATCH v2 1/3] clk: microchip: drop POLARFIRE from ARCH_MICROCHIP_POLARFIRE
Posted by Conor Dooley 1 month ago
On Fri, Jan 09, 2026 at 09:22:25AM +0200, claudiu beznea wrote:
> 
> 
> On 12/8/25 20:02, Conor Dooley wrote:
> > On Sat, Dec 06, 2025 at 01:18:30PM +0200, Claudiu Beznea wrote:
> > > 
> > > 
> > > On 11/21/25 15:44, Conor Dooley wrote:
> > > > From: Conor Dooley <conor.dooley@microchip.com>
> > > > 
> > > > This driver is used by non-polarfire devices now, and the ARCH_MICROCHIP
> > > > symbol has been defined for some time on RISCV so drop it without any
> > > > functional change.
> > > > 
> > > > Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
> > > > ---
> > > >   drivers/clk/microchip/Kconfig | 4 ++--
> > > >   1 file changed, 2 insertions(+), 2 deletions(-)
> > > > 
> > > > diff --git a/drivers/clk/microchip/Kconfig b/drivers/clk/microchip/Kconfig
> > > > index cab9a909893b..a0ef14310417 100644
> > > > --- a/drivers/clk/microchip/Kconfig
> > > > +++ b/drivers/clk/microchip/Kconfig
> > > > @@ -5,8 +5,8 @@ config COMMON_CLK_PIC32
> > > >   config MCHP_CLK_MPFS
> > > >   	bool "Clk driver for PolarFire SoC"
> > > > -	depends on ARCH_MICROCHIP_POLARFIRE || COMPILE_TEST
> > > > -	default ARCH_MICROCHIP_POLARFIRE
> > > > +	depends on ARCH_MICROCHIP || COMPILE_TEST
> > > > +	default y
> > > >   	depends on MFD_SYSCON
> > > >   	select AUXILIARY_BUS
> > > >   	select COMMON_CLK_DIVIDER_REGMAP
> > > 
> > > OK, I found v2 in my inbox. Same symptom here. It doesn't apply on top of
> > > the current at91-next either.
> > 
> > I think this should sort itself out after -rc1, but I'll resend if it
> > doesn't.
> 
> Still doesn't apply. It conflicts at least with
> commit c6f2dddfa7f9 ("clk: microchip: mpfs: use regmap for clocks")

Right, I'll resend. Thought it was based on the aforementioned patch
tbh!