All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC
@ 2020-08-24  2:20 Kuninori Morimoto
  2020-08-24  2:21 ` [PATCH v3 1/4] " Kuninori Morimoto
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:20 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, linux-renesas-soc


Hi Geert

These are v3 patch for Renesas SoC.

Renesas related SoC settings are located on TOP level menu
when menuconfig, thus it is very verbose.
This patch groups Renesas related settings into
"Renesas SoC driver support" menu.

These are based on "renesas-drivers-2020-08-18-v5.9-rc1"

v2 -> v3
	- add Geert's Reviewed-by if v2 get it.
	- Some SoC has "SoC Platform support" on description.
	- each description has if xxx
	- all drivers are sorted

v1 -> v2
	- align description
	- sort description


Kuninori Morimoto (4):
  soc: renesas: use menu for Renesas SoC
  soc: renesas: align driver description title
  soc: renesas: Merge if xxx into menu description
  soc: renesas: sort driver description title

 drivers/soc/renesas/Kconfig | 349 ++++++++++++++++++------------------
 1 file changed, 171 insertions(+), 178 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH v3 1/4] soc: renesas: use menu for Renesas SoC
  2020-08-24  2:20 [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC Kuninori Morimoto
@ 2020-08-24  2:21 ` Kuninori Morimoto
  2020-08-24  2:22 ` [PATCH v3 2/4] soc: renesas: align driver description title Kuninori Morimoto
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 9+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:21 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Renesas related SoC settings are located on TOP level menu,
thus it is very verbose.
This patch groups Renesas related settings into
"Renesas SoC driver support" menu.

And it aligns config menu names.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/soc/renesas/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 30984659df90..265a59345909 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0
-config SOC_RENESAS
+menuconfig SOC_RENESAS
 	bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
 	default y if ARCH_RENESAS
 	select SOC_BUS
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [PATCH v3 2/4] soc: renesas: align driver description title
  2020-08-24  2:20 [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC Kuninori Morimoto
  2020-08-24  2:21 ` [PATCH v3 1/4] " Kuninori Morimoto
@ 2020-08-24  2:22 ` Kuninori Morimoto
  2020-09-01  7:46   ` Geert Uytterhoeven
  2020-08-24  2:22 ` [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description Kuninori Morimoto
  2020-08-24  2:22 ` [PATCH v3 4/4] soc: renesas: sort driver description title Kuninori Morimoto
  3 siblings, 1 reply; 9+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:22 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Now, Renesas SoC drivers are under menu,
but current description are not aligned.
This patch align these.

	- R-Car H2 System Controller support
	- R-Car M2-W/N System Controller support
	- R-Car V2H System Controller support
	- R-Car E2 System Controller support
	- R-Car H3 System Controller support
	- R-Car M3-W System Controller support
	- R-Car M3-W+ System Controller support
	- R-Car M3-N System Controller support

	+ System Controller support for R-Car H2
	+ System Controller support for R-Car M2-W/N
	+ System Controller support for R-Car V2H
	+ System Controller support for R-Car E2
	+ System Controller support for R-Car H3
	+ System Controller support for R-Car M3-W
	+ System Controller support for R-Car M3-W+
	+ System Controller support for R-Car M3-N

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/soc/renesas/Kconfig | 112 ++++++++++++++++++------------------
 1 file changed, 56 insertions(+), 56 deletions(-)

diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 265a59345909..7d63a13e5b78 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -49,12 +49,12 @@ if ARM && ARCH_RENESAS
 #comment "Renesas ARM SoCs System Type"
 
 config ARCH_EMEV2
-	bool "Emma Mobile EV2"
+	bool "SoC Platform support for Emma Mobile EV2"
 	select HAVE_ARM_SCU if SMP
 	select SYS_SUPPORTS_EM_STI
 
 config ARCH_R7S72100
-	bool "RZ/A1H (R7S72100)"
+	bool "SoC Platform support for RZ/A1H"
 	select ARM_ERRATA_754322
 	select PM
 	select PM_GENERIC_DOMAINS
@@ -63,14 +63,14 @@ config ARCH_R7S72100
 	select SYS_SUPPORTS_SH_MTU2
 
 config ARCH_R7S9210
-	bool "RZ/A2 (R7S9210)"
+	bool "SoC Platform support for RZ/A2"
 	select PM
 	select PM_GENERIC_DOMAINS
 	select RENESAS_OSTM
 	select RENESAS_RZA1_IRQC
 
 config ARCH_R8A73A4
-	bool "R-Mobile APE6 (R8A73A40)"
+	bool "SoC Platform support for R-Mobile APE6"
 	select ARCH_RMOBILE
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
@@ -78,49 +78,49 @@ config ARCH_R8A73A4
 	select RENESAS_IRQC
 
 config ARCH_R8A7740
-	bool "R-Mobile A1 (R8A77400)"
+	bool "SoC Platform support for R-Mobile A1"
 	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
 	select RENESAS_INTC_IRQPIN
 
 config ARCH_R8A7742
-	bool "RZ/G1H (R8A77420)"
+	bool "SoC Platform support for RZ/G1H"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
 	select SYSC_R8A7742
 
 config ARCH_R8A7743
-	bool "RZ/G1M (R8A77430)"
+	bool "SoC Platform support for RZ/G1M"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7743
 
 config ARCH_R8A7744
