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