mbox series

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

Message ID 20230131223529.11905-1-prabhakar.mahadev-lad.rj@bp.renesas.com
Headers
Series Share RZ/G2L SoC DTSI with RZ/V2L SoC |

Message

Lad, Prabhakar Jan. 31, 2023, 10:35 p.m. UTC 
  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(-)

Comments

Geert Uytterhoeven Feb. 13, 2023, 2:16 p.m. UTC | #1 
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 March 10, 2023, 10:41 a.m. UTC | #2 
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
Lad, Prabhakar March 12, 2023, 8:07 p.m. UTC | #3 
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