-	bool "RZ/G1N (R8A77440)"
+	bool "SoC Platform support for RZ/G1N"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7743
 
 config ARCH_R8A7745
-	bool "RZ/G1E (R8A77450)"
+	bool "SoC Platform support for RZ/G1E"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A7745
 
 config ARCH_R8A77470
-	bool "RZ/G1C (R8A77470)"
+	bool "SoC Platform support for RZ/G1C"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A77470
 
 config ARCH_R8A7778
-	bool "R-Car M1A (R8A77781)"
+	bool "SoC Platform support for R-Car M1A"
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 
 config ARCH_R8A7779
-	bool "R-Car H1 (R8A77790)"
+	bool "SoC Platform support for R-Car H1"
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 	select ARM_GLOBAL_TIMER
@@ -129,7 +129,7 @@ config ARCH_R8A7779
 	select SYSC_R8A7779
 
 config ARCH_R8A7790
-	bool "R-Car H2 (R8A77900)"
+	bool "SoC Platform support for R-Car H2"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
@@ -137,38 +137,38 @@ config ARCH_R8A7790
 	select SYSC_R8A7790
 
 config ARCH_R8A7791
-	bool "R-Car M2-W (R8A77910)"
+	bool "SoC Platform support for R-Car M2-W"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select I2C
 	select SYSC_R8A7791
 
 config ARCH_R8A7792
-	bool "R-Car V2H (R8A77920)"
+	bool "SoC Platform support for R-Car V2H"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7792
 
 config ARCH_R8A7793
-	bool "R-Car M2-N (R8A7793)"
+	bool "SoC Platform support for R-Car M2-N"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select I2C
 	select SYSC_R8A7791
 
 config ARCH_R8A7794
-	bool "R-Car E2 (R8A77940)"
+	bool "SoC Platform support for R-Car E2"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A7794
 
 config ARCH_R9A06G032
-	bool "RZ/N1D (R9A06G032)"
+	bool "SoC Platform support for RZ/N1D"
 	select ARCH_RZN1
 	select ARM_ERRATA_814220
 
 config ARCH_SH73A0
-	bool "SH-Mobile AG5 (R8A73A00)"
+	bool "SoC Platform support for SH-Mobile AG5"
 	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
 	select ARM_GLOBAL_TIMER
@@ -181,42 +181,42 @@ endif # ARM
 if ARM64
 
 config ARCH_R8A774A1
-	bool "Renesas RZ/G2M SoC Platform"
+	bool "SoC Platform support for RZ/G2M"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774A1
 	help
 	  This enables support for the Renesas RZ/G2M SoC.
 
 config ARCH_R8A774B1
-	bool "Renesas RZ/G2N SoC Platform"
+	bool "SoC Platform support for RZ/G2N"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774B1
 	help
 	  This enables support for the Renesas RZ/G2N SoC.
 
 config ARCH_R8A774C0
-	bool "Renesas RZ/G2E SoC Platform"
+	bool "SoC Platform support for RZ/G2E"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774C0
 	help
 	  This enables support for the Renesas RZ/G2E SoC.
 
 config ARCH_R8A774E1
-	bool "Renesas RZ/G2H SoC Platform"
+	bool "SoC Platform support for RZ/G2H"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774E1
 	help
 	  This enables support for the Renesas RZ/G2H SoC.
 
 config ARCH_R8A77950
-	bool "Renesas R-Car H3 ES1.x SoC Platform"
+	bool "SoC Platform support for R-Car H3 ES1.x"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A7795
 	help
 	  This enables support for the Renesas R-Car H3 SoC (revision 1.x).
 
 config ARCH_R8A77951
-	bool "Renesas R-Car H3 ES2.0+ SoC Platform"
+	bool "SoC Platform support for R-Car H3 ES2.0+"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A7795
 	help
@@ -224,49 +224,49 @@ config ARCH_R8A77951
 	  later).
 
 config ARCH_R8A77960
-	bool "Renesas R-Car M3-W SoC Platform"
+	bool "SoC Platform support for R-Car M3-W"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77960
 	help
 	  This enables support for the Renesas R-Car M3-W SoC.
 
 config ARCH_R8A77961
-	bool "Renesas R-Car M3-W+ SoC Platform"
+	bool "SoC Platform support for R-Car M3-W+"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77961
 	help
 	  This enables support for the Renesas R-Car M3-W+ SoC.
 
 config ARCH_R8A77965
-	bool "Renesas R-Car M3-N SoC Platform"
+	bool "SoC Platform support for R-Car M3-N"
 	select ARCH_RCAR_GEN3
 	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"
+	bool "SoC Platform support for R-Car V3M"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77970
 	help
 	  This enables support for the Renesas R-Car V3M SoC.
 
 config ARCH_R8A77980
-	bool "Renesas R-Car V3H SoC Platform"
+	bool "SoC Platform support for R-Car V3H"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77980
 	help
 	  This enables support for the Renesas R-Car V3H SoC.
 
 config ARCH_R8A77990
-	bool "Renesas R-Car E3 SoC Platform"
+	bool "SoC Platform support for R-Car E3"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77990
 	help
 	  This enables support for the Renesas R-Car E3 SoC.
 
 config ARCH_R8A77995
-	bool "Renesas R-Car D3 SoC Platform"
+	bool "SoC Platform support for R-Car D3"
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77995
 	help
