diff mbox series

[v2,3/3] soc: renesas: sort driver description title

Message ID 87h7t1ho6f.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show
Series soc: renesas: use menu for Renesas SoC | expand

Commit Message

Kuninori Morimoto Aug. 17, 2020, 4:51 a.m. UTC 
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Some Renesas drivers are not organized, or organized by Chip number.
Because of it, menu table is not readable.
This patch sort these.

This patch do
	- Collect RZ/xx in one place
	- Collect SH-Mobile xx in one place
	- align chip number on each groups

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

Comments

Geert Uytterhoeven Aug. 21, 2020, 2:25 p.m. UTC | #1 
Hi Morimoto-san,

Thanks for your patch!

On Mon, Aug 17, 2020 at 6:51 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> Some Renesas drivers are not organized, or organized by Chip number.

They are sorted by SoC part number?

> Because of it, menu table is not readable.
> This patch sort these.
>
> This patch do
>         - Collect RZ/xx in one place
>         - Collect SH-Mobile xx in one place
>         - align chip number on each groups
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

So now we have:

    $ grep 'bool "' drivers/soc/renesas/Kconfig  | sed -e 's/   /        /g'
        bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
        bool "Emma Mobile EV2"
        bool "SH-Mobile AG5 (R8A73A00)"
        bool "R-Mobile APE6 (R8A73A40)"
        bool "R-Mobile A1   (R8A77400)"
        bool "RZ/A1H (R7S72100)"
        bool "RZ/A2  (R7S9210)"
        bool "RZ/G1H (R8A77420)"
        bool "RZ/G1M (R8A77430)"
        bool "RZ/G1N (R8A77440)"
        bool "RZ/G1E (R8A77450)"
        bool "RZ/G1C (R8A77470)"
        bool "RZ/N1D (R9A06G032)"
        bool "R-Car M1A  (R8A77781)"
        bool "R-Car H1   (R8A77790)"
        bool "R-Car H2   (R8A77900)"
        bool "R-Car M2-W (R8A77910)"
        bool "R-Car V2H  (R8A77920)"
        bool "R-Car M2-N (R8A7793)"
        bool "R-Car E2   (R8A77940)"

What about changing all the above to match the below?
I.e. "SoC Platform support for <foo>"?

        bool "SoC Platform support for RZ/G2M"
        bool "SoC Platform support for RZ/G2N"
        bool "SoC Platform support for RZ/G2E"
        bool "SoC Platform support for RZ/G2H"
        bool "SoC Platform support for R-Car H3 ES1.x"
        bool "SoC Platform support for R-Car H3 ES2.0+"
        bool "SoC Platform support for R-Car M3-W"
        bool "SoC Platform support for R-Car M3-W+"
        bool "SoC Platform support for R-Car M3-N"
        bool "SoC Platform support for R-Car V3M"
        bool "SoC Platform support for R-Car V3H"
        bool "SoC Platform support for R-Car E3"
        bool "SoC Platform support for R-Car D3"
        bool "System Controller support for RZ/G1H" if COMPILE_TEST
        bool "System Controller support for RZ/G1M" if COMPILE_TEST
        bool "System Controller support for RZ/G1E" if COMPILE_TEST
        bool "System Controller support for RZ/G1C" if COMPILE_TEST
        bool "System Controller support for RZ/G2M" if COMPILE_TEST
        bool "System Controller support for RZ/G2N" if COMPILE_TEST
        bool "System Controller support for RZ/G2E" if COMPILE_TEST
        bool "System Controller support for RZ/G2H" if COMPILE_TEST
        bool "System Controller support for R-Car H1" if COMPILE_TEST
        bool "System Controller support for R-Car H2" if COMPILE_TEST
        bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
        bool "System Controller support for R-Car V2H" if COMPILE_TEST
        bool "System Controller support for R-Car E2" if COMPILE_TEST
        bool "System Controller support for R-Car H3" if COMPILE_TEST
        bool "System Controller support for R-Car M3-W" if COMPILE_TEST
        bool "System Controller support for R-Car M3-W+" if COMPILE_TEST
        bool "System Controller support for R-Car M3-N" if COMPILE_TEST
        bool "System Controller support for R-Car V3M" if COMPILE_TEST
        bool "System Controller support for R-Car V3H" if COMPILE_TEST
        bool "System Controller support for R-Car E3" if COMPILE_TEST
        bool "System Controller support for R-Car D3" if COMPILE_TEST
        bool "Reset  Controller support for R-Car" if COMPILE_TEST
        bool "System Controller support for R-Car" if COMPILE_TEST
        bool "System Controller support for R-Mobile" if COMPILE_TEST



