2020-05-05 15:10:41

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 00/15] ARM: Drop unneeded select of multi-platform selected options

Hi arm-soc folks,

This patch series drops select statements from the various
platform-specific Kconfig files, for symbols that are already selected
by the various multi-platform related config options
(ARCH_MULTIPLATFORM, ARCH_MULTI_V*, and ARM_SINGLE_ARMV7M).
This makes it easier to e.g. identify platforms that are not yet part of
multi-platform builds, but already use some multi-platform features
(e.g. "COMMON_CLK" is used by multi-platform + s3c24xx).

This series contains the patches from [1] and [2] that haven't been
applied yet.

Changes compared to v1:
- Add Acked-by, Reviewed-by.

Thanks for applying!

[1] "[PATCH] ARM: arch timer: Drop unneeded select GENERIC_CLOCKEVENTS"
https://lore.kernel.org/r/[email protected]
[2] "[PATCH 00/20] ARM: Drop unneeded select of multi-platform selected
options"
https://lore.kernel.org/r/[email protected]

Geert Uytterhoeven (15):
ARM: arch timer: Drop unneeded select GENERIC_CLOCKEVENTS
ARM: actions: Drop unneeded select of COMMON_CLK
ARM: alpine: Drop unneeded select of HAVE_SMP
ARM: asm9260: Drop unneeded select of GENERIC_CLOCKEVENTS
ARM: aspeed: Drop unneeded select of HAVE_SMP
ARM: berlin: Drop unneeded select of HAVE_SMP
ARM: clps711x: Drop unneeded select of multi-platform selected options
ARM: davinci: Drop unneeded select of TIMER_OF
ARM: integrator: Drop unneeded select of SPARSE_IRQ
ARM: mmp: Drop unneeded select of COMMON_CLK
ARM: mvebu: Drop unneeded select of HAVE_SMP
ARM: omap2plus: Drop unneeded select of MIGHT_HAVE_CACHE_L2X0
ARM: prima2: Drop unneeded select of HAVE_SMP
ARM: realview: Drop unneeded select of multi-platform features
ARM: socfpga: Drop unneeded select of PCI_DOMAINS_GENERIC

arch/arm/Kconfig | 1 -
arch/arm/mach-actions/Kconfig | 1 -
arch/arm/mach-alpine/Kconfig | 1 -
arch/arm/mach-asm9260/Kconfig | 1 -
arch/arm/mach-aspeed/Kconfig | 1 -
arch/arm/mach-berlin/Kconfig | 1 -
arch/arm/mach-clps711x/Kconfig | 5 -----
arch/arm/mach-davinci/Kconfig | 1 -
arch/arm/mach-integrator/Kconfig | 1 -
arch/arm/mach-mmp/Kconfig | 1 -
arch/arm/mach-mvebu/Kconfig | 3 ---
arch/arm/mach-omap2/Kconfig | 1 -
arch/arm/mach-prima2/Kconfig | 1 -
arch/arm/mach-realview/Kconfig | 8 --------
arch/arm/mach-socfpga/Kconfig | 1 -
15 files changed, 28 deletions(-)

--
2.17.1

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


2020-05-05 15:10:55

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 11/15] ARM: mvebu: Drop unneeded select of HAVE_SMP

Support for Marvell Armada 375, 380, 385, and 39x SoCs depends on
ARCH_MULTI_V7.
As the latter selects HAVE_SMP, there is no need for MACH_ARMADA_375,
MACH_ARMADA_38X, and MACH_ARMADA_39X to select HAVE_SMP.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Jason Cooper <[email protected]>
Cc: Andrew Lunn <[email protected]>
Cc: Gregory Clement <[email protected]>
Cc: Sebastian Hesselbarth <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-mvebu/Kconfig | 3 ---
1 file changed, 3 deletions(-)

diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
index 7a5629b9bede4e3f..34dbeaab94b07e52 100644
--- a/arch/arm/mach-mvebu/Kconfig
+++ b/arch/arm/mach-mvebu/Kconfig
@@ -47,7 +47,6 @@ config MACH_ARMADA_375
select ARMADA_375_CLK
select HAVE_ARM_SCU
select HAVE_ARM_TWD if SMP
- select HAVE_SMP
select MACH_MVEBU_V7
select PINCTRL_ARMADA_375
help
@@ -66,7 +65,6 @@ config MACH_ARMADA_38X
select ARMADA_38X_CLK
select HAVE_ARM_SCU
select HAVE_ARM_TWD if SMP
- select HAVE_SMP
select MACH_MVEBU_V7
select PINCTRL_ARMADA_38X
help
@@ -82,7 +80,6 @@ config MACH_ARMADA_39X
select CACHE_L2X0
select HAVE_ARM_SCU
select HAVE_ARM_TWD if SMP
- select HAVE_SMP
select MACH_MVEBU_V7
select PINCTRL_ARMADA_39X
help
--
2.17.1

