Re: [PATCH 5/5] soc: renesas: remove r8a77950 arch

[Geert Uytterhoeven]

Hi Wolfram,

On Tue, Mar 7, 2023 at 11:57 AM Wolfram Sang
<wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote:
> R-Car H3 ES1.* was only available to an internal development group and
> needed a lot of quirks and workarounds. These become a maintenance
> burden now, so our development group decided to remove upstream support
> and disable booting for this SoC. Public users only have ES2 onwards.
>
> Remove the architecture which describes H3 ES1.*.
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>

Thanks for your patch!

> ---
>  arch/arm64/configs/defconfig      | 1 -
>  drivers/soc/renesas/Kconfig       | 7 -------
>  drivers/soc/renesas/renesas-soc.c | 2 +-

I will split this in two patches (defconfig and drivers/soc)...

> --- a/drivers/soc/renesas/renesas-soc.c
> +++ b/drivers/soc/renesas/renesas-soc.c
> @@ -330,7 +330,7 @@ static const struct of_device_id renesas_socs[] __initconst = {
>  #ifdef CONFIG_ARCH_R8A7794
>         { .compatible = "renesas,r8a7794",      .data = &soc_rcar_e2 },
>  #endif
> -#if defined(CONFIG_ARCH_R8A77950) || defined(CONFIG_ARCH_R8A77951)
> +#if defined(CONFIG_ARCH_R8A77951)
>         { .compatible = "renesas,r8a7795",      .data = &soc_rcar_h3 },

... and merge this with the section below ...

>  #endif
>  #ifdef CONFIG_ARCH_R8A77951

... while queuing in renesas-devel for v6.4.

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