Gr{oetje,eeting}s,

                        Geert
Kuninori Morimoto Aug. 24, 2020, 12:02 a.m. UTC | #2 
Hi Geert

Thank you for your review

>         bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS
>         bool "Emma Mobile EV2"
>         bool "SH-Mobile AG5 (R8A73A00)"
>         bool "R-Mobile APE6 (R8A73A40)"
>         bool "R-Mobile A1   (R8A77400)"
>         bool "RZ/A1H (R7S72100)"
>         bool "RZ/A2  (R7S9210)"
>         bool "RZ/G1H (R8A77420)"
>         bool "RZ/G1M (R8A77430)"
>         bool "RZ/G1N (R8A77440)"
>         bool "RZ/G1E (R8A77450)"
>         bool "RZ/G1C (R8A77470)"
>         bool "RZ/N1D (R9A06G032)"
>         bool "R-Car M1A  (R8A77781)"
>         bool "R-Car H1   (R8A77790)"
>         bool "R-Car H2   (R8A77900)"
>         bool "R-Car M2-W (R8A77910)"
>         bool "R-Car V2H  (R8A77920)"
>         bool "R-Car M2-N (R8A7793)"
>         bool "R-Car E2   (R8A77940)"
(snip)
>         bool "SoC Platform support for RZ/G2M"
>         bool "SoC Platform support for RZ/G2N"
>         bool "SoC Platform support for RZ/G2E"
>         bool "SoC Platform support for RZ/G2H"
>         bool "SoC Platform support for R-Car H3 ES1.x"
>         bool "SoC Platform support for R-Car H3 ES2.0+"
>         bool "SoC Platform support for R-Car M3-W"
>         bool "SoC Platform support for R-Car M3-W+"
>         bool "SoC Platform support for R-Car M3-N"
>         bool "SoC Platform support for R-Car V3M"
>         bool "SoC Platform support for R-Car V3H"
>         bool "SoC Platform support for R-Car E3"
>         bool "SoC Platform support for R-Car D3"
>         bool "System Controller support for RZ/G1H" if COMPILE_TEST
>         bool "System Controller support for RZ/G1M" if COMPILE_TEST
>         bool "System Controller support for RZ/G1E" if COMPILE_TEST
>         bool "System Controller support for RZ/G1C" if COMPILE_TEST
>         bool "System Controller support for RZ/G2M" if COMPILE_TEST
>         bool "System Controller support for RZ/G2N" if COMPILE_TEST
>         bool "System Controller support for RZ/G2E" if COMPILE_TEST
>         bool "System Controller support for RZ/G2H" if COMPILE_TEST
>         bool "System Controller support for R-Car H1" if COMPILE_TEST
>         bool "System Controller support for R-Car H2" if COMPILE_TEST
>         bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST
>         bool "System Controller support for R-Car V2H" if COMPILE_TEST
>         bool "System Controller support for R-Car E2" if COMPILE_TEST
>         bool "System Controller support for R-Car H3" if COMPILE_TEST
>         bool "System Controller support for R-Car M3-W" if COMPILE_TEST
>         bool "System Controller support for R-Car M3-W+" if COMPILE_TEST
>         bool "System Controller support for R-Car M3-N" if COMPILE_TEST
>         bool "System Controller support for R-Car V3M" if COMPILE_TEST
>         bool "System Controller support for R-Car V3H" if COMPILE_TEST
>         bool "System Controller support for R-Car E3" if COMPILE_TEST
>         bool "System Controller support for R-Car D3" if COMPILE_TEST
>         bool "Reset  Controller support for R-Car" if COMPILE_TEST
>         bool "System Controller support for R-Car" if COMPILE_TEST
>         bool "System Controller support for R-Mobile" if COMPILE_TEST
(snip)
> What about changing all the above to match the below?
> I.e. "SoC Platform support for <foo>"?

It sounds good for me. Will do in v3.

And can we rename/merge "SoC Platform support" and "System Controller support" ?
These are unser "Renesas SoC driver support" menu, so just "support for xxx" ?

Thank you for your help !!

Best regards
---
Kuninori Morimoto
Kuninori Morimoto Aug. 24, 2020, 1:16 a.m. UTC | #3 
Hi Geert, again

> And can we rename/merge "SoC Platform support" and "System Controller support" ?
> These are unser "Renesas SoC driver support" menu, so just "support for xxx" ?