@@ -276,97 +276,97 @@ endif # ARM64
 
 # SoC
 config SYSC_R8A7742
-	bool "RZ/G1H System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G1H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7743
-	bool "RZ/G1M System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G1M" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7745
-	bool "RZ/G1E System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G1E" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77470
-	bool "RZ/G1C System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G1C" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A774A1
-	bool "RZ/G2M System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G2M" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A774B1
-	bool "RZ/G2N System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G2N" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A774C0
-	bool "RZ/G2E System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G2E" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A774E1
-	bool "RZ/G2H System Controller support" if COMPILE_TEST
+	bool "System Controller support for RZ/G2H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7779
-	bool "R-Car H1 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car H1" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7790
-	bool "R-Car H2 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car H2" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7791
-	bool "R-Car M2-W/N System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7792
-	bool "R-Car V2H System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car V2H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7794
-	bool "R-Car E2 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car E2" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7795
-	bool "R-Car H3 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car H3" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77960
-	bool "R-Car M3-W System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car M3-W" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77961
-	bool "R-Car M3-W+ System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car M3-W+" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77965
-	bool "R-Car M3-N System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car M3-N" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77970
-	bool "R-Car V3M System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car V3M" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77980
-	bool "R-Car V3H System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car V3H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77990
-	bool "R-Car E3 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car E3" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77995
-	bool "R-Car D3 System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car D3" if COMPILE_TEST
 	select SYSC_RCAR
 
 # Family
 config RST_RCAR
-	bool "R-Car Reset Controller support" if COMPILE_TEST
+	bool "Reset Controller support for R-Car" if COMPILE_TEST
 
 config SYSC_RCAR
-	bool "R-Car System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Car" if COMPILE_TEST
 
 config SYSC_RMOBILE
