* [PATCH/RFC 0/2] arm: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-11 8:55 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: Arnd Bergmann, Simon Horman, Magnus Damm
Cc: Catalin Marinas, Will Deacon, Olof Johansson, Kevin Hilman,
linux-renesas-soc, linux-arm-kernel, Geert Uytterhoeven
Hi all,
As pointed out by Arnd[*], there are lots of SoC-specific Kconfig
symbols in the Renesas section of arch/arm64/Kconfig.platforms, while
other vendors expose a single Kconfig symbol there.
Hence this patch series moves them to drivers/soc/renesas/Kconfig,
reducing the footprint and associated amount of churn in
arch/arm64/Kconfig.platforms, while preserving the benefits for users of
automatic selection of e.g. clock and pinctrl drivers.
As my OCD just loves consistency, the second patch does the same for
arm32 SoCs.
Note that e.g. drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y and ARCH_RENESAS=n, questions will be asked about clock
drivers before they can be auto-selected by the corresponding
SoC-specific Kconfig symbols.
Of course this will trigger some defconfig updates later...
Thanks for your comments!
[*] https://www.spinics.net/lists/arm-kernel/msg679390.html
Geert Uytterhoeven (2):
[RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
[RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
arch/arm/mach-shmobile/Kconfig | 101 -----------------
arch/arm64/Kconfig.platforms | 54 ---------
drivers/soc/renesas/Kconfig | 201 +++++++++++++++++++++++++++++----
3 files changed, 180 insertions(+), 176 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH/RFC 0/2] arm: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-11 8:55 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: linux-arm-kernel
Hi all,
As pointed out by Arnd[*], there are lots of SoC-specific Kconfig
symbols in the Renesas section of arch/arm64/Kconfig.platforms, while
other vendors expose a single Kconfig symbol there.
Hence this patch series moves them to drivers/soc/renesas/Kconfig,
reducing the footprint and associated amount of churn in
arch/arm64/Kconfig.platforms, while preserving the benefits for users of
automatic selection of e.g. clock and pinctrl drivers.
As my OCD just loves consistency, the second patch does the same for
arm32 SoCs.
Note that e.g. drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y and ARCH_RENESAS=n, questions will be asked about clock
drivers before they can be auto-selected by the corresponding
SoC-specific Kconfig symbols.
Of course this will trigger some defconfig updates later...
Thanks for your comments!
[*] https://www.spinics.net/lists/arm-kernel/msg679390.html
Geert Uytterhoeven (2):
[RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
[RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
arch/arm/mach-shmobile/Kconfig | 101 -----------------
arch/arm64/Kconfig.platforms | 54 ---------
drivers/soc/renesas/Kconfig | 201 +++++++++++++++++++++++++++++----
3 files changed, 180 insertions(+), 176 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-11 8:55 ` Geert Uytterhoeven
@ 2018-10-11 8:55 ` Geert Uytterhoeven
-1 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: Arnd Bergmann, Simon Horman, Magnus Damm
Cc: Catalin Marinas, Will Deacon, Olof Johansson, Kevin Hilman,
linux-renesas-soc, linux-arm-kernel, Geert Uytterhoeven
For consistency with other vendors, which have a single Kconfig symbol
in arch/arm64/Kconfig.platforms.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Note that drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y, questions will be asked about clock drivers before they
can be auto-selected by SoC support.
Question: Should we introduce a family-specific Kconfig symbol for R-Car
Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
RST_RCAR?
---
arch/arm64/Kconfig.platforms | 54 ------------------------
drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
2 files changed, 68 insertions(+), 67 deletions(-)
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index fecdfc70828c2ac3..e7f1659064f82790 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -165,60 +165,6 @@ config ARCH_RENESAS
help
This enables support for the ARMv8 based Renesas SoCs.
-config ARCH_R8A774A1
- bool "Renesas RZ/G2M SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas RZ/G2M SoC.
-
-config ARCH_R8A774C0
- bool "Renesas RZ/G2E SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas RZ/G2E SoC.
-
-config ARCH_R8A7795
- bool "Renesas R-Car H3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car H3 SoC.
-
-config ARCH_R8A7796
- bool "Renesas R-Car M3-W SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car M3-W SoC.
-
-config ARCH_R8A77965
- bool "Renesas R-Car M3-N SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car M3-N SoC.
-
-config ARCH_R8A77970
- bool "Renesas R-Car V3M SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car V3M SoC.
-
-config ARCH_R8A77980
- bool "Renesas R-Car V3H SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car V3H SoC.
-
-config ARCH_R8A77990
- bool "Renesas R-Car E3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car E3 SoC.
-
-config ARCH_R8A77995
- bool "Renesas R-Car D3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car D3 SoC.
-
config ARCH_ROCKCHIP
bool "Rockchip Platforms"
select ARCH_HAS_RESET_CONTROLLER
diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 407f02c80e8b721c..0ab62024fd20be56 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -3,30 +3,85 @@ config SOC_RENESAS
bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
default y if ARCH_RENESAS
select SOC_BUS
- select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
- ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
- ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
- ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
+ select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
select SYSC_R8A7745 if ARCH_R8A7745
select SYSC_R8A77470 if ARCH_R8A77470
- select SYSC_R8A774A1 if ARCH_R8A774A1
- select SYSC_R8A774C0 if ARCH_R8A774C0
select SYSC_R8A7779 if ARCH_R8A7779
select SYSC_R8A7790 if ARCH_R8A7790
select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
select SYSC_R8A7792 if ARCH_R8A7792
select SYSC_R8A7794 if ARCH_R8A7794
- select SYSC_R8A7795 if ARCH_R8A7795
- select SYSC_R8A7796 if ARCH_R8A7796
- select SYSC_R8A77965 if ARCH_R8A77965
- select SYSC_R8A77970 if ARCH_R8A77970
- select SYSC_R8A77980 if ARCH_R8A77980
- select SYSC_R8A77990 if ARCH_R8A77990
- select SYSC_R8A77995 if ARCH_R8A77995
if SOC_RENESAS
+if ARM64
+
+config ARCH_R8A774A1
+ bool "Renesas RZ/G2M SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A774A1
+ help
+ This enables support for the Renesas RZ/G2M SoC.
+
+config ARCH_R8A774C0
+ bool "Renesas RZ/G2E SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A774C0
+ help
+ This enables support for the Renesas RZ/G2E SoC.
+
+config ARCH_R8A7795
+ bool "Renesas R-Car H3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A7795
+ help
+ This enables support for the Renesas R-Car H3 SoC.
+
+config ARCH_R8A7796
+ bool "Renesas R-Car M3-W SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A7796
+ help
+ This enables support for the Renesas R-Car M3-W SoC.
+
+config ARCH_R8A77965
+ bool "Renesas R-Car M3-N SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77965
+ help
+ This enables support for the Renesas R-Car M3-N SoC.
+
+config ARCH_R8A77970
+ bool "Renesas R-Car V3M SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77970
+ help
+ This enables support for the Renesas R-Car V3M SoC.
+
+config ARCH_R8A77980
+ bool "Renesas R-Car V3H SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77980
+ help
+ This enables support for the Renesas R-Car V3H SoC.
+
+config ARCH_R8A77990
+ bool "Renesas R-Car E3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77990
+ help
+ This enables support for the Renesas R-Car E3 SoC.
+
+config ARCH_R8A77995
+ bool "Renesas R-Car D3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77995
+ help
+ This enables support for the Renesas R-Car D3 SoC.
+
+endif # ARM64
+
# SoC
config SYSC_R8A7743
bool "RZ/G1M System Controller support" if COMPILE_TEST
--
2.17.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-11 8:55 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: linux-arm-kernel
For consistency with other vendors, which have a single Kconfig symbol
in arch/arm64/Kconfig.platforms.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Note that drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y, questions will be asked about clock drivers before they
can be auto-selected by SoC support.
Question: Should we introduce a family-specific Kconfig symbol for R-Car
Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
RST_RCAR?
---
arch/arm64/Kconfig.platforms | 54 ------------------------
drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
2 files changed, 68 insertions(+), 67 deletions(-)
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index fecdfc70828c2ac3..e7f1659064f82790 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -165,60 +165,6 @@ config ARCH_RENESAS
help
This enables support for the ARMv8 based Renesas SoCs.
-config ARCH_R8A774A1
- bool "Renesas RZ/G2M SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas RZ/G2M SoC.
-
-config ARCH_R8A774C0
- bool "Renesas RZ/G2E SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas RZ/G2E SoC.
-
-config ARCH_R8A7795
- bool "Renesas R-Car H3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car H3 SoC.
-
-config ARCH_R8A7796
- bool "Renesas R-Car M3-W SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car M3-W SoC.
-
-config ARCH_R8A77965
- bool "Renesas R-Car M3-N SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car M3-N SoC.
-
-config ARCH_R8A77970
- bool "Renesas R-Car V3M SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car V3M SoC.
-
-config ARCH_R8A77980
- bool "Renesas R-Car V3H SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car V3H SoC.
-
-config ARCH_R8A77990
- bool "Renesas R-Car E3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car E3 SoC.
-
-config ARCH_R8A77995
- bool "Renesas R-Car D3 SoC Platform"
- depends on ARCH_RENESAS
- help
- This enables support for the Renesas R-Car D3 SoC.
-
config ARCH_ROCKCHIP
bool "Rockchip Platforms"
select ARCH_HAS_RESET_CONTROLLER
diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 407f02c80e8b721c..0ab62024fd20be56 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -3,30 +3,85 @@ config SOC_RENESAS
bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
default y if ARCH_RENESAS
select SOC_BUS
- select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
- ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
- ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
- ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
+ select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
select SYSC_R8A7745 if ARCH_R8A7745
select SYSC_R8A77470 if ARCH_R8A77470
- select SYSC_R8A774A1 if ARCH_R8A774A1
- select SYSC_R8A774C0 if ARCH_R8A774C0
select SYSC_R8A7779 if ARCH_R8A7779
select SYSC_R8A7790 if ARCH_R8A7790
select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
select SYSC_R8A7792 if ARCH_R8A7792
select SYSC_R8A7794 if ARCH_R8A7794
- select SYSC_R8A7795 if ARCH_R8A7795
- select SYSC_R8A7796 if ARCH_R8A7796
- select SYSC_R8A77965 if ARCH_R8A77965
- select SYSC_R8A77970 if ARCH_R8A77970
- select SYSC_R8A77980 if ARCH_R8A77980
- select SYSC_R8A77990 if ARCH_R8A77990
- select SYSC_R8A77995 if ARCH_R8A77995
if SOC_RENESAS
+if ARM64
+
+config ARCH_R8A774A1
+ bool "Renesas RZ/G2M SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A774A1
+ help
+ This enables support for the Renesas RZ/G2M SoC.
+
+config ARCH_R8A774C0
+ bool "Renesas RZ/G2E SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A774C0
+ help
+ This enables support for the Renesas RZ/G2E SoC.
+
+config ARCH_R8A7795
+ bool "Renesas R-Car H3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A7795
+ help
+ This enables support for the Renesas R-Car H3 SoC.
+
+config ARCH_R8A7796
+ bool "Renesas R-Car M3-W SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A7796
+ help
+ This enables support for the Renesas R-Car M3-W SoC.
+
+config ARCH_R8A77965
+ bool "Renesas R-Car M3-N SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77965
+ help
+ This enables support for the Renesas R-Car M3-N SoC.
+
+config ARCH_R8A77970
+ bool "Renesas R-Car V3M SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77970
+ help
+ This enables support for the Renesas R-Car V3M SoC.
+
+config ARCH_R8A77980
+ bool "Renesas R-Car V3H SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77980
+ help
+ This enables support for the Renesas R-Car V3H SoC.
+
+config ARCH_R8A77990
+ bool "Renesas R-Car E3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77990
+ help
+ This enables support for the Renesas R-Car E3 SoC.
+
+config ARCH_R8A77995
+ bool "Renesas R-Car D3 SoC Platform"
+ select RST_RCAR
+ select SYSC_R8A77995
+ help
+ This enables support for the Renesas R-Car D3 SoC.
+
+endif # ARM64
+
# SoC
config SYSC_R8A7743
bool "RZ/G1M System Controller support" if COMPILE_TEST
--
2.17.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 2/2] [RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-11 8:55 ` Geert Uytterhoeven
@ 2018-10-11 8:55 ` Geert Uytterhoeven
-1 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: Arnd Bergmann, Simon Horman, Magnus Damm
Cc: Catalin Marinas, Will Deacon, Olof Johansson, Kevin Hilman,
linux-renesas-soc, linux-arm-kernel, Geert Uytterhoeven
For consistency with arm64, where vendors have a single Kconfig symbol
in arch/arm64/Kconfig.platforms.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Note that drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y, questions will be asked about clock drivers before they
can be auto-selected by SoC support.
Question: Should the family-specific Kconfig symbols be moved, too?
Not much would be left in arch/arm/mach-shmobile/Kconfig,
though.
---
arch/arm/mach-shmobile/Kconfig | 101 ---------------------------
drivers/soc/renesas/Kconfig | 120 ++++++++++++++++++++++++++++++---
2 files changed, 112 insertions(+), 109 deletions(-)
diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index b100c26a858f9015..50267ad76f990f11 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -36,104 +36,3 @@ menuconfig ARCH_RENESAS
select PINCTRL
select SOC_BUS
select ZONE_DMA if ARM_LPAE
-
-if ARCH_RENESAS
-
-#comment "Renesas ARM SoCs System Type"
-
-config ARCH_EMEV2
- bool "Emma Mobile EV2"
- select SYS_SUPPORTS_EM_STI
-
-config ARCH_R7S72100
- bool "RZ/A1H (R7S72100)"
- select PM
- select PM_GENERIC_DOMAINS
- select SYS_SUPPORTS_SH_MTU2
- select RENESAS_OSTM
-
-config ARCH_R7S9210
- bool "RZ/A2 (R7S9210)"
- select PM
- select PM_GENERIC_DOMAINS
- select RENESAS_OSTM
-
-config ARCH_R8A73A4
- bool "R-Mobile APE6 (R8A73A40)"
- select ARCH_RMOBILE
- select ARM_ERRATA_798181 if SMP
- select HAVE_ARM_ARCH_TIMER
- select RENESAS_IRQC
-
-config ARCH_R8A7740
- bool "R-Mobile A1 (R8A77400)"
- select ARCH_RMOBILE
- select RENESAS_INTC_IRQPIN
-
-config ARCH_R8A7743
- bool "RZ/G1M (R8A77430)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7744
- bool "RZ/G1N (R8A77440)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7745
- bool "RZ/G1E (R8A77450)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R8A77470
- bool "RZ/G1C (R8A77470)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R8A7778
- bool "R-Car M1A (R8A77781)"
- select ARCH_RCAR_GEN1
-
-config ARCH_R8A7779
- bool "R-Car H1 (R8A77790)"
- select ARCH_RCAR_GEN1
-
-config ARCH_R8A7790
- bool "R-Car H2 (R8A77900)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7791
- bool "R-Car M2-W (R8A77910)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7792
- bool "R-Car V2H (R8A77920)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7793
- bool "R-Car M2-N (R8A7793)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7794
- bool "R-Car E2 (R8A77940)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R9A06G032
- bool "RZ/N1D (R9A06G032)"
- select ARCH_RZN1
-
-config ARCH_RZN1
- bool "RZ/N1 (R9A06G0xx) Family"
- select ARM_AMBA
- select CPU_V7
-
-config ARCH_SH73A0
- bool "SH-Mobile AG5 (R8A73A00)"
- select ARCH_RMOBILE
- select RENESAS_INTC_IRQPIN
-endif
diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 0ab62024fd20be56..5089c65b544909c2 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -4,17 +4,121 @@ config SOC_RENESAS
default y if ARCH_RENESAS
select SOC_BUS
select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
- select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
- select SYSC_R8A7745 if ARCH_R8A7745
- select SYSC_R8A77470 if ARCH_R8A77470
- select SYSC_R8A7779 if ARCH_R8A7779
- select SYSC_R8A7790 if ARCH_R8A7790
- select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
- select SYSC_R8A7792 if ARCH_R8A7792
- select SYSC_R8A7794 if ARCH_R8A7794
if SOC_RENESAS
+if ARM
+
+#comment "Renesas ARM SoCs System Type"
+
+config ARCH_EMEV2
+ bool "Emma Mobile EV2"
+ select SYS_SUPPORTS_EM_STI
+
+config ARCH_R7S72100
+ bool "RZ/A1H (R7S72100)"
+ select PM
+ select PM_GENERIC_DOMAINS
+ select SYS_SUPPORTS_SH_MTU2
+ select RENESAS_OSTM
+
+config ARCH_R7S9210
+ bool "RZ/A2 (R7S9210)"
+ select PM
+ select PM_GENERIC_DOMAINS
+ select RENESAS_OSTM
+
+config ARCH_R8A73A4
+ bool "R-Mobile APE6 (R8A73A40)"
+ select ARCH_RMOBILE
+ select ARM_ERRATA_798181 if SMP
+ select HAVE_ARM_ARCH_TIMER
+ select RENESAS_IRQC
+
+config ARCH_R8A7740
+ bool "R-Mobile A1 (R8A77400)"
+ select ARCH_RMOBILE
+ select RENESAS_INTC_IRQPIN
+
+config ARCH_R8A7743
+ bool "RZ/G1M (R8A77430)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7743
+
+config ARCH_R8A7744
+ bool "RZ/G1N (R8A77440)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7743
+
+config ARCH_R8A7745
+ bool "RZ/G1E (R8A77450)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A7745
+
+config ARCH_R8A77470
+ bool "RZ/G1C (R8A77470)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A77470
+
+config ARCH_R8A7778
+ bool "R-Car M1A (R8A77781)"
+ select ARCH_RCAR_GEN1
+
+config ARCH_R8A7779
+ bool "R-Car H1 (R8A77790)"
+ select ARCH_RCAR_GEN1
+ select SYSC_R8A7779
+
+config ARCH_R8A7790
+ bool "R-Car H2 (R8A77900)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7790
+
+config ARCH_R8A7791
+ bool "R-Car M2-W (R8A77910)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7791
+
+config ARCH_R8A7792
+ bool "R-Car V2H (R8A77920)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7792
+
+config ARCH_R8A7793
+ bool "R-Car M2-N (R8A7793)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7791
+
+config ARCH_R8A7794
+ bool "R-Car E2 (R8A77940)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A7794
+
+config ARCH_R9A06G032
+ bool "RZ/N1D (R9A06G032)"
+ select ARCH_RZN1
+
+config ARCH_RZN1
+ bool "RZ/N1 (R9A06G0xx) Family"
+ select ARM_AMBA
+ select CPU_V7
+
+config ARCH_SH73A0
+ bool "SH-Mobile AG5 (R8A73A00)"
+ select ARCH_RMOBILE
+ select RENESAS_INTC_IRQPIN
+
+endif # ARM
+
if ARM64
config ARCH_R8A774A1
--
2.17.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 2/2] [RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-11 8:55 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-11 8:55 UTC (permalink / raw)
To: linux-arm-kernel
For consistency with arm64, where vendors have a single Kconfig symbol
in arch/arm64/Kconfig.platforms.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Note that drivers/clk/ is included before drivers/soc/. Hence when
COMPILE_TEST=y, questions will be asked about clock drivers before they
can be auto-selected by SoC support.
Question: Should the family-specific Kconfig symbols be moved, too?
Not much would be left in arch/arm/mach-shmobile/Kconfig,
though.
---
arch/arm/mach-shmobile/Kconfig | 101 ---------------------------
drivers/soc/renesas/Kconfig | 120 ++++++++++++++++++++++++++++++---
2 files changed, 112 insertions(+), 109 deletions(-)
diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index b100c26a858f9015..50267ad76f990f11 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -36,104 +36,3 @@ menuconfig ARCH_RENESAS
select PINCTRL
select SOC_BUS
select ZONE_DMA if ARM_LPAE
-
-if ARCH_RENESAS
-
-#comment "Renesas ARM SoCs System Type"
-
-config ARCH_EMEV2
- bool "Emma Mobile EV2"
- select SYS_SUPPORTS_EM_STI
-
-config ARCH_R7S72100
- bool "RZ/A1H (R7S72100)"
- select PM
- select PM_GENERIC_DOMAINS
- select SYS_SUPPORTS_SH_MTU2
- select RENESAS_OSTM
-
-config ARCH_R7S9210
- bool "RZ/A2 (R7S9210)"
- select PM
- select PM_GENERIC_DOMAINS
- select RENESAS_OSTM
-
-config ARCH_R8A73A4
- bool "R-Mobile APE6 (R8A73A40)"
- select ARCH_RMOBILE
- select ARM_ERRATA_798181 if SMP
- select HAVE_ARM_ARCH_TIMER
- select RENESAS_IRQC
-
-config ARCH_R8A7740
- bool "R-Mobile A1 (R8A77400)"
- select ARCH_RMOBILE
- select RENESAS_INTC_IRQPIN
-
-config ARCH_R8A7743
- bool "RZ/G1M (R8A77430)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7744
- bool "RZ/G1N (R8A77440)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7745
- bool "RZ/G1E (R8A77450)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R8A77470
- bool "RZ/G1C (R8A77470)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R8A7778
- bool "R-Car M1A (R8A77781)"
- select ARCH_RCAR_GEN1
-
-config ARCH_R8A7779
- bool "R-Car H1 (R8A77790)"
- select ARCH_RCAR_GEN1
-
-config ARCH_R8A7790
- bool "R-Car H2 (R8A77900)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7791
- bool "R-Car M2-W (R8A77910)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7792
- bool "R-Car V2H (R8A77920)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
-
-config ARCH_R8A7793
- bool "R-Car M2-N (R8A7793)"
- select ARCH_RCAR_GEN2
- select ARM_ERRATA_798181 if SMP
- select I2C
-
-config ARCH_R8A7794
- bool "R-Car E2 (R8A77940)"
- select ARCH_RCAR_GEN2
-
-config ARCH_R9A06G032
- bool "RZ/N1D (R9A06G032)"
- select ARCH_RZN1
-
-config ARCH_RZN1
- bool "RZ/N1 (R9A06G0xx) Family"
- select ARM_AMBA
- select CPU_V7
-
-config ARCH_SH73A0
- bool "SH-Mobile AG5 (R8A73A00)"
- select ARCH_RMOBILE
- select RENESAS_INTC_IRQPIN
-endif
diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 0ab62024fd20be56..5089c65b544909c2 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -4,17 +4,121 @@ config SOC_RENESAS
default y if ARCH_RENESAS
select SOC_BUS
select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
- select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
- select SYSC_R8A7745 if ARCH_R8A7745
- select SYSC_R8A77470 if ARCH_R8A77470
- select SYSC_R8A7779 if ARCH_R8A7779
- select SYSC_R8A7790 if ARCH_R8A7790
- select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
- select SYSC_R8A7792 if ARCH_R8A7792
- select SYSC_R8A7794 if ARCH_R8A7794
if SOC_RENESAS
+if ARM
+
+#comment "Renesas ARM SoCs System Type"
+
+config ARCH_EMEV2
+ bool "Emma Mobile EV2"
+ select SYS_SUPPORTS_EM_STI
+
+config ARCH_R7S72100
+ bool "RZ/A1H (R7S72100)"
+ select PM
+ select PM_GENERIC_DOMAINS
+ select SYS_SUPPORTS_SH_MTU2
+ select RENESAS_OSTM
+
+config ARCH_R7S9210
+ bool "RZ/A2 (R7S9210)"
+ select PM
+ select PM_GENERIC_DOMAINS
+ select RENESAS_OSTM
+
+config ARCH_R8A73A4
+ bool "R-Mobile APE6 (R8A73A40)"
+ select ARCH_RMOBILE
+ select ARM_ERRATA_798181 if SMP
+ select HAVE_ARM_ARCH_TIMER
+ select RENESAS_IRQC
+
+config ARCH_R8A7740
+ bool "R-Mobile A1 (R8A77400)"
+ select ARCH_RMOBILE
+ select RENESAS_INTC_IRQPIN
+
+config ARCH_R8A7743
+ bool "RZ/G1M (R8A77430)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7743
+
+config ARCH_R8A7744
+ bool "RZ/G1N (R8A77440)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7743
+
+config ARCH_R8A7745
+ bool "RZ/G1E (R8A77450)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A7745
+
+config ARCH_R8A77470
+ bool "RZ/G1C (R8A77470)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A77470
+
+config ARCH_R8A7778
+ bool "R-Car M1A (R8A77781)"
+ select ARCH_RCAR_GEN1
+
+config ARCH_R8A7779
+ bool "R-Car H1 (R8A77790)"
+ select ARCH_RCAR_GEN1
+ select SYSC_R8A7779
+
+config ARCH_R8A7790
+ bool "R-Car H2 (R8A77900)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7790
+
+config ARCH_R8A7791
+ bool "R-Car M2-W (R8A77910)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7791
+
+config ARCH_R8A7792
+ bool "R-Car V2H (R8A77920)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select SYSC_R8A7792
+
+config ARCH_R8A7793
+ bool "R-Car M2-N (R8A7793)"
+ select ARCH_RCAR_GEN2
+ select ARM_ERRATA_798181 if SMP
+ select I2C
+ select SYSC_R8A7791
+
+config ARCH_R8A7794
+ bool "R-Car E2 (R8A77940)"
+ select ARCH_RCAR_GEN2
+ select SYSC_R8A7794
+
+config ARCH_R9A06G032
+ bool "RZ/N1D (R9A06G032)"
+ select ARCH_RZN1
+
+config ARCH_RZN1
+ bool "RZ/N1 (R9A06G0xx) Family"
+ select ARM_AMBA
+ select CPU_V7
+
+config ARCH_SH73A0
+ bool "SH-Mobile AG5 (R8A73A00)"
+ select ARCH_RMOBILE
+ select RENESAS_INTC_IRQPIN
+
+endif # ARM
+
if ARM64
config ARCH_R8A774A1
--
2.17.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-11 8:55 ` Geert Uytterhoeven
@ 2018-10-11 13:54 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-11 13:54 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Arnd Bergmann, Magnus Damm, Catalin Marinas, Will Deacon,
Olof Johansson, Kevin Hilman, linux-renesas-soc,
linux-arm-kernel
On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> For consistency with other vendors, which have a single Kconfig symbol
> in arch/arm64/Kconfig.platforms.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Note that drivers/clk/ is included before drivers/soc/. Hence when
> COMPILE_TEST=y, questions will be asked about clock drivers before they
> can be auto-selected by SoC support.
>
> Question: Should we introduce a family-specific Kconfig symbol for R-Car
> Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
s/1/3/?
> RST_RCAR?
Given that it would be consistent with R-Car Gen 1 and 2,
that seems like a good idea to me.
> ---
> arch/arm64/Kconfig.platforms | 54 ------------------------
> drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
> 2 files changed, 68 insertions(+), 67 deletions(-)
>
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index fecdfc70828c2ac3..e7f1659064f82790 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -165,60 +165,6 @@ config ARCH_RENESAS
> help
> This enables support for the ARMv8 based Renesas SoCs.
>
> -config ARCH_R8A774A1
> - bool "Renesas RZ/G2M SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas RZ/G2M SoC.
> -
> -config ARCH_R8A774C0
> - bool "Renesas RZ/G2E SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas RZ/G2E SoC.
> -
> -config ARCH_R8A7795
> - bool "Renesas R-Car H3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car H3 SoC.
> -
> -config ARCH_R8A7796
> - bool "Renesas R-Car M3-W SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car M3-W SoC.
> -
> -config ARCH_R8A77965
> - bool "Renesas R-Car M3-N SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car M3-N SoC.
> -
> -config ARCH_R8A77970
> - bool "Renesas R-Car V3M SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car V3M SoC.
> -
> -config ARCH_R8A77980
> - bool "Renesas R-Car V3H SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car V3H SoC.
> -
> -config ARCH_R8A77990
> - bool "Renesas R-Car E3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car E3 SoC.
> -
> -config ARCH_R8A77995
> - bool "Renesas R-Car D3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car D3 SoC.
> -
> config ARCH_ROCKCHIP
> bool "Rockchip Platforms"
> select ARCH_HAS_RESET_CONTROLLER
> diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> index 407f02c80e8b721c..0ab62024fd20be56 100644
> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -3,30 +3,85 @@ config SOC_RENESAS
> bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
> default y if ARCH_RENESAS
> select SOC_BUS
> - select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
> - ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
> - ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
> - ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
> + select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> select SYSC_R8A7745 if ARCH_R8A7745
> select SYSC_R8A77470 if ARCH_R8A77470
> - select SYSC_R8A774A1 if ARCH_R8A774A1
> - select SYSC_R8A774C0 if ARCH_R8A774C0
> select SYSC_R8A7779 if ARCH_R8A7779
> select SYSC_R8A7790 if ARCH_R8A7790
> select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> select SYSC_R8A7792 if ARCH_R8A7792
> select SYSC_R8A7794 if ARCH_R8A7794
> - select SYSC_R8A7795 if ARCH_R8A7795
> - select SYSC_R8A7796 if ARCH_R8A7796
> - select SYSC_R8A77965 if ARCH_R8A77965
> - select SYSC_R8A77970 if ARCH_R8A77970
> - select SYSC_R8A77980 if ARCH_R8A77980
> - select SYSC_R8A77990 if ARCH_R8A77990
> - select SYSC_R8A77995 if ARCH_R8A77995
>
> if SOC_RENESAS
>
> +if ARM64
> +
> +config ARCH_R8A774A1
> + bool "Renesas RZ/G2M SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A774A1
> + help
> + This enables support for the Renesas RZ/G2M SoC.
> +
> +config ARCH_R8A774C0
> + bool "Renesas RZ/G2E SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A774C0
> + help
> + This enables support for the Renesas RZ/G2E SoC.
> +
> +config ARCH_R8A7795
> + bool "Renesas R-Car H3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A7795
> + help
> + This enables support for the Renesas R-Car H3 SoC.
> +
> +config ARCH_R8A7796
> + bool "Renesas R-Car M3-W SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A7796
> + help
> + This enables support for the Renesas R-Car M3-W SoC.
> +
> +config ARCH_R8A77965
> + bool "Renesas R-Car M3-N SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77965
> + help
> + This enables support for the Renesas R-Car M3-N SoC.
> +
> +config ARCH_R8A77970
> + bool "Renesas R-Car V3M SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77970
> + help
> + This enables support for the Renesas R-Car V3M SoC.
> +
> +config ARCH_R8A77980
> + bool "Renesas R-Car V3H SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77980
> + help
> + This enables support for the Renesas R-Car V3H SoC.
> +
> +config ARCH_R8A77990
> + bool "Renesas R-Car E3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77990
> + help
> + This enables support for the Renesas R-Car E3 SoC.
> +
> +config ARCH_R8A77995
> + bool "Renesas R-Car D3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77995
> + help
> + This enables support for the Renesas R-Car D3 SoC.
> +
> +endif # ARM64
> +
> # SoC
> config SYSC_R8A7743
> bool "RZ/G1M System Controller support" if COMPILE_TEST
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-11 13:54 ` Simon Horman
0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-11 13:54 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> For consistency with other vendors, which have a single Kconfig symbol
> in arch/arm64/Kconfig.platforms.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Note that drivers/clk/ is included before drivers/soc/. Hence when
> COMPILE_TEST=y, questions will be asked about clock drivers before they
> can be auto-selected by SoC support.
>
> Question: Should we introduce a family-specific Kconfig symbol for R-Car
> Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
s/1/3/?
> RST_RCAR?
Given that it would be consistent with R-Car Gen 1 and 2,
that seems like a good idea to me.
> ---
> arch/arm64/Kconfig.platforms | 54 ------------------------
> drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
> 2 files changed, 68 insertions(+), 67 deletions(-)
>
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index fecdfc70828c2ac3..e7f1659064f82790 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -165,60 +165,6 @@ config ARCH_RENESAS
> help
> This enables support for the ARMv8 based Renesas SoCs.
>
> -config ARCH_R8A774A1
> - bool "Renesas RZ/G2M SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas RZ/G2M SoC.
> -
> -config ARCH_R8A774C0
> - bool "Renesas RZ/G2E SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas RZ/G2E SoC.
> -
> -config ARCH_R8A7795
> - bool "Renesas R-Car H3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car H3 SoC.
> -
> -config ARCH_R8A7796
> - bool "Renesas R-Car M3-W SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car M3-W SoC.
> -
> -config ARCH_R8A77965
> - bool "Renesas R-Car M3-N SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car M3-N SoC.
> -
> -config ARCH_R8A77970
> - bool "Renesas R-Car V3M SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car V3M SoC.
> -
> -config ARCH_R8A77980
> - bool "Renesas R-Car V3H SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car V3H SoC.
> -
> -config ARCH_R8A77990
> - bool "Renesas R-Car E3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car E3 SoC.
> -
> -config ARCH_R8A77995
> - bool "Renesas R-Car D3 SoC Platform"
> - depends on ARCH_RENESAS
> - help
> - This enables support for the Renesas R-Car D3 SoC.
> -
> config ARCH_ROCKCHIP
> bool "Rockchip Platforms"
> select ARCH_HAS_RESET_CONTROLLER
> diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> index 407f02c80e8b721c..0ab62024fd20be56 100644
> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -3,30 +3,85 @@ config SOC_RENESAS
> bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
> default y if ARCH_RENESAS
> select SOC_BUS
> - select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
> - ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
> - ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
> - ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
> + select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> select SYSC_R8A7745 if ARCH_R8A7745
> select SYSC_R8A77470 if ARCH_R8A77470
> - select SYSC_R8A774A1 if ARCH_R8A774A1
> - select SYSC_R8A774C0 if ARCH_R8A774C0
> select SYSC_R8A7779 if ARCH_R8A7779
> select SYSC_R8A7790 if ARCH_R8A7790
> select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> select SYSC_R8A7792 if ARCH_R8A7792
> select SYSC_R8A7794 if ARCH_R8A7794
> - select SYSC_R8A7795 if ARCH_R8A7795
> - select SYSC_R8A7796 if ARCH_R8A7796
> - select SYSC_R8A77965 if ARCH_R8A77965
> - select SYSC_R8A77970 if ARCH_R8A77970
> - select SYSC_R8A77980 if ARCH_R8A77980
> - select SYSC_R8A77990 if ARCH_R8A77990
> - select SYSC_R8A77995 if ARCH_R8A77995
>
> if SOC_RENESAS
>
> +if ARM64
> +
> +config ARCH_R8A774A1
> + bool "Renesas RZ/G2M SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A774A1
> + help
> + This enables support for the Renesas RZ/G2M SoC.
> +
> +config ARCH_R8A774C0
> + bool "Renesas RZ/G2E SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A774C0
> + help
> + This enables support for the Renesas RZ/G2E SoC.
> +
> +config ARCH_R8A7795
> + bool "Renesas R-Car H3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A7795
> + help
> + This enables support for the Renesas R-Car H3 SoC.
> +
> +config ARCH_R8A7796
> + bool "Renesas R-Car M3-W SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A7796
> + help
> + This enables support for the Renesas R-Car M3-W SoC.
> +
> +config ARCH_R8A77965
> + bool "Renesas R-Car M3-N SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77965
> + help
> + This enables support for the Renesas R-Car M3-N SoC.
> +
> +config ARCH_R8A77970
> + bool "Renesas R-Car V3M SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77970
> + help
> + This enables support for the Renesas R-Car V3M SoC.
> +
> +config ARCH_R8A77980
> + bool "Renesas R-Car V3H SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77980
> + help
> + This enables support for the Renesas R-Car V3H SoC.
> +
> +config ARCH_R8A77990
> + bool "Renesas R-Car E3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77990
> + help
> + This enables support for the Renesas R-Car E3 SoC.
> +
> +config ARCH_R8A77995
> + bool "Renesas R-Car D3 SoC Platform"
> + select RST_RCAR
> + select SYSC_R8A77995
> + help
> + This enables support for the Renesas R-Car D3 SoC.
> +
> +endif # ARM64
> +
> # SoC
> config SYSC_R8A7743
> bool "RZ/G1M System Controller support" if COMPILE_TEST
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-11 13:54 ` Simon Horman
@ 2018-10-15 15:40 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-15 15:40 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Arnd Bergmann, Magnus Damm, Catalin Marinas, Will Deacon,
Olof Johansson, Kevin Hilman, linux-renesas-soc,
linux-arm-kernel
On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > For consistency with other vendors, which have a single Kconfig symbol
> > in arch/arm64/Kconfig.platforms.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > can be auto-selected by SoC support.
> >
> > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
>
> s/1/3/?
>
> > RST_RCAR?
>
> Given that it would be consistent with R-Car Gen 1 and 2,
> that seems like a good idea to me.
Hi Geert,
there has been no other feedback on this series.
How would you like to proceed?
>
> > ---
> > arch/arm64/Kconfig.platforms | 54 ------------------------
> > drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
> > 2 files changed, 68 insertions(+), 67 deletions(-)
> >
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index fecdfc70828c2ac3..e7f1659064f82790 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -165,60 +165,6 @@ config ARCH_RENESAS
> > help
> > This enables support for the ARMv8 based Renesas SoCs.
> >
> > -config ARCH_R8A774A1
> > - bool "Renesas RZ/G2M SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas RZ/G2M SoC.
> > -
> > -config ARCH_R8A774C0
> > - bool "Renesas RZ/G2E SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas RZ/G2E SoC.
> > -
> > -config ARCH_R8A7795
> > - bool "Renesas R-Car H3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car H3 SoC.
> > -
> > -config ARCH_R8A7796
> > - bool "Renesas R-Car M3-W SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car M3-W SoC.
> > -
> > -config ARCH_R8A77965
> > - bool "Renesas R-Car M3-N SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car M3-N SoC.
> > -
> > -config ARCH_R8A77970
> > - bool "Renesas R-Car V3M SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car V3M SoC.
> > -
> > -config ARCH_R8A77980
> > - bool "Renesas R-Car V3H SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car V3H SoC.
> > -
> > -config ARCH_R8A77990
> > - bool "Renesas R-Car E3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car E3 SoC.
> > -
> > -config ARCH_R8A77995
> > - bool "Renesas R-Car D3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car D3 SoC.
> > -
> > config ARCH_ROCKCHIP
> > bool "Rockchip Platforms"
> > select ARCH_HAS_RESET_CONTROLLER
> > diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> > index 407f02c80e8b721c..0ab62024fd20be56 100644
> > --- a/drivers/soc/renesas/Kconfig
> > +++ b/drivers/soc/renesas/Kconfig
> > @@ -3,30 +3,85 @@ config SOC_RENESAS
> > bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
> > default y if ARCH_RENESAS
> > select SOC_BUS
> > - select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
> > - ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
> > - ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
> > - ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
> > + select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> > select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> > select SYSC_R8A7745 if ARCH_R8A7745
> > select SYSC_R8A77470 if ARCH_R8A77470
> > - select SYSC_R8A774A1 if ARCH_R8A774A1
> > - select SYSC_R8A774C0 if ARCH_R8A774C0
> > select SYSC_R8A7779 if ARCH_R8A7779
> > select SYSC_R8A7790 if ARCH_R8A7790
> > select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> > select SYSC_R8A7792 if ARCH_R8A7792
> > select SYSC_R8A7794 if ARCH_R8A7794
> > - select SYSC_R8A7795 if ARCH_R8A7795
> > - select SYSC_R8A7796 if ARCH_R8A7796
> > - select SYSC_R8A77965 if ARCH_R8A77965
> > - select SYSC_R8A77970 if ARCH_R8A77970
> > - select SYSC_R8A77980 if ARCH_R8A77980
> > - select SYSC_R8A77990 if ARCH_R8A77990
> > - select SYSC_R8A77995 if ARCH_R8A77995
> >
> > if SOC_RENESAS
> >
> > +if ARM64
> > +
> > +config ARCH_R8A774A1
> > + bool "Renesas RZ/G2M SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A774A1
> > + help
> > + This enables support for the Renesas RZ/G2M SoC.
> > +
> > +config ARCH_R8A774C0
> > + bool "Renesas RZ/G2E SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A774C0
> > + help
> > + This enables support for the Renesas RZ/G2E SoC.
> > +
> > +config ARCH_R8A7795
> > + bool "Renesas R-Car H3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A7795
> > + help
> > + This enables support for the Renesas R-Car H3 SoC.
> > +
> > +config ARCH_R8A7796
> > + bool "Renesas R-Car M3-W SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A7796
> > + help
> > + This enables support for the Renesas R-Car M3-W SoC.
> > +
> > +config ARCH_R8A77965
> > + bool "Renesas R-Car M3-N SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77965
> > + help
> > + This enables support for the Renesas R-Car M3-N SoC.
> > +
> > +config ARCH_R8A77970
> > + bool "Renesas R-Car V3M SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77970
> > + help
> > + This enables support for the Renesas R-Car V3M SoC.
> > +
> > +config ARCH_R8A77980
> > + bool "Renesas R-Car V3H SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77980
> > + help
> > + This enables support for the Renesas R-Car V3H SoC.
> > +
> > +config ARCH_R8A77990
> > + bool "Renesas R-Car E3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77990
> > + help
> > + This enables support for the Renesas R-Car E3 SoC.
> > +
> > +config ARCH_R8A77995
> > + bool "Renesas R-Car D3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77995
> > + help
> > + This enables support for the Renesas R-Car D3 SoC.
> > +
> > +endif # ARM64
> > +
> > # SoC
> > config SYSC_R8A7743
> > bool "RZ/G1M System Controller support" if COMPILE_TEST
> > --
> > 2.17.1
> >
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-15 15:40 ` Simon Horman
0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-15 15:40 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > For consistency with other vendors, which have a single Kconfig symbol
> > in arch/arm64/Kconfig.platforms.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > can be auto-selected by SoC support.
> >
> > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
>
> s/1/3/?
>
> > RST_RCAR?
>
> Given that it would be consistent with R-Car Gen 1 and 2,
> that seems like a good idea to me.
Hi Geert,
there has been no other feedback on this series.
How would you like to proceed?
>
> > ---
> > arch/arm64/Kconfig.platforms | 54 ------------------------
> > drivers/soc/renesas/Kconfig | 81 ++++++++++++++++++++++++++++++------
> > 2 files changed, 68 insertions(+), 67 deletions(-)
> >
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index fecdfc70828c2ac3..e7f1659064f82790 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -165,60 +165,6 @@ config ARCH_RENESAS
> > help
> > This enables support for the ARMv8 based Renesas SoCs.
> >
> > -config ARCH_R8A774A1
> > - bool "Renesas RZ/G2M SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas RZ/G2M SoC.
> > -
> > -config ARCH_R8A774C0
> > - bool "Renesas RZ/G2E SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas RZ/G2E SoC.
> > -
> > -config ARCH_R8A7795
> > - bool "Renesas R-Car H3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car H3 SoC.
> > -
> > -config ARCH_R8A7796
> > - bool "Renesas R-Car M3-W SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car M3-W SoC.
> > -
> > -config ARCH_R8A77965
> > - bool "Renesas R-Car M3-N SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car M3-N SoC.
> > -
> > -config ARCH_R8A77970
> > - bool "Renesas R-Car V3M SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car V3M SoC.
> > -
> > -config ARCH_R8A77980
> > - bool "Renesas R-Car V3H SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car V3H SoC.
> > -
> > -config ARCH_R8A77990
> > - bool "Renesas R-Car E3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car E3 SoC.
> > -
> > -config ARCH_R8A77995
> > - bool "Renesas R-Car D3 SoC Platform"
> > - depends on ARCH_RENESAS
> > - help
> > - This enables support for the Renesas R-Car D3 SoC.
> > -
> > config ARCH_ROCKCHIP
> > bool "Rockchip Platforms"
> > select ARCH_HAS_RESET_CONTROLLER
> > diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> > index 407f02c80e8b721c..0ab62024fd20be56 100644
> > --- a/drivers/soc/renesas/Kconfig
> > +++ b/drivers/soc/renesas/Kconfig
> > @@ -3,30 +3,85 @@ config SOC_RENESAS
> > bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
> > default y if ARCH_RENESAS
> > select SOC_BUS
> > - select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2 || \
> > - ARCH_R8A774A1 || ARCH_R8A774C0 || ARCH_R8A7795 || \
> > - ARCH_R8A7796 || ARCH_R8A77965 || ARCH_R8A77970 || \
> > - ARCH_R8A77980 || ARCH_R8A77990 || ARCH_R8A77995
> > + select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> > select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> > select SYSC_R8A7745 if ARCH_R8A7745
> > select SYSC_R8A77470 if ARCH_R8A77470
> > - select SYSC_R8A774A1 if ARCH_R8A774A1
> > - select SYSC_R8A774C0 if ARCH_R8A774C0
> > select SYSC_R8A7779 if ARCH_R8A7779
> > select SYSC_R8A7790 if ARCH_R8A7790
> > select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> > select SYSC_R8A7792 if ARCH_R8A7792
> > select SYSC_R8A7794 if ARCH_R8A7794
> > - select SYSC_R8A7795 if ARCH_R8A7795
> > - select SYSC_R8A7796 if ARCH_R8A7796
> > - select SYSC_R8A77965 if ARCH_R8A77965
> > - select SYSC_R8A77970 if ARCH_R8A77970
> > - select SYSC_R8A77980 if ARCH_R8A77980
> > - select SYSC_R8A77990 if ARCH_R8A77990
> > - select SYSC_R8A77995 if ARCH_R8A77995
> >
> > if SOC_RENESAS
> >
> > +if ARM64
> > +
> > +config ARCH_R8A774A1
> > + bool "Renesas RZ/G2M SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A774A1
> > + help
> > + This enables support for the Renesas RZ/G2M SoC.
> > +
> > +config ARCH_R8A774C0
> > + bool "Renesas RZ/G2E SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A774C0
> > + help
> > + This enables support for the Renesas RZ/G2E SoC.
> > +
> > +config ARCH_R8A7795
> > + bool "Renesas R-Car H3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A7795
> > + help
> > + This enables support for the Renesas R-Car H3 SoC.
> > +
> > +config ARCH_R8A7796
> > + bool "Renesas R-Car M3-W SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A7796
> > + help
> > + This enables support for the Renesas R-Car M3-W SoC.
> > +
> > +config ARCH_R8A77965
> > + bool "Renesas R-Car M3-N SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77965
> > + help
> > + This enables support for the Renesas R-Car M3-N SoC.
> > +
> > +config ARCH_R8A77970
> > + bool "Renesas R-Car V3M SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77970
> > + help
> > + This enables support for the Renesas R-Car V3M SoC.
> > +
> > +config ARCH_R8A77980
> > + bool "Renesas R-Car V3H SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77980
> > + help
> > + This enables support for the Renesas R-Car V3H SoC.
> > +
> > +config ARCH_R8A77990
> > + bool "Renesas R-Car E3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77990
> > + help
> > + This enables support for the Renesas R-Car E3 SoC.
> > +
> > +config ARCH_R8A77995
> > + bool "Renesas R-Car D3 SoC Platform"
> > + select RST_RCAR
> > + select SYSC_R8A77995
> > + help
> > + This enables support for the Renesas R-Car D3 SoC.
> > +
> > +endif # ARM64
> > +
> > # SoC
> > config SYSC_R8A7743
> > bool "RZ/G1M System Controller support" if COMPILE_TEST
> > --
> > 2.17.1
> >
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-15 15:40 ` Simon Horman
@ 2018-10-17 9:56 ` Geert Uytterhoeven
-1 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-17 9:56 UTC (permalink / raw)
To: Simon Horman
Cc: Geert Uytterhoeven, Arnd Bergmann, Magnus Damm, Catalin Marinas,
Will Deacon, Olof Johansson, Kevin Hilman, Linux-Renesas,
Linux ARM
Hi Simon,
On Mon, Oct 15, 2018 at 5:40 PM Simon Horman <horms@verge.net.au> wrote:
> On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> > On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > > For consistency with other vendors, which have a single Kconfig symbol
> > > in arch/arm64/Kconfig.platforms.
> > >
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > ---
> > > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > > can be auto-selected by SoC support.
> > >
> > > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
> >
> > s/1/3/?
Sure.
> > >
> > > RST_RCAR?
> >
> > Given that it would be consistent with R-Car Gen 1 and 2,
> > that seems like a good idea to me.
>
> there has been no other feedback on this series.
> How would you like to proceed?
I will add a new symbol ARCH_RCAR_GEN3 for v2?
Any comments about PATCH 2/2?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-17 9:56 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2018-10-17 9:56 UTC (permalink / raw)
To: linux-arm-kernel
Hi Simon,
On Mon, Oct 15, 2018 at 5:40 PM Simon Horman <horms@verge.net.au> wrote:
> On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> > On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > > For consistency with other vendors, which have a single Kconfig symbol
> > > in arch/arm64/Kconfig.platforms.
> > >
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > ---
> > > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > > can be auto-selected by SoC support.
> > >
> > > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
> >
> > s/1/3/?
Sure.
> > >
> > > RST_RCAR?
> >
> > Given that it would be consistent with R-Car Gen 1 and 2,
> > that seems like a good idea to me.
>
> there has been no other feedback on this series.
> How would you like to proceed?
I will add a new symbol ARCH_RCAR_GEN3 for v2?
Any comments about PATCH 2/2?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
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
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] [RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-11 8:55 ` Geert Uytterhoeven
@ 2018-10-17 10:03 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-17 10:03 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Arnd Bergmann, Magnus Damm, Catalin Marinas, Will Deacon,
Olof Johansson, Kevin Hilman, linux-renesas-soc,
linux-arm-kernel
On Thu, Oct 11, 2018 at 10:55:08AM +0200, Geert Uytterhoeven wrote:
> For consistency with arm64, where vendors have a single Kconfig symbol
> in arch/arm64/Kconfig.platforms.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Note that drivers/clk/ is included before drivers/soc/. Hence when
> COMPILE_TEST=y, questions will be asked about clock drivers before they
> can be auto-selected by SoC support.
>
> Question: Should the family-specific Kconfig symbols be moved, too?
> Not much would be left in arch/arm/mach-shmobile/Kconfig,
> though.
I have no strong opinion either way. But I lean slightly towards
moving the family-specific symbols along with the SoC-specific symbols.
> ---
> arch/arm/mach-shmobile/Kconfig | 101 ---------------------------
> drivers/soc/renesas/Kconfig | 120 ++++++++++++++++++++++++++++++---
> 2 files changed, 112 insertions(+), 109 deletions(-)
>
> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index b100c26a858f9015..50267ad76f990f11 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -36,104 +36,3 @@ menuconfig ARCH_RENESAS
> select PINCTRL
> select SOC_BUS
> select ZONE_DMA if ARM_LPAE
> -
> -if ARCH_RENESAS
> -
> -#comment "Renesas ARM SoCs System Type"
> -
> -config ARCH_EMEV2
> - bool "Emma Mobile EV2"
> - select SYS_SUPPORTS_EM_STI
> -
> -config ARCH_R7S72100
> - bool "RZ/A1H (R7S72100)"
> - select PM
> - select PM_GENERIC_DOMAINS
> - select SYS_SUPPORTS_SH_MTU2
> - select RENESAS_OSTM
> -
> -config ARCH_R7S9210
> - bool "RZ/A2 (R7S9210)"
> - select PM
> - select PM_GENERIC_DOMAINS
> - select RENESAS_OSTM
> -
> -config ARCH_R8A73A4
> - bool "R-Mobile APE6 (R8A73A40)"
> - select ARCH_RMOBILE
> - select ARM_ERRATA_798181 if SMP
> - select HAVE_ARM_ARCH_TIMER
> - select RENESAS_IRQC
> -
> -config ARCH_R8A7740
> - bool "R-Mobile A1 (R8A77400)"
> - select ARCH_RMOBILE
> - select RENESAS_INTC_IRQPIN
> -
> -config ARCH_R8A7743
> - bool "RZ/G1M (R8A77430)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7744
> - bool "RZ/G1N (R8A77440)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7745
> - bool "RZ/G1E (R8A77450)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R8A77470
> - bool "RZ/G1C (R8A77470)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R8A7778
> - bool "R-Car M1A (R8A77781)"
> - select ARCH_RCAR_GEN1
> -
> -config ARCH_R8A7779
> - bool "R-Car H1 (R8A77790)"
> - select ARCH_RCAR_GEN1
> -
> -config ARCH_R8A7790
> - bool "R-Car H2 (R8A77900)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7791
> - bool "R-Car M2-W (R8A77910)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7792
> - bool "R-Car V2H (R8A77920)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7793
> - bool "R-Car M2-N (R8A7793)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7794
> - bool "R-Car E2 (R8A77940)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R9A06G032
> - bool "RZ/N1D (R9A06G032)"
> - select ARCH_RZN1
> -
> -config ARCH_RZN1
> - bool "RZ/N1 (R9A06G0xx) Family"
> - select ARM_AMBA
> - select CPU_V7
> -
> -config ARCH_SH73A0
> - bool "SH-Mobile AG5 (R8A73A00)"
> - select ARCH_RMOBILE
> - select RENESAS_INTC_IRQPIN
> -endif
> diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> index 0ab62024fd20be56..5089c65b544909c2 100644
> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -4,17 +4,121 @@ config SOC_RENESAS
> default y if ARCH_RENESAS
> select SOC_BUS
> select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> - select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> - select SYSC_R8A7745 if ARCH_R8A7745
> - select SYSC_R8A77470 if ARCH_R8A77470
> - select SYSC_R8A7779 if ARCH_R8A7779
> - select SYSC_R8A7790 if ARCH_R8A7790
> - select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> - select SYSC_R8A7792 if ARCH_R8A7792
> - select SYSC_R8A7794 if ARCH_R8A7794
>
> if SOC_RENESAS
>
> +if ARM
> +
> +#comment "Renesas ARM SoCs System Type"
> +
> +config ARCH_EMEV2
> + bool "Emma Mobile EV2"
> + select SYS_SUPPORTS_EM_STI
> +
> +config ARCH_R7S72100
> + bool "RZ/A1H (R7S72100)"
> + select PM
> + select PM_GENERIC_DOMAINS
> + select SYS_SUPPORTS_SH_MTU2
> + select RENESAS_OSTM
> +
> +config ARCH_R7S9210
> + bool "RZ/A2 (R7S9210)"
> + select PM
> + select PM_GENERIC_DOMAINS
> + select RENESAS_OSTM
> +
> +config ARCH_R8A73A4
> + bool "R-Mobile APE6 (R8A73A40)"
> + select ARCH_RMOBILE
> + select ARM_ERRATA_798181 if SMP
> + select HAVE_ARM_ARCH_TIMER
> + select RENESAS_IRQC
> +
> +config ARCH_R8A7740
> + bool "R-Mobile A1 (R8A77400)"
> + select ARCH_RMOBILE
> + select RENESAS_INTC_IRQPIN
> +
> +config ARCH_R8A7743
> + bool "RZ/G1M (R8A77430)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7743
> +
> +config ARCH_R8A7744
> + bool "RZ/G1N (R8A77440)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7743
> +
> +config ARCH_R8A7745
> + bool "RZ/G1E (R8A77450)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A7745
> +
> +config ARCH_R8A77470
> + bool "RZ/G1C (R8A77470)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A77470
> +
> +config ARCH_R8A7778
> + bool "R-Car M1A (R8A77781)"
> + select ARCH_RCAR_GEN1
> +
> +config ARCH_R8A7779
> + bool "R-Car H1 (R8A77790)"
> + select ARCH_RCAR_GEN1
> + select SYSC_R8A7779
> +
> +config ARCH_R8A7790
> + bool "R-Car H2 (R8A77900)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7790
> +
> +config ARCH_R8A7791
> + bool "R-Car M2-W (R8A77910)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7791
> +
> +config ARCH_R8A7792
> + bool "R-Car V2H (R8A77920)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7792
> +
> +config ARCH_R8A7793
> + bool "R-Car M2-N (R8A7793)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7791
> +
> +config ARCH_R8A7794
> + bool "R-Car E2 (R8A77940)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A7794
> +
> +config ARCH_R9A06G032
> + bool "RZ/N1D (R9A06G032)"
> + select ARCH_RZN1
> +
> +config ARCH_RZN1
> + bool "RZ/N1 (R9A06G0xx) Family"
> + select ARM_AMBA
> + select CPU_V7
> +
> +config ARCH_SH73A0
> + bool "SH-Mobile AG5 (R8A73A00)"
> + select ARCH_RMOBILE
> + select RENESAS_INTC_IRQPIN
> +
> +endif # ARM
> +
> if ARM64
>
> config ARCH_R8A774A1
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 2/2] [RFC] ARM: shmobile: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-17 10:03 ` Simon Horman
0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-17 10:03 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Oct 11, 2018 at 10:55:08AM +0200, Geert Uytterhoeven wrote:
> For consistency with arm64, where vendors have a single Kconfig symbol
> in arch/arm64/Kconfig.platforms.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Note that drivers/clk/ is included before drivers/soc/. Hence when
> COMPILE_TEST=y, questions will be asked about clock drivers before they
> can be auto-selected by SoC support.
>
> Question: Should the family-specific Kconfig symbols be moved, too?
> Not much would be left in arch/arm/mach-shmobile/Kconfig,
> though.
I have no strong opinion either way. But I lean slightly towards
moving the family-specific symbols along with the SoC-specific symbols.
> ---
> arch/arm/mach-shmobile/Kconfig | 101 ---------------------------
> drivers/soc/renesas/Kconfig | 120 ++++++++++++++++++++++++++++++---
> 2 files changed, 112 insertions(+), 109 deletions(-)
>
> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index b100c26a858f9015..50267ad76f990f11 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
> @@ -36,104 +36,3 @@ menuconfig ARCH_RENESAS
> select PINCTRL
> select SOC_BUS
> select ZONE_DMA if ARM_LPAE
> -
> -if ARCH_RENESAS
> -
> -#comment "Renesas ARM SoCs System Type"
> -
> -config ARCH_EMEV2
> - bool "Emma Mobile EV2"
> - select SYS_SUPPORTS_EM_STI
> -
> -config ARCH_R7S72100
> - bool "RZ/A1H (R7S72100)"
> - select PM
> - select PM_GENERIC_DOMAINS
> - select SYS_SUPPORTS_SH_MTU2
> - select RENESAS_OSTM
> -
> -config ARCH_R7S9210
> - bool "RZ/A2 (R7S9210)"
> - select PM
> - select PM_GENERIC_DOMAINS
> - select RENESAS_OSTM
> -
> -config ARCH_R8A73A4
> - bool "R-Mobile APE6 (R8A73A40)"
> - select ARCH_RMOBILE
> - select ARM_ERRATA_798181 if SMP
> - select HAVE_ARM_ARCH_TIMER
> - select RENESAS_IRQC
> -
> -config ARCH_R8A7740
> - bool "R-Mobile A1 (R8A77400)"
> - select ARCH_RMOBILE
> - select RENESAS_INTC_IRQPIN
> -
> -config ARCH_R8A7743
> - bool "RZ/G1M (R8A77430)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7744
> - bool "RZ/G1N (R8A77440)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7745
> - bool "RZ/G1E (R8A77450)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R8A77470
> - bool "RZ/G1C (R8A77470)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R8A7778
> - bool "R-Car M1A (R8A77781)"
> - select ARCH_RCAR_GEN1
> -
> -config ARCH_R8A7779
> - bool "R-Car H1 (R8A77790)"
> - select ARCH_RCAR_GEN1
> -
> -config ARCH_R8A7790
> - bool "R-Car H2 (R8A77900)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7791
> - bool "R-Car M2-W (R8A77910)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7792
> - bool "R-Car V2H (R8A77920)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> -
> -config ARCH_R8A7793
> - bool "R-Car M2-N (R8A7793)"
> - select ARCH_RCAR_GEN2
> - select ARM_ERRATA_798181 if SMP
> - select I2C
> -
> -config ARCH_R8A7794
> - bool "R-Car E2 (R8A77940)"
> - select ARCH_RCAR_GEN2
> -
> -config ARCH_R9A06G032
> - bool "RZ/N1D (R9A06G032)"
> - select ARCH_RZN1
> -
> -config ARCH_RZN1
> - bool "RZ/N1 (R9A06G0xx) Family"
> - select ARM_AMBA
> - select CPU_V7
> -
> -config ARCH_SH73A0
> - bool "SH-Mobile AG5 (R8A73A00)"
> - select ARCH_RMOBILE
> - select RENESAS_INTC_IRQPIN
> -endif
> diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> index 0ab62024fd20be56..5089c65b544909c2 100644
> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -4,17 +4,121 @@ config SOC_RENESAS
> default y if ARCH_RENESAS
> select SOC_BUS
> select RST_RCAR if ARCH_RCAR_GEN1 || ARCH_RCAR_GEN2
> - select SYSC_R8A7743 if ARCH_R8A7743 || ARCH_R8A7744
> - select SYSC_R8A7745 if ARCH_R8A7745
> - select SYSC_R8A77470 if ARCH_R8A77470
> - select SYSC_R8A7779 if ARCH_R8A7779
> - select SYSC_R8A7790 if ARCH_R8A7790
> - select SYSC_R8A7791 if ARCH_R8A7791 || ARCH_R8A7793
> - select SYSC_R8A7792 if ARCH_R8A7792
> - select SYSC_R8A7794 if ARCH_R8A7794
>
> if SOC_RENESAS
>
> +if ARM
> +
> +#comment "Renesas ARM SoCs System Type"
> +
> +config ARCH_EMEV2
> + bool "Emma Mobile EV2"
> + select SYS_SUPPORTS_EM_STI
> +
> +config ARCH_R7S72100
> + bool "RZ/A1H (R7S72100)"
> + select PM
> + select PM_GENERIC_DOMAINS
> + select SYS_SUPPORTS_SH_MTU2
> + select RENESAS_OSTM
> +
> +config ARCH_R7S9210
> + bool "RZ/A2 (R7S9210)"
> + select PM
> + select PM_GENERIC_DOMAINS
> + select RENESAS_OSTM
> +
> +config ARCH_R8A73A4
> + bool "R-Mobile APE6 (R8A73A40)"
> + select ARCH_RMOBILE
> + select ARM_ERRATA_798181 if SMP
> + select HAVE_ARM_ARCH_TIMER
> + select RENESAS_IRQC
> +
> +config ARCH_R8A7740
> + bool "R-Mobile A1 (R8A77400)"
> + select ARCH_RMOBILE
> + select RENESAS_INTC_IRQPIN
> +
> +config ARCH_R8A7743
> + bool "RZ/G1M (R8A77430)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7743
> +
> +config ARCH_R8A7744
> + bool "RZ/G1N (R8A77440)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7743
> +
> +config ARCH_R8A7745
> + bool "RZ/G1E (R8A77450)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A7745
> +
> +config ARCH_R8A77470
> + bool "RZ/G1C (R8A77470)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A77470
> +
> +config ARCH_R8A7778
> + bool "R-Car M1A (R8A77781)"
> + select ARCH_RCAR_GEN1
> +
> +config ARCH_R8A7779
> + bool "R-Car H1 (R8A77790)"
> + select ARCH_RCAR_GEN1
> + select SYSC_R8A7779
> +
> +config ARCH_R8A7790
> + bool "R-Car H2 (R8A77900)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7790
> +
> +config ARCH_R8A7791
> + bool "R-Car M2-W (R8A77910)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7791
> +
> +config ARCH_R8A7792
> + bool "R-Car V2H (R8A77920)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select SYSC_R8A7792
> +
> +config ARCH_R8A7793
> + bool "R-Car M2-N (R8A7793)"
> + select ARCH_RCAR_GEN2
> + select ARM_ERRATA_798181 if SMP
> + select I2C
> + select SYSC_R8A7791
> +
> +config ARCH_R8A7794
> + bool "R-Car E2 (R8A77940)"
> + select ARCH_RCAR_GEN2
> + select SYSC_R8A7794
> +
> +config ARCH_R9A06G032
> + bool "RZ/N1D (R9A06G032)"
> + select ARCH_RZN1
> +
> +config ARCH_RZN1
> + bool "RZ/N1 (R9A06G0xx) Family"
> + select ARM_AMBA
> + select CPU_V7
> +
> +config ARCH_SH73A0
> + bool "SH-Mobile AG5 (R8A73A00)"
> + select ARCH_RMOBILE
> + select RENESAS_INTC_IRQPIN
> +
> +endif # ARM
> +
> if ARM64
>
> config ARCH_R8A774A1
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
2018-10-17 9:56 ` Geert Uytterhoeven
@ 2018-10-17 10:03 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-17 10:03 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Geert Uytterhoeven, Arnd Bergmann, Magnus Damm, Catalin Marinas,
Will Deacon, Olof Johansson, Kevin Hilman, Linux-Renesas,
Linux ARM
On Wed, Oct 17, 2018 at 11:56:43AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Mon, Oct 15, 2018 at 5:40 PM Simon Horman <horms@verge.net.au> wrote:
> > On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> > > On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > > > For consistency with other vendors, which have a single Kconfig symbol
> > > > in arch/arm64/Kconfig.platforms.
> > > >
> > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > > ---
> > > > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > > > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > > > can be auto-selected by SoC support.
> > > >
> > > > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > > > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
> > >
> > > s/1/3/?
>
> Sure.
>
> > > >
> > > > RST_RCAR?
> > >
> > > Given that it would be consistent with R-Car Gen 1 and 2,
> > > that seems like a good idea to me.
> >
> > there has been no other feedback on this series.
> > How would you like to proceed?
>
> I will add a new symbol ARCH_RCAR_GEN3 for v2?
Good plan.
> Any comments about PATCH 2/2?
Thanks, I have now commented.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/2] [RFC] arm64: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/
@ 2018-10-17 10:03 ` Simon Horman
0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2018-10-17 10:03 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 17, 2018 at 11:56:43AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Mon, Oct 15, 2018 at 5:40 PM Simon Horman <horms@verge.net.au> wrote:
> > On Thu, Oct 11, 2018 at 03:54:55PM +0200, Simon Horman wrote:
> > > On Thu, Oct 11, 2018 at 10:55:07AM +0200, Geert Uytterhoeven wrote:
> > > > For consistency with other vendors, which have a single Kconfig symbol
> > > > in arch/arm64/Kconfig.platforms.
> > > >
> > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > > ---
> > > > Note that drivers/clk/ is included before drivers/soc/. Hence when
> > > > COMPILE_TEST=y, questions will be asked about clock drivers before they
> > > > can be auto-selected by SoC support.
> > > >
> > > > Question: Should we introduce a family-specific Kconfig symbol for R-Car
> > > > Gen3 (ARCH_RCAR_GEN1), which could be used for enabling
> > >
> > > s/1/3/?
>
> Sure.
>
> > > >
> > > > RST_RCAR?
> > >
> > > Given that it would be consistent with R-Car Gen 1 and 2,
> > > that seems like a good idea to me.
> >
> > there has been no other feedback on this series.
> > How would you like to proceed?
>
> I will add a new symbol ARCH_RCAR_GEN3 for v2?
Good plan.
> Any comments about PATCH 2/2?
Thanks, I have now commented.
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2018-10-17 17:58 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-11 8:55 [PATCH/RFC 0/2] arm: renesas: Move SoC Kconfig symbols to drivers/soc/renesas/ Geert Uytterhoeven
2018-10-11 8:55 ` Geert Uytterhoeven
2018-10-11 8:55 ` [PATCH 1/2] [RFC] arm64: " Geert Uytterhoeven
2018-10-11 8:55 ` Geert Uytterhoeven
2018-10-11 13:54 ` Simon Horman
2018-10-11 13:54 ` Simon Horman
2018-10-15 15:40 ` Simon Horman
2018-10-15 15:40 ` Simon Horman
2018-10-17 9:56 ` Geert Uytterhoeven
2018-10-17 9:56 ` Geert Uytterhoeven
2018-10-17 10:03 ` Simon Horman
2018-10-17 10:03 ` Simon Horman
2018-10-11 8:55 ` [PATCH 2/2] [RFC] ARM: shmobile: " Geert Uytterhoeven
2018-10-11 8:55 ` Geert Uytterhoeven
2018-10-17 10:03 ` Simon Horman
2018-10-17 10:03 ` Simon Horman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.