Hi Shimoda-san, On Tue, Nov 16, 2021 at 8:42 AM Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: > Add support for identifying the R-Car S4-8 (R8A779F0) SoC. > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> Thanks for your patch! > --- a/drivers/soc/renesas/Kconfig > +++ b/drivers/soc/renesas/Kconfig > @@ -256,6 +256,13 @@ config ARCH_R8A779A0 > help > This enables support for the Renesas R-Car V3U SoC. > > +config ARCH_R8A779F0 > + bool "ARM64 Platform support for R-Car S4-8" > + select ARCH_RCAR_GEN3 > + select SYSC_R8A779F0 "SYSC_R8A779F0" is only defined in [PATCH 10/16], so you may want to reorder your series. Or I will while applying later ;-) > + help > + This enables support for the Renesas R-Car S4-8 SoC. > + > config ARCH_R8A774C0 > bool "ARM64 Platform support for RZ/G2E" > select ARCH_RCAR_GEN3 > diff --git a/drivers/soc/renesas/renesas-soc.c b/drivers/soc/renesas/renesas-soc.c > index 7961b0be1850..857a42a82747 100644 > --- a/drivers/soc/renesas/renesas-soc.c > +++ b/drivers/soc/renesas/renesas-soc.c > @@ -33,6 +33,11 @@ static const struct renesas_family fam_rcar_gen3 __initconst __maybe_unused = { > .reg = 0xfff00044, /* PRR (Product Register) */ > }; > > +static const struct renesas_family fam_rcar_gen4 __initconst __maybe_unused = { > + .name = "R-Car Gen4", > + .reg = 0xfff00044, /* PRR (Product Register) */ > +}; > + > static const struct renesas_family fam_rmobile __initconst __maybe_unused = { > .name = "R-Mobile", > .reg = 0xe600101c, /* CCCR (Common Chip Code Register) */ > @@ -214,6 +219,11 @@ static const struct renesas_soc soc_rcar_v3u __initconst __maybe_unused = { > .id = 0x59, > }; > > +static const struct renesas_soc soc_rcar_s4 __initconst __maybe_unused = { > + .family = &fam_rcar_gen4, > + .id = 0x5a, > +}; > + > static const struct renesas_soc soc_shmobile_ag5 __initconst __maybe_unused = { > .family = &fam_shmobile, > .id = 0x37, > @@ -319,6 +329,9 @@ static const struct of_device_id renesas_socs[] __initconst = { > #ifdef CONFIG_ARCH_R8A779A0 > { .compatible = "renesas,r8a779a0", .data = &soc_rcar_v3u }, > #endif > +#ifdef CONFIG_ARCH_R8A779F0 > + { .compatible = "renesas,r8a779f0", .data = &soc_rcar_s4 }, > +#endif > #if defined(CONFIG_ARCH_R9A07G044) > { .compatible = "renesas,r9a07g044", .data = &soc_rz_g2l }, > #endif > -- > 2.25.1 > -- Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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