2020-05-05 15:11:07

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 06/15] ARM: berlin: Drop unneeded select of HAVE_SMP

Support for Marvell Berlin SoCs depends on ARCH_MULTI_V7.
As the latter selects HAVE_SMP, there is no need for MACH_BERLIN_BG2 to
select HAVE_SMP.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Jisheng Zhang <[email protected]>
Cc: Sebastian Hesselbarth <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-berlin/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-berlin/Kconfig b/arch/arm/mach-berlin/Kconfig
index 5b1f61fd78780300..01861fa72c9714b7 100644
--- a/arch/arm/mach-berlin/Kconfig
+++ b/arch/arm/mach-berlin/Kconfig
@@ -19,7 +19,6 @@ config MACH_BERLIN_BG2
select CPU_PJ4B
select HAVE_ARM_SCU if SMP
select HAVE_ARM_TWD if SMP
- select HAVE_SMP
select PINCTRL_BERLIN_BG2

config MACH_BERLIN_BG2CD
--
2.17.1

2020-05-05 15:11:13

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 10/15] ARM: mmp: Drop unneeded select of COMMON_CLK

Support for Marvell MMP ARMv5 platforms depends on ARCH_MULTI_V5, and
thus on ARCH_MULTIPLATFORM.
As the latter selects COMMON_CLK, there is no need for MACH_MMP_DT to
select COMMON_CLK.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Lubomir Rintel <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
Acked-by: Lubomir Rintel <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-mmp/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-mmp/Kconfig b/arch/arm/mach-mmp/Kconfig
index b58a03b18bdef14c..6fe1550f43ec6aef 100644
--- a/arch/arm/mach-mmp/Kconfig
+++ b/arch/arm/mach-mmp/Kconfig
@@ -110,7 +110,6 @@ config MACH_MMP_DT
depends on ARCH_MULTI_V5
select PINCTRL
select PINCTRL_SINGLE
- select COMMON_CLK
select ARCH_HAS_RESET_CONTROLLER
select CPU_MOHAWK
help
--
2.17.1

2020-05-05 15:14:09

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 04/15] ARM: asm9260: Drop unneeded select of GENERIC_CLOCKEVENTS

Support for the Alphascale ASM9260 platform depends on ARCH_MULTI_V5,
and thus on ARCH_MULTIPLATFORM.
As the latter selects GENERIC_CLOCKEVENTS, there is no need for
MACH_ASM9260 to select GENERIC_CLOCKEVENTS.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Oleksij Rempel <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-asm9260/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-asm9260/Kconfig b/arch/arm/mach-asm9260/Kconfig
index e42dbaa53bc61b20..a2e1d0aaf2529aa5 100644
--- a/arch/arm/mach-asm9260/Kconfig
+++ b/arch/arm/mach-asm9260/Kconfig
@@ -4,6 +4,5 @@ config MACH_ASM9260
depends on ARCH_MULTI_V5
select CPU_ARM926T
select ASM9260_TIMER
- select GENERIC_CLOCKEVENTS
help
Support for Alphascale ASM9260 based platform.
--
2.17.1

2020-05-05 15:14:10

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 12/15] ARM: omap2plus: Drop unneeded select of MIGHT_HAVE_CACHE_L2X0

Support for TI AM43x SoCs depends on ARCH_MULTI_V7, which selects
ARCH_MULTI_V6_V7.
As the latter selects MIGHT_HAVE_CACHE_L2X0, there is no need for
SOC_AM43XX to select MIGHT_HAVE_CACHE_L2X0.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Acked-by: Arnd Bergmann <[email protected]>
Acked-by: Tony Lindgren <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-omap2/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dca7d06c0b938619..ea23205bf70f4df6 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -66,7 +66,6 @@ config SOC_AM43XX
select ARCH_OMAP2PLUS
select ARM_GIC
select MACH_OMAP_GENERIC
- select MIGHT_HAVE_CACHE_L2X0
select HAVE_ARM_SCU
select GENERIC_CLOCKEVENTS_BROADCAST
select HAVE_ARM_TWD
--
2.17.1

2020-05-05 15:14:10

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH v2 15/15] ARM: socfpga: Drop unneeded select of PCI_DOMAINS_GENERIC

Support for Altera SOCFPGA systems depends on ARCH_MULTI_V7, and thus on
ARCH_MULTIPLATFORM.
As the latter selects PCI_DOMAINS_GENERIC, there is no need for
ARCH_SOCFPGA to select PCI_DOMAINS_GENERIC.