Ahh, we can't do this...

Thank you for your help !!

Best regards
---
Kuninori Morimoto
diff mbox series

Patch

diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index bd9a9ab10d21..d69d02a65f0a 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -53,21 +53,14 @@  config ARCH_EMEV2
 	select HAVE_ARM_SCU if SMP
 	select SYS_SUPPORTS_EM_STI
 
-config ARCH_R7S72100
-	bool "RZ/A1H (R7S72100)"
+config ARCH_SH73A0
+	bool "SH-Mobile AG5 (R8A73A00)"
+	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
-	select PM
-	select PM_GENERIC_DOMAINS
-	select RENESAS_OSTM
-	select RENESAS_RZA1_IRQC
-	select SYS_SUPPORTS_SH_MTU2
-
-config ARCH_R7S9210
-	bool "RZ/A2 (R7S9210)"
-	select PM
-	select PM_GENERIC_DOMAINS
-	select RENESAS_OSTM
-	select RENESAS_RZA1_IRQC
+	select ARM_GLOBAL_TIMER
+	select HAVE_ARM_SCU if SMP
+	select HAVE_ARM_TWD if SMP
+	select RENESAS_INTC_IRQPIN
 
 config ARCH_R8A73A4
 	bool "R-Mobile APE6 (R8A73A40)"
@@ -78,11 +71,27 @@  config ARCH_R8A73A4
 	select RENESAS_IRQC
 
 config ARCH_R8A7740
-	bool "R-Mobile A1 (R8A77400)"
+	bool "R-Mobile A1   (R8A77400)"
 	select ARCH_RMOBILE
 	select ARM_ERRATA_754322
 	select RENESAS_INTC_IRQPIN
 
+config ARCH_R7S72100
+	bool "RZ/A1H (R7S72100)"
+	select ARM_ERRATA_754322
+	select PM
+	select PM_GENERIC_DOMAINS
+	select RENESAS_OSTM
+	select RENESAS_RZA1_IRQC
+	select SYS_SUPPORTS_SH_MTU2
+
+config ARCH_R7S9210
+	bool "RZ/A2  (R7S9210)"
+	select PM
+	select PM_GENERIC_DOMAINS
+	select RENESAS_OSTM
+	select RENESAS_RZA1_IRQC
+
 config ARCH_R8A7742
 	bool "RZ/G1H (R8A77420)"
 	select ARCH_RCAR_GEN2
@@ -114,13 +123,18 @@  config ARCH_R8A77470
 	select ARM_ERRATA_814220
 	select SYSC_R8A77470
 
+config ARCH_R9A06G032
+	bool "RZ/N1D (R9A06G032)"
+	select ARCH_RZN1
+	select ARM_ERRATA_814220
+
 config ARCH_R8A7778
-	bool "R-Car M1A (R8A77781)"
+	bool "R-Car M1A  (R8A77781)"
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 
 config ARCH_R8A7779
-	bool "R-Car H1 (R8A77790)"
+	bool "R-Car H1   (R8A77790)"
 	select ARCH_RCAR_GEN1
 	select ARM_ERRATA_754322
 	select ARM_GLOBAL_TIMER
@@ -129,7 +143,7 @@  config ARCH_R8A7779
 	select SYSC_R8A7779
 
 config ARCH_R8A7790
-	bool "R-Car H2 (R8A77900)"
+	bool "R-Car H2   (R8A77900)"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select ARM_ERRATA_814220
@@ -144,7 +158,7 @@  config ARCH_R8A7791
 	select SYSC_R8A7791
 
 config ARCH_R8A7792
-	bool "R-Car V2H (R8A77920)"
+	bool "R-Car V2H  (R8A77920)"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_798181 if SMP
 	select SYSC_R8A7792
@@ -157,25 +171,11 @@  config ARCH_R8A7793
 	select SYSC_R8A7791
 
 config ARCH_R8A7794
-	bool "R-Car E2 (R8A77940)"
+	bool "R-Car E2   (R8A77940)"
 	select ARCH_RCAR_GEN2
 	select ARM_ERRATA_814220
 	select SYSC_R8A7794
 
-config ARCH_R9A06G032
-	bool "RZ/N1D (R9A06G032)"
-	select ARCH_RZN1
-	select ARM_ERRATA_814220
-
-config ARCH_SH73A0
-	bool "SH-Mobile AG5 (R8A73A00)"
-	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
-
 endif # ARM
 
 if ARM64