[PATCH v2 0/2] Share RZ/G2L SoC DTSI with RZ/V2L SoC

Prabhakar posted 2 patches 2 years, 7 months ago
arch/arm64/boot/dts/renesas/r9a07g044.dtsi |  254 ++---
arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 1149 +++-----------------
2 files changed, 256 insertions(+), 1147 deletions(-)
[PATCH v2 0/2] Share RZ/G2L SoC DTSI with RZ/V2L SoC
Posted by Prabhakar 2 years, 7 months ago
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Hi All,

This series aims to reuse RZ/G2L SoC DTSI with RZ/V2L as both the SoCs are
almost identical.

v1 -> v2
* Patch 1/2 unchanged, for patch 2/2 sorted the nodes based on the names.

v1: https://patchwork.kernel.org/project/linux-renesas-soc/cover/20230127133909.144774-1-prabhakar.mahadev-lad.rj@bp.renesas.com/

Cheers,
Prabhakar

Lad Prabhakar (2):
  arm64: dts: renesas: r9a07g044: Use SoC specific macro for CPG and
    RESET
  arm64: dts: renesas: r9a07g054: Reuse RZ/G2L SoC DTSI

 arch/arm64/boot/dts/renesas/r9a07g044.dtsi |  254 ++---
 arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 1149 +++-----------------
 2 files changed, 256 insertions(+), 1147 deletions(-)

-- 
2.25.1
Re: [PATCH v2 0/2] Share RZ/G2L SoC DTSI with RZ/V2L SoC
Posted by Geert Uytterhoeven 2 years, 7 months ago
Hi Rob, Krzysztof,

On Tue, Jan 31, 2023 at 11:42 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> This series aims to reuse RZ/G2L SoC DTSI with RZ/V2L as both the SoCs are
> almost identical.
>
> v1 -> v2
> * Patch 1/2 unchanged, for patch 2/2 sorted the nodes based on the names.
>
> v1: https://patchwork.kernel.org/project/linux-renesas-soc/cover/20230127133909.144774-1-prabhakar.mahadev-lad.rj@bp.renesas.com/
>
> Cheers,
> Prabhakar
>
> Lad Prabhakar (2):
>   arm64: dts: renesas: r9a07g044: Use SoC specific macro for CPG and
>     RESET
>   arm64: dts: renesas: r9a07g054: Reuse RZ/G2L SoC DTSI
>
>  arch/arm64/boot/dts/renesas/r9a07g044.dtsi |  254 ++---
>  arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 1149 +++-----------------
>  2 files changed, 256 insertions(+), 1147 deletions(-)

Do you have an opinion on this series?
Is this acceptable for you?
The final generated DTBs are identical to before.

Thanks!

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
Re: [PATCH v2 0/2] Share RZ/G2L SoC DTSI with RZ/V2L SoC
Posted by Geert Uytterhoeven 2 years, 6 months ago
Hi Prabhakar,

On Mon, Feb 13, 2023 at 3:16 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Tue, Jan 31, 2023 at 11:42 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> > This series aims to reuse RZ/G2L SoC DTSI with RZ/V2L as both the SoCs are
> > almost identical.
> >
> > v1 -> v2
> > * Patch 1/2 unchanged, for patch 2/2 sorted the nodes based on the names.
> >
> > v1: https://patchwork.kernel.org/project/linux-renesas-soc/cover/20230127133909.144774-1-prabhakar.mahadev-lad.rj@bp.renesas.com/
> >
> > Cheers,
> > Prabhakar
> >
> > Lad Prabhakar (2):
> >   arm64: dts: renesas: r9a07g044: Use SoC specific macro for CPG and
> >     RESET
> >   arm64: dts: renesas: r9a07g054: Reuse RZ/G2L SoC DTSI
> >
> >  arch/arm64/boot/dts/renesas/r9a07g044.dtsi |  254 ++---
> >  arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 1149 +++-----------------
> >  2 files changed, 256 insertions(+), 1147 deletions(-)
>
> Do you have an opinion on this series?
> Is this acceptable for you?
> The final generated DTBs are identical to before.

Given the feedback from Krzysztof and Rob on patch 1, and on IRC,
I'm rejecting this series.

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
Re: [PATCH v2 0/2] Share RZ/G2L SoC DTSI with RZ/V2L SoC
Posted by Lad, Prabhakar 2 years, 6 months ago
Hi Geert,

On Fri, Mar 10, 2023 at 10:41 AM Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
>
> Hi Prabhakar,
>
> On Mon, Feb 13, 2023 at 3:16 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Tue, Jan 31, 2023 at 11:42 PM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> > > This series aims to reuse RZ/G2L SoC DTSI with RZ/V2L as both the SoCs are
> > > almost identical.
> > >
> > > v1 -> v2
> > > * Patch 1/2 unchanged, for patch 2/2 sorted the nodes based on the names.
> > >
> > > v1: https://patchwork.kernel.org/project/linux-renesas-soc/cover/20230127133909.144774-1-prabhakar.mahadev-lad.rj@bp.renesas.com/
> > >
> > > Cheers,
> > > Prabhakar
> > >
> > > Lad Prabhakar (2):
> > >   arm64: dts: renesas: r9a07g044: Use SoC specific macro for CPG and
> > >     RESET
> > >   arm64: dts: renesas: r9a07g054: Reuse RZ/G2L SoC DTSI
> > >
> > >  arch/arm64/boot/dts/renesas/r9a07g044.dtsi |  254 ++---
> > >  arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 1149 +++-----------------
> > >  2 files changed, 256 insertions(+), 1147 deletions(-)
> >
> > Do you have an opinion on this series?
> > Is this acceptable for you?
> > The final generated DTBs are identical to before.
>
> Given the feedback from Krzysztof and Rob on patch 1, and on IRC,
> I'm rejecting this series.
>
Ok, I'll rebase CRU work by dropping this series.

Cheers,
Prabhakar