| 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 |
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
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
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 --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