Signed-off-by: Geert Uytterhoeven <[email protected]>
Cc: Dinh Nguyen <[email protected]>
Acked-by: Arnd Bergmann <[email protected]>
Acked-by: Dinh Nguyen <[email protected]>
---
v2:
- Add Acked-by.
---
arch/arm/mach-socfpga/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-socfpga/Kconfig b/arch/arm/mach-socfpga/Kconfig
index 22af5e308db6cc81..c3bb68d57cea2e51 100644
--- a/arch/arm/mach-socfpga/Kconfig
+++ b/arch/arm/mach-socfpga/Kconfig
@@ -11,7 +11,6 @@ menuconfig ARCH_SOCFPGA
select HAVE_ARM_SCU
select HAVE_ARM_TWD if SMP
select MFD_SYSCON
- select PCI_DOMAINS_GENERIC if PCI
select ARM_ERRATA_754322
select ARM_ERRATA_764369 if SMP
select ARM_ERRATA_775420
--
2.17.1

2020-05-05 15:15:50

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH v2 11/15] ARM: mvebu: Drop unneeded select of HAVE_SMP

On Tue, May 05, 2020 at 05:07:18PM +0200, Geert Uytterhoeven wrote:
> Support for Marvell Armada 375, 380, 385, and 39x SoCs depends on
> ARCH_MULTI_V7.
> As the latter selects HAVE_SMP, there is no need for MACH_ARMADA_375,
> MACH_ARMADA_38X, and MACH_ARMADA_39X to select HAVE_SMP.
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> Cc: Jason Cooper <[email protected]>
> Cc: Andrew Lunn <[email protected]>
> Cc: Gregory Clement <[email protected]>
> Cc: Sebastian Hesselbarth <[email protected]>
> Acked-by: Arnd Bergmann <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2020-05-05 18:30:31

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH v2 12/15] ARM: omap2plus: Drop unneeded select of MIGHT_HAVE_CACHE_L2X0

* Geert Uytterhoeven <[email protected]> [200505 08:08]:
> Support for TI AM43x SoCs depends on ARCH_MULTI_V7, which selects
> ARCH_MULTI_V6_V7.
> As the latter selects MIGHT_HAVE_CACHE_L2X0, there is no need for
> SOC_AM43XX to select MIGHT_HAVE_CACHE_L2X0.
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Acked-by: Arnd Bergmann <[email protected]>
> Acked-by: Tony Lindgren <[email protected]>
> ---
> v2:
> - Add Acked-by.

I'll queue this into omap-for-v5.8/soc tomorrow assuming nobody else
has it already applied.

Tony

> ---
> arch/arm/mach-omap2/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index dca7d06c0b938619..ea23205bf70f4df6 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -66,7 +66,6 @@ config SOC_AM43XX
> select ARCH_OMAP2PLUS
> select ARM_GIC
> select MACH_OMAP_GENERIC
> - select MIGHT_HAVE_CACHE_L2X0
> select HAVE_ARM_SCU
> select GENERIC_CLOCKEVENTS_BROADCAST
> select HAVE_ARM_TWD
> --
> 2.17.1
>

2020-05-07 17:53:09

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH v2 12/15] ARM: omap2plus: Drop unneeded select of MIGHT_HAVE_CACHE_L2X0

* Tony Lindgren <[email protected]> [200505 18:27]:
> * Geert Uytterhoeven <[email protected]> [200505 08:08]:
> > Support for TI AM43x SoCs depends on ARCH_MULTI_V7, which selects
> > ARCH_MULTI_V6_V7.
> > As the latter selects MIGHT_HAVE_CACHE_L2X0, there is no need for
> > SOC_AM43XX to select MIGHT_HAVE_CACHE_L2X0.
> >
> > Signed-off-by: Geert Uytterhoeven <[email protected]>
> > Cc: Tony Lindgren <[email protected]>
> > Cc: [email protected]
> > Acked-by: Arnd Bergmann <[email protected]>
> > Acked-by: Tony Lindgren <[email protected]>
> > ---
> > v2:
> > - Add Acked-by.
>
> I'll queue this into omap-for-v5.8/soc tomorrow assuming nobody else
> has it already applied.

OK applying int omap-for-v5.8/soc thanks.

Tony

2020-05-22 10:15:59

by Jisheng Zhang

[permalink] [raw]
Subject: Re: [PATCH v2 06/15] ARM: berlin: Drop unneeded select of HAVE_SMP

Hi Arnd, Kevin, Olof,

On Tue, 5 May 2020 17:07:13 +0200 Geert Uytterhoeven wrote:

