Hi Shimoda-san, On Tue, Nov 16, 2021 at 8:42 AM Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: > Add support for R-Car S4-8 (R8A779F0) to the R-Car RST driver. > The register map of R-Car S4-8 is the same as R-Car V3U so that > renames "V3U" and "r8a779a0" with "Gen4". > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> Thanks for your patch! > --- a/drivers/soc/renesas/rcar-rst.c > +++ b/drivers/soc/renesas/rcar-rst.c > @@ -12,7 +12,7 @@ > > #define WDTRSTCR_RESET 0xA55A0002 > #define WDTRSTCR 0x0054 > -#define V3U_WDTRSTCR 0x0010 > +#define GEN4_WDTRSTCR 0x0010 V3U_WDTRSTCR handling is not present upstream, as it should be handled by the boot loader (ATF?), like on other R-Car Gen3 SoCs. Likewise, GEN4_WDTRSTCR should not become present upstream. So please split this in two patches: 1. A patch against upstream, just adding basic R-Car S4-8 support, 2. An optional second patch to enable GEN4_WDTRSTCR in renesas-drivers, to serve as an interim solution until the bootloader is fixed. 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