-	bool "R-Mobile System Controller support" if COMPILE_TEST
+	bool "System Controller support for R-Mobile" if COMPILE_TEST
 
 endif # SOC_RENESAS
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description
  2020-08-24  2:20 [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC Kuninori Morimoto
  2020-08-24  2:21 ` [PATCH v3 1/4] " Kuninori Morimoto
  2020-08-24  2:22 ` [PATCH v3 2/4] soc: renesas: align driver description title Kuninori Morimoto
@ 2020-08-24  2:22 ` Kuninori Morimoto
  2020-09-01  7:55   ` Geert Uytterhoeven
  2020-08-24  2:22 ` [PATCH v3 4/4] soc: renesas: sort driver description title Kuninori Morimoto
  3 siblings, 1 reply; 9+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:22 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Current Kconfig is using if - endif.
This patch merges it into each menu description.
This is prepare for menu sorting.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 drivers/soc/renesas/Kconfig | 72 +++++++++++++++++--------------------
 1 file changed, 32 insertions(+), 40 deletions(-)

diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index 7d63a13e5b78..a6ab506377fb 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -44,17 +44,15 @@ config ARCH_RZN1
 	bool
 	select ARM_AMBA
 
-if ARM && ARCH_RENESAS
-
 #comment "Renesas ARM SoCs System Type"
 
 config ARCH_EMEV2
-	bool "SoC Platform support for Emma Mobile EV2"
+	bool "SoC Platform support for Emma Mobile EV2" if ARM && ARCH_RENESAS
 	select HAVE_ARM_SCU if SMP
 	select SYS_SUPPORTS_EM_STI
 
 config ARCH_R7S72100
-	bool "SoC Platform support for RZ/A1H"
+	bool "SoC Platform support for RZ/A1H" if ARM && ARCH_RENESAS
 	select ARM_ERRATA_754322
 	select PM
 	select PM_GENERIC_DOMAINS
@@ -63,14 +61,14 @@ config ARCH_R7S72100
 	select SYS_SUPPORTS_SH_MTU2
 
 config ARCH_R7S9210
-	bool "SoC Platform support for RZ/A2"
+	bool "SoC Platform support for RZ/A2" if ARM && ARCH_RENESAS
 	select PM
 	select PM_GENERIC_DOMAINS
 	select RENESAS_OSTM
 	select RENESAS_RZA1_IRQC
 
 config ARCH_R8A73A4
-	bool "SoC Platform support for R-Mobile APE6"
+	bool "SoC Platform support for R-Mobile APE6" if ARM && ARCH_RENESAS
 	select ARCH_RMOBILE
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
@@ -78,49 +76,49 @@ config ARCH_R8A73A4
 	select RENESAS_IRQC
 
 config ARCH_R8A7740
-	bool "SoC Platform support for R-Mobile A1"
+	bool "SoC Platform support for R-Mobile A1" if ARM && ARCH_RENESAS
 	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
 	select RENESAS_INTC_IRQPIN
 
 config ARCH_R8A7742
-	bool "SoC Platform support for RZ/G1H"
+	bool "SoC Platform support for RZ/G1H" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
 	select SYSC_R8A7742
 
 config ARCH_R8A7743
-	bool "SoC Platform support for RZ/G1M"
+	bool "SoC Platform support for RZ/G1M" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7743
 
 config ARCH_R8A7744
-	bool "SoC Platform support for RZ/G1N"
+	bool "SoC Platform support for RZ/G1N" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7743
 
 config ARCH_R8A7745
-	bool "SoC Platform support for RZ/G1E"
+	bool "SoC Platform support for RZ/G1E" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A7745
 
 config ARCH_R8A77470
-	bool "SoC Platform support for RZ/G1C"
+	bool "SoC Platform support for RZ/G1C" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A77470
 
 config ARCH_R8A7778
-	bool "SoC Platform support for R-Car M1A"
+	bool "SoC Platform support for R-Car M1A" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 
 config ARCH_R8A7779
-	bool "SoC Platform support for R-Car H1"
+	bool "SoC Platform support for R-Car H1" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 	select ARM_GLOBAL_TIMER
@@ -129,7 +127,7 @@ config ARCH_R8A7779
 	select SYSC_R8A7779
 
 config ARCH_R8A7790
-	bool "SoC Platform support for R-Car H2"
+	bool "SoC Platform support for R-Car H2" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
@@ -137,38 +135,38 @@ config ARCH_R8A7790
 	select SYSC_R8A7790
 
 config ARCH_R8A7791
-	bool "SoC Platform support for R-Car M2-W"
+	bool "SoC Platform support for R-Car M2-W" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select I2C
 	select SYSC_R8A7791
 
 config ARCH_R8A7792
-	bool "SoC Platform support for R-Car V2H"
+	bool "SoC Platform support for R-Car V2H" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7792
 
 config ARCH_R8A7793
-	bool "SoC Platform support for R-Car M2-N"
+	bool "SoC Platform support for R-Car M2-N" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select I2C
 	select SYSC_R8A7791
 
 config ARCH_R8A7794
-	bool "SoC Platform support for R-Car E2"
+	bool "SoC Platform support for R-Car E2" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A7794
 
 config ARCH_R9A06G032
-	bool "SoC Platform support for RZ/N1D"
+	bool "SoC Platform support for RZ/N1D" if ARM && ARCH_RENESAS
 	select ARCH_RZN1
 	select ARM_ERRATA_814220
 
 config ARCH_SH73A0
-	bool "SoC Platform support for SH-Mobile AG5"
+	bool "SoC Platform support for SH-Mobile AG5" if ARM && ARCH_RENESAS
 	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
 	select ARM_GLOBAL_TIMER
@@ -176,47 +174,43 @@ config ARCH_SH73A0
 	select HAVE_ARM_TWD if SMP
 	select RENESAS_INTC_IRQPIN
 
-endif # ARM
-
-if ARM64
-
 config ARCH_R8A774A1
-	bool "SoC Platform support for RZ/G2M"
+	bool "SoC Platform support for RZ/G2M" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774A1
 	help
 	  This enables support for the Renesas RZ/G2M SoC.
 
 config ARCH_R8A774B1
-	bool "SoC Platform support for RZ/G2N"
+	bool "SoC Platform support for RZ/G2N" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774B1
 	help
 	  This enables support for the Renesas RZ/G2N SoC.
 
 config ARCH_R8A774C0
-	bool "SoC Platform support for RZ/G2E"
+	bool "SoC Platform support for RZ/G2E" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774C0
 	help
 	  This enables support for the Renesas RZ/G2E SoC.
 
 config ARCH_R8A774E1
-	bool "SoC Platform support for RZ/G2H"
+	bool "SoC Platform support for RZ/G2H" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A774E1
 	help
 	  This enables support for the Renesas RZ/G2H SoC.
 
 config ARCH_R8A77950
-	bool "SoC Platform support for R-Car H3 ES1.x"
+	bool "SoC Platform support for R-Car H3 ES1.x" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A7795
 	help
 	  This enables support for the Renesas R-Car H3 SoC (revision 1.x).
 
 config ARCH_R8A77951
-	bool "SoC Platform support for R-Car H3 ES2.0+"
+	bool "SoC Platform support for R-Car H3 ES2.0+" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A7795
 	help
@@ -224,56 +218,54 @@ config ARCH_R8A77951
 	  later).
 
 config ARCH_R8A77960
-	bool "SoC Platform support for R-Car M3-W"
+	bool "SoC Platform support for R-Car M3-W" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77960
 	help
 	  This enables support for the Renesas R-Car M3-W SoC.
 
 config ARCH_R8A77961
-	bool "SoC Platform support for R-Car M3-W+"
+	bool "SoC Platform support for R-Car M3-W+" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77961
 	help
 	  This enables support for the Renesas R-Car M3-W+ SoC.
 
 config ARCH_R8A77965
-	bool "SoC Platform support for R-Car M3-N"
+	bool "SoC Platform support for R-Car M3-N" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77965
 	help
 	  This enables support for the Renesas R-Car M3-N SoC.
 
 config ARCH_R8A77970
-	bool "SoC Platform support for R-Car V3M"
+	bool "SoC Platform support for R-Car V3M" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77970
 	help
 	  This enables support for the Renesas R-Car V3M SoC.
 
 config ARCH_R8A77980
-	bool "SoC Platform support for R-Car V3H"
+	bool "SoC Platform support for R-Car V3H" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77980
 	help
 	  This enables support for the Renesas R-Car V3H SoC.
 
 config ARCH_R8A77990
-	bool "SoC Platform support for R-Car E3"
+	bool "SoC Platform support for R-Car E3" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77990
 	help
 	  This enables support for the Renesas R-Car E3 SoC.
 
 config ARCH_R8A77995
-	bool "SoC Platform support for R-Car D3"
+	bool "SoC Platform support for R-Car D3" if ARM64
 	select ARCH_RCAR_GEN3
 	select SYSC_R8A77995
 	help
 	  This enables support for the Renesas R-Car D3 SoC.
 
-endif # ARM64
-
 # SoC
 config SYSC_R8A7742
 	bool "System Controller support for RZ/G1H" if COMPILE_TEST
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [PATCH v3 4/4] soc: renesas: sort driver description title
  2020-08-24  2:20 [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC Kuninori Morimoto
                   ` (2 preceding siblings ...)
  2020-08-24  2:22 ` [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description Kuninori Morimoto
@ 2020-08-24  2:22 ` Kuninori Morimoto
  3 siblings, 0 replies; 9+ messages in thread
From: Kuninori Morimoto @ 2020-08-24  2:22 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, linux-renesas-soc


From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

This patch sorts each driver by description title alphabetical order.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 drivers/soc/renesas/Kconfig | 295 ++++++++++++++++++------------------
 1 file changed, 148 insertions(+), 147 deletions(-)

diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index a6ab506377fb..a2334e58fef6 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -67,19 +67,17 @@ config ARCH_R7S9210
 	select RENESAS_OSTM
 	select RENESAS_RZA1_IRQC
 
-config ARCH_R8A73A4
-	bool "SoC Platform support for R-Mobile APE6" if ARM && ARCH_RENESAS
-	select ARCH_RMOBILE
-	select ARM_ERRATA_798181 if SMP
+config ARCH_R8A77470
+	bool "SoC Platform support for RZ/G1C" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
-	select HAVE_ARM_ARCH_TIMER
-	select RENESAS_IRQC
+	select SYSC_R8A77470
 
-config ARCH_R8A7740
-	bool "SoC Platform support for R-Mobile A1" if ARM && ARCH_RENESAS
-	select ARCH_RMOBILE
-	select ARM_ERRATA_754322
-	select RENESAS_INTC_IRQPIN
+config ARCH_R8A7745
+	bool "SoC Platform support for RZ/G1E" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN2
+	select ARM_ERRATA_814220
+	select SYSC_R8A7745
 
 config ARCH_R8A7742
 	bool "SoC Platform support for RZ/G1H" if ARM && ARCH_RENESAS
@@ -100,22 +98,58 @@ config ARCH_R8A7744
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7743
 
-config ARCH_R8A7745
-	bool "SoC Platform support for RZ/G1E" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN2
+config ARCH_R8A774C0
+	bool "SoC Platform support for RZ/G2E" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A774C0
+	help
+	  This enables support for the Renesas RZ/G2E SoC.
+
+config ARCH_R8A774E1
+	bool "SoC Platform support for RZ/G2H" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A774E1
+	help
+	  This enables support for the Renesas RZ/G2H SoC.
+
+config ARCH_R8A774A1
+	bool "SoC Platform support for RZ/G2M" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A774A1
+	help
+	  This enables support for the Renesas RZ/G2M SoC.
+
+config ARCH_R8A774B1
+	bool "SoC Platform support for RZ/G2N" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A774B1
+	help
+	  This enables support for the Renesas RZ/G2N SoC.
+
+config ARCH_R9A06G032
+	bool "SoC Platform support for RZ/N1D" if ARM && ARCH_RENESAS
+	select ARCH_RZN1
 	select ARM_ERRATA_814220
-	select SYSC_R8A7745
 
-config ARCH_R8A77470
-	bool "SoC Platform support for RZ/G1C" if ARM && ARCH_RENESAS
+config ARCH_R8A77995
+	bool "SoC Platform support for R-Car D3" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A77995
+	help
+	  This enables support for the Renesas R-Car D3 SoC.
+
+config ARCH_R8A7794
+	bool "SoC Platform support for R-Car E2" if ARM && ARCH_RENESAS
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
-	select SYSC_R8A77470
+	select SYSC_R8A7794
 
-config ARCH_R8A7778
-	bool "SoC Platform support for R-Car M1A" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN1
-	select ARM_ERRATA_754322
+config ARCH_R8A77990
+	bool "SoC Platform support for R-Car E3" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A77990
+	help
+	  This enables support for the Renesas R-Car E3 SoC.
 
 config ARCH_R8A7779
 	bool "SoC Platform support for R-Car H1" if ARM && ARCH_RENESAS
@@ -134,74 +168,6 @@ config ARCH_R8A7790
 	select I2C
 	select SYSC_R8A7790
 
-config ARCH_R8A7791
-	bool "SoC Platform support for R-Car M2-W" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN2
-	select ARM_ERRATA_798181 if SMP
-	select I2C
-	select SYSC_R8A7791
-
-config ARCH_R8A7792
-	bool "SoC Platform support for R-Car V2H" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN2
-	select ARM_ERRATA_798181 if SMP
-	select SYSC_R8A7792
-
-config ARCH_R8A7793
-	bool "SoC Platform support for R-Car M2-N" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN2
-	select ARM_ERRATA_798181 if SMP
-	select I2C
-	select SYSC_R8A7791
-
-config ARCH_R8A7794
-	bool "SoC Platform support for R-Car E2" if ARM && ARCH_RENESAS
-	select ARCH_RCAR_GEN2
-	select ARM_ERRATA_814220
-	select SYSC_R8A7794
-
-config ARCH_R9A06G032
-	bool "SoC Platform support for RZ/N1D" if ARM && ARCH_RENESAS
-	select ARCH_RZN1
-	select ARM_ERRATA_814220
-
-config ARCH_SH73A0
-	bool "SoC Platform support for SH-Mobile AG5" if ARM && ARCH_RENESAS
-	select ARCH_RMOBILE
-	select ARM_ERRATA_754322
-	select ARM_GLOBAL_TIMER
-	select HAVE_ARM_SCU if SMP
-	select HAVE_ARM_TWD if SMP
-	select RENESAS_INTC_IRQPIN
-
-config ARCH_R8A774A1
-	bool "SoC Platform support for RZ/G2M" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A774A1
-	help
-	  This enables support for the Renesas RZ/G2M SoC.
-
-config ARCH_R8A774B1
-	bool "SoC Platform support for RZ/G2N" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A774B1
-	help
-	  This enables support for the Renesas RZ/G2N SoC.
-
-config ARCH_R8A774C0
-	bool "SoC Platform support for RZ/G2E" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A774C0
-	help
-	  This enables support for the Renesas RZ/G2E SoC.
-
-config ARCH_R8A774E1
-	bool "SoC Platform support for RZ/G2H" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A774E1
-	help
-	  This enables support for the Renesas RZ/G2H SoC.
-
 config ARCH_R8A77950
 	bool "SoC Platform support for R-Car H3 ES1.x" if ARM64
 	select ARCH_RCAR_GEN3
@@ -217,6 +183,32 @@ config ARCH_R8A77951
 	  This enables support for the Renesas R-Car H3 SoC (revisions 2.0 and
 	  later).
 
+config ARCH_R8A7778
+	bool "SoC Platform support for R-Car M1A" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN1
+	select ARM_ERRATA_754322
+
+config ARCH_R8A7793
+	bool "SoC Platform support for R-Car M2-N" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN2
+	select ARM_ERRATA_798181 if SMP
+	select I2C
+	select SYSC_R8A7791
+
+config ARCH_R8A7791
+	bool "SoC Platform support for R-Car M2-W" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN2
+	select ARM_ERRATA_798181 if SMP
+	select I2C
+	select SYSC_R8A7791
+
+config ARCH_R8A77965
+	bool "SoC Platform support for R-Car M3-N" if ARM64
+	select ARCH_RCAR_GEN3
+	select SYSC_R8A77965
+	help
+	  This enables support for the Renesas R-Car M3-N SoC.
+
 config ARCH_R8A77960
 	bool "SoC Platform support for R-Car M3-W" if ARM64
 	select ARCH_RCAR_GEN3
@@ -231,12 +223,18 @@ config ARCH_R8A77961
 	help
 	  This enables support for the Renesas R-Car M3-W+ SoC.
 
-config ARCH_R8A77965
-	bool "SoC Platform support for R-Car M3-N" if ARM64
+config ARCH_R8A7792
+	bool "SoC Platform support for R-Car V2H" if ARM && ARCH_RENESAS
+	select ARCH_RCAR_GEN2
+	select ARM_ERRATA_798181 if SMP
+	select SYSC_R8A7792
+
+config ARCH_R8A77980
+	bool "SoC Platform support for R-Car V3H" if ARM64
 	select ARCH_RCAR_GEN3
-	select SYSC_R8A77965
+	select SYSC_R8A77980
 	help
-	  This enables support for the Renesas R-Car M3-N SoC.
+	  This enables support for the Renesas R-Car V3H SoC.
 
 config ARCH_R8A77970
 	bool "SoC Platform support for R-Car V3M" if ARM64
@@ -245,28 +243,39 @@ config ARCH_R8A77970
 	help
 	  This enables support for the Renesas R-Car V3M SoC.
 
-config ARCH_R8A77980
-	bool "SoC Platform support for R-Car V3H" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A77980
-	help
-	  This enables support for the Renesas R-Car V3H SoC.
+config ARCH_R8A7740
+	bool "SoC Platform support for R-Mobile A1" if ARM && ARCH_RENESAS
+	select ARCH_RMOBILE
+	select ARM_ERRATA_754322
+	select RENESAS_INTC_IRQPIN
 
-config ARCH_R8A77990
-	bool "SoC Platform support for R-Car E3" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A77990
-	help
-	  This enables support for the Renesas R-Car E3 SoC.
+config ARCH_R8A73A4
+	bool "SoC Platform support for R-Mobile APE6" if ARM && ARCH_RENESAS
+	select ARCH_RMOBILE
+	select ARM_ERRATA_798181 if SMP
+	select ARM_ERRATA_814220
+	select HAVE_ARM_ARCH_TIMER
+	select RENESAS_IRQC
+
+config ARCH_SH73A0
+	bool "SoC Platform support for SH-Mobile AG5" if ARM && ARCH_RENESAS
+	select ARCH_RMOBILE
+	select ARM_ERRATA_754322
+	select ARM_GLOBAL_TIMER
+	select HAVE_ARM_SCU if SMP
+	select HAVE_ARM_TWD if SMP
+	select RENESAS_INTC_IRQPIN
 
-config ARCH_R8A77995
-	bool "SoC Platform support for R-Car D3" if ARM64
-	select ARCH_RCAR_GEN3
-	select SYSC_R8A77995
-	help
-	  This enables support for the Renesas R-Car D3 SoC.
 
 # SoC
+config SYSC_R8A77470
+	bool "System Controller support for RZ/G1C" if COMPILE_TEST
+	select SYSC_RCAR
+
+config SYSC_R8A7745
+	bool "System Controller support for RZ/G1E" if COMPILE_TEST
+	select SYSC_RCAR
+
 config SYSC_R8A7742
 	bool "System Controller support for RZ/G1H" if COMPILE_TEST
 	select SYSC_RCAR
@@ -275,12 +284,12 @@ config SYSC_R8A7743
 	bool "System Controller support for RZ/G1M" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A7745
-	bool "System Controller support for RZ/G1E" if COMPILE_TEST
+config SYSC_R8A774C0
+	bool "System Controller support for RZ/G2E" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A77470
-	bool "System Controller support for RZ/G1C" if COMPILE_TEST
+config SYSC_R8A774E1
+	bool "System Controller support for RZ/G2H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A774A1
@@ -291,12 +300,19 @@ config SYSC_R8A774B1
 	bool "System Controller support for RZ/G2N" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A774C0
-	bool "System Controller support for RZ/G2E" if COMPILE_TEST
+config SYSC_RCAR
+	bool "System Controller support for R-Car" if COMPILE_TEST
+
+config SYSC_R8A77995
+	bool "System Controller support for R-Car D3" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A774E1
-	bool "System Controller support for RZ/G2H" if COMPILE_TEST
+config SYSC_R8A7794
+	bool "System Controller support for R-Car E2" if COMPILE_TEST
+	select SYSC_RCAR
+
+config SYSC_R8A77990
+	bool "System Controller support for R-Car E3" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A7779
@@ -307,20 +323,16 @@ config SYSC_R8A7790
 	bool "System Controller support for R-Car H2" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A7791
-	bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
-	select SYSC_RCAR
-
-config SYSC_R8A7792
-	bool "System Controller support for R-Car V2H" if COMPILE_TEST
+config SYSC_R8A7795
+	bool "System Controller support for R-Car H3" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A7794
-	bool "System Controller support for R-Car E2" if COMPILE_TEST
+config SYSC_R8A7791
+	bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A7795
-	bool "System Controller support for R-Car H3" if COMPILE_TEST
+config SYSC_R8A77965
+	bool "System Controller support for R-Car M3-N" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77960
@@ -331,34 +343,23 @@ config SYSC_R8A77961
 	bool "System Controller support for R-Car M3-W+" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A77965
-	bool "System Controller support for R-Car M3-N" if COMPILE_TEST
-	select SYSC_RCAR
-
-config SYSC_R8A77970
-	bool "System Controller support for R-Car V3M" if COMPILE_TEST
+config SYSC_R8A7792
+	bool "System Controller support for R-Car V2H" if COMPILE_TEST
 	select SYSC_RCAR
 
 config SYSC_R8A77980
 	bool "System Controller support for R-Car V3H" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A77990
-	bool "System Controller support for R-Car E3" if COMPILE_TEST
+config SYSC_R8A77970
+	bool "System Controller support for R-Car V3M" if COMPILE_TEST
 	select SYSC_RCAR
 
-config SYSC_R8A77995
-	bool "System Controller support for R-Car D3" if COMPILE_TEST
-	select SYSC_RCAR
+config SYSC_RMOBILE
+	bool "System Controller support for R-Mobile" if COMPILE_TEST
 
-# Family
 config RST_RCAR
 	bool "Reset Controller support for R-Car" if COMPILE_TEST
 
-config SYSC_RCAR
-	bool "System Controller support for R-Car" if COMPILE_TEST
-
-config SYSC_RMOBILE
-	bool "System Controller support for R-Mobile" if COMPILE_TEST
 
 endif # SOC_RENESAS
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/4] soc: renesas: align driver description title
  2020-08-24  2:22 ` [PATCH v3 2/4] soc: renesas: align driver description title Kuninori Morimoto
@ 2020-09-01  7:46   ` Geert Uytterhoeven
  0 siblings, 0 replies; 9+ messages in thread
