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

From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Wolfram Sang <wsa+renesas@sang-engineering.com>
Cc: linux-renesas-soc@vger.kernel.org,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	Magnus Damm <magnus.damm@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 5/5] soc: renesas: remove r8a77950 arch
Date: Wed, 8 Mar 2023 10:57:54 +0100	[thread overview]
Message-ID: <CAMuHMdXr=WGTEOnAhckPW4B1WUoRZUokkRjEZXjsU+Qr-y60iQ@mail.gmail.com> (raw)
In-Reply-To: <20230307105645.5285-6-wsa+renesas@sang-engineering.com>

Hi Wolfram,

On Tue, Mar 7, 2023 at 11:57 AM Wolfram Sang
<wsa+renesas@sang-engineering.com> 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@ideasonboard.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

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@linux-m68k.org

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