>
>
> Support for Marvell Berlin SoCs depends on ARCH_MULTI_V7.
> As the latter selects HAVE_SMP, there is no need for MACH_BERLIN_BG2 to
> select HAVE_SMP.
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> Cc: Jisheng Zhang <[email protected]>
> Cc: Sebastian Hesselbarth <[email protected]>
> Acked-by: Arnd Bergmann <[email protected]>

The patch looks good to me. I want to know what will be the mainline
path of this series. SoC maintainers take it then send A PR to arm-soc?
Or each SoC maintainers ack it, arm-soc will take the whole series?
If later, then

Acked-by: Jisheng Zhang <[email protected]>


This is the first time I see a series touch different SoC platforms.

Thanks in advance,
Jisheng

> ---
> v2:
> - Add Acked-by.
> ---
> arch/arm/mach-berlin/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/mach-berlin/Kconfig b/arch/arm/mach-berlin/Kconfig
> index 5b1f61fd78780300..01861fa72c9714b7 100644
> --- a/arch/arm/mach-berlin/Kconfig
> +++ b/arch/arm/mach-berlin/Kconfig
> @@ -19,7 +19,6 @@ config MACH_BERLIN_BG2
> select CPU_PJ4B
> select HAVE_ARM_SCU if SMP
> select HAVE_ARM_TWD if SMP
> - select HAVE_SMP
> select PINCTRL_BERLIN_BG2
>
> config MACH_BERLIN_BG2CD
> --
> 2.17.1
>

2020-05-22 10:37:10

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH v2 06/15] ARM: berlin: Drop unneeded select of HAVE_SMP

On Fri, May 22, 2020 at 12:13 PM Jisheng Zhang
<[email protected]> wrote:
>
> Hi Arnd, Kevin, Olof,
>
> On Tue, 5 May 2020 17:07:13 +0200 Geert Uytterhoeven wrote:
>
> >
> >
> > Support for Marvell Berlin SoCs depends on ARCH_MULTI_V7.
> > As the latter selects HAVE_SMP, there is no need for MACH_BERLIN_BG2 to
> > select HAVE_SMP.
> >
> > Signed-off-by: Geert Uytterhoeven <[email protected]>
> > Cc: Jisheng Zhang <[email protected]>
> > Cc: Sebastian Hesselbarth <[email protected]>
> > Acked-by: Arnd Bergmann <[email protected]>
>
> The patch looks good to me. I want to know what will be the mainline
> path of this series. SoC maintainers take it then send A PR to arm-soc?
> Or each SoC maintainers ack it, arm-soc will take the whole series?
> If later, then
>
> Acked-by: Jisheng Zhang <[email protected]>
>
>
> This is the first time I see a series touch different SoC platforms.

I have already merged it. The normal way we do this is that platform
maintainers can choose to merge individual patches when they
are happy with them on the early review, or provide an Ack for
them to get merged as a branch.

I picked up v2 of the series as there seemed to be a sufficient
number of Acks and everyone that commented had agreed
in principle.

Arnd

2020-05-22 10:39:28

by Jisheng Zhang

[permalink] [raw]
Subject: Re: [PATCH v2 06/15] ARM: berlin: Drop unneeded select of HAVE_SMP

On Fri, 22 May 2020 12:34:43 +0200 Arnd Bergmann wrote:


>
>
> On Fri, May 22, 2020 at 12:13 PM Jisheng Zhang
> <[email protected]> wrote:
> >
> > Hi Arnd, Kevin, Olof,
> >
> > On Tue, 5 May 2020 17:07:13 +0200 Geert Uytterhoeven wrote:
> >
> > >
> > >
> > > Support for Marvell Berlin SoCs depends on ARCH_MULTI_V7.
> > > As the latter selects HAVE_SMP, there is no need for MACH_BERLIN_BG2 to
> > > select HAVE_SMP.
> > >
> > > Signed-off-by: Geert Uytterhoeven <[email protected]>
> > > Cc: Jisheng Zhang <[email protected]>
> > > Cc: Sebastian Hesselbarth <[email protected]>
> > > Acked-by: Arnd Bergmann <[email protected]>
> >
> > The patch looks good to me. I want to know what will be the mainline
> > path of this series. SoC maintainers take it then send A PR to arm-soc?
> > Or each SoC maintainers ack it, arm-soc will take the whole series?
> > If later, then
> >
> > Acked-by: Jisheng Zhang <[email protected]>
> >
> >
> > This is the first time I see a series touch different SoC platforms.
>
> I have already merged it. The normal way we do this is that platform
> maintainers can choose to merge individual patches when they
> are happy with them on the early review, or provide an Ack for
> them to get merged as a branch.
>
> I picked up v2 of the series as there seemed to be a sufficient
> number of Acks and everyone that commented had agreed
> in principle.
>

Nice. Thanks a lot