Hi Prabhakar,
On Fri, Jul 22, 2022 at 12:12 AM Lad Prabhakar
<prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote:
> Add support for identifying the (R9A07G043) RZ/Five SoC.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
Thanks for your patch!
> --- a/drivers/soc/renesas/Kconfig
> +++ b/drivers/soc/renesas/Kconfig
> @@ -333,6 +333,16 @@ config ARCH_R9A09G011
>
> endif # ARM64
>
> +if RISCV
> +
> +config ARCH_R9A07G043
> + bool "RISCV Platform support for RZ/Five"
RISC-V
> + select ARCH_RZG2L
> + help
> + This enables support for the Renesas RZ/Five SoC.
> +
> +endif # RISCV
> +
> config RST_RCAR
> bool "Reset Controller support for R-Car" if COMPILE_TEST
>
> @@ -102,6 +106,11 @@ static const struct renesas_soc soc_rmobile_a1 __initconst __maybe_unused = {
> .id = 0x40,
> };
>
> +static const struct renesas_soc soc_rz_five __initconst __maybe_unused = {
> + .family = &fam_rzfive,
> + .id = 0x847c447,
The Hardware User's Manual Rev.1.00 says H’x841_C447?
> @@ -398,6 +411,7 @@ static const struct renesas_id id_prr __initconst = {
>
> static const struct of_device_id renesas_ids[] __initconst = {
> { .compatible = "renesas,bsid", .data = &id_bsid },
> + { .compatible = "renesas,r9a07g043-rzfive-sysc", .data = &id_rzg2l },
> { .compatible = "renesas,r9a07g043-sysc", .data = &id_rzg2l },
> { .compatible = "renesas,r9a07g044-sysc", .data = &id_rzg2l },
> { .compatible = "renesas,r9a07g054-sysc", .data = &id_rzg2l },
This change might not be needed (cfr. my comments on patch 1/2).
The rest LGTM.
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