From: Geert Uytterhoeven @ 2020-09-01  7:46 UTC (permalink / raw)
  To: Kuninori Morimoto; +Cc: Magnus, Linux-Renesas

On Mon, Aug 24, 2020 at 4:22 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> Now, Renesas SoC drivers are under menu,
> but current description are not aligned.
> This patch align these.
>
>         - R-Car H2 System Controller support
>         - R-Car M2-W/N System Controller support
>         - R-Car V2H System Controller support
>         - R-Car E2 System Controller support
>         - R-Car H3 System Controller support
>         - R-Car M3-W System Controller support
>         - R-Car M3-W+ System Controller support
>         - R-Car M3-N System Controller support
>
>         + System Controller support for R-Car H2
>         + System Controller support for R-Car M2-W/N
>         + System Controller support for R-Car V2H
>         + System Controller support for R-Car E2
>         + System Controller support for R-Car H3
>         + System Controller support for R-Car M3-W
>         + System Controller support for R-Car M3-W+
>         + System Controller support for R-Car M3-N
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Will queue in renesas-devel for v5.10.

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] 9+ messages in thread
* Re: [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description
  2020-08-24  2:22 ` [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description Kuninori Morimoto
@ 2020-09-01  7:55   ` Geert Uytterhoeven
  2020-09-04  0:03     ` Kuninori Morimoto
  0 siblings, 1 reply; 9+ messages in thread
From: Geert Uytterhoeven @ 2020-09-01  7:55 UTC (permalink / raw)
  To: Kuninori Morimoto; +Cc: Magnus, Linux-Renesas

Hi Morimoto-san,

On Mon, Aug 24, 2020 at 4:22 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> Current Kconfig is using if - endif.
> This patch merges it into each menu description.
> This is prepare for menu sorting.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thanks for your patch!

> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -44,17 +44,15 @@ config ARCH_RZN1
>         bool
>         select ARM_AMBA
>
> -if ARM && ARCH_RENESAS
> -
>  #comment "Renesas ARM SoCs System Type"
>
>  config ARCH_EMEV2
> -       bool "SoC Platform support for Emma Mobile EV2"
> +       bool "SoC Platform support for Emma Mobile EV2" if ARM && ARCH_RENESAS
>         select HAVE_ARM_SCU if SMP
>         select SYS_SUPPORTS_EM_STI

I'm not convinced duplicating these dependencies is better than a
big #if block.  It does allow you to sort all SoCs alphabetically, but that
mixes ARM32 and ARM64 parts, while choosing between ARM32 and ARM64
is an early and important decision (different ARCH= and CROSS_COMPILE=,
too).
In addition, only one set of Kconfig options is shown to the user
anyway.

What do other people think?

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] 9+ messages in thread
* Re: [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description
  2020-09-01  7:55   ` Geert Uytterhoeven
@ 2020-09-04  0:03     ` Kuninori Morimoto
  2020-09-04  7:14       ` Geert Uytterhoeven
  0 siblings, 1 reply; 9+ messages in thread
From: Kuninori Morimoto @ 2020-09-04  0:03 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus, Linux-Renesas


Hi Geert

> >  config ARCH_EMEV2
> > -       bool "SoC Platform support for Emma Mobile EV2"
> > +       bool "SoC Platform support for Emma Mobile EV2" if ARM && ARCH_RENESAS
> >         select HAVE_ARM_SCU if SMP
> >         select SYS_SUPPORTS_EM_STI
> 
> I'm not convinced duplicating these dependencies is better than a
> big #if block.  It does allow you to sort all SoCs alphabetically, but that
> mixes ARM32 and ARM64 parts, while choosing between ARM32 and ARM64
> is an early and important decision (different ARCH= and CROSS_COMPILE=,
> too).
> In addition, only one set of Kconfig options is shown to the user
> anyway.

OK, so let's keep big #if block.
So how about to having
	"ARM32 platform support ..." and
	"ARM64 platform support" ?
We can keep both #if block and alphabetical order

Thank you for your help !!

Best regards
---
Kuninori Morimoto

^ permalink raw reply	[flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description
  2020-09-04  0:03     ` Kuninori Morimoto
@ 2020-09-04  7:14       ` Geert Uytterhoeven
  0 siblings, 0 replies; 9+ messages in thread
From: Geert Uytterhoeven @ 2020-09-04  7:14 UTC (permalink / raw)
  To: Kuninori Morimoto; +Cc: Magnus, Linux-Renesas

Hi Morimoto-san,

On Fri, Sep 4, 2020 at 2:03 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> > >  config ARCH_EMEV2
> > > -       bool "SoC Platform support for Emma Mobile EV2"
> > > +       bool "SoC Platform support for Emma Mobile EV2" if ARM && ARCH_RENESAS
> > >         select HAVE_ARM_SCU if SMP
> > >         select SYS_SUPPORTS_EM_STI
> >
> > I'm not convinced duplicating these dependencies is better than a
> > big #if block.  It does allow you to sort all SoCs alphabetically, but that
> > mixes ARM32 and ARM64 parts, while choosing between ARM32 and ARM64
> > is an early and important decision (different ARCH= and CROSS_COMPILE=,
> > too).
> > In addition, only one set of Kconfig options is shown to the user
> > anyway.
>
> OK, so let's keep big #if block.
> So how about to having
>         "ARM32 platform support ..." and
>         "ARM64 platform support" ?
> We can keep both #if block and alphabetical order

Yes, that looks fine to me. Thanks!

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] 9+ messages in thread
end of thread, other threads:[~2020-09-04  7:14 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-24  2:20 [PATCH v3 0/4] soc: renesas: use menu for Renesas SoC Kuninori Morimoto
2020-08-24  2:21 ` [PATCH v3 1/4] " Kuninori Morimoto
2020-08-24  2:22 ` [PATCH v3 2/4] soc: renesas: align driver description title Kuninori Morimoto
2020-09-01  7:46   ` Geert Uytterhoeven
2020-08-24  2:22 ` [PATCH v3 3/4] soc: renesas: Merge if xxx into menu description Kuninori Morimoto
2020-09-01  7:55   ` Geert Uytterhoeven
2020-09-04  0:03     ` Kuninori Morimoto
2020-09-04  7:14       ` Geert Uytterhoeven
2020-08-24  2:22 ` [PATCH v3 4/4] soc: renesas: sort driver description title Kuninori Morimoto

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.