Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi

[Niklas Söderlund]

Hello,

On 2025-12-13 15:20:37 +0900, Laurent Pinchart wrote:
> Hi Rob,
> 
> CC'ing Jacopo.
> 
> On Fri, Dec 12, 2025 at 02:32:07PM -0600, Rob Herring (Arm) wrote:
> > These .dtsi files are not included anywhere in the tree and can't be
> > tested.
> > 
> > Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> > ---
> >  .../renesas/gr-peach-audiocamerashield.dtsi   |  75 ----
> 
> Jacopo, any opinion on this ? Should it be conevrted to an overlay, or
> dropped ?
> 
> >  .../dts/renesas/r8a77xx-aa121td01-panel.dtsi  |  39 --
> 
> I don't have access to this panel any more. I'm fine dropping this
> .dtsi, it can always be resurrected and converted to an overlay if
> someone needs it.
> 
> >  arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi | 332 ------------------
> 
> This would be nice to turn into an overlay, but I'm not sure anyone
> still cares.

I started to write a reply saying I care about this one for V3M and that 
it should be converted to a .dtso. I then looked at the last time I used 
it, ~2 years ago, and the next ting I would need it for is to play with 
the GMSL framework being worked on by Cosmin and streams whenever that 
is merged. I also looked at the file itself and there is some work 
needed split it into more useful .dtso structure.

Inertia hit and I deleted the reply thinking I will do a new .dtso when 
I test the above as the verification of a new overlay alone will require 
the same setup and verification work.

tl;dr; I care about the function, I'm OK with removing the .dtsi; I will 
recreate the functionality as .dtso files in future.

> 
> >  arch/arm64/boot/dts/renesas/r8a779m0.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r8a779m2.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r8a779m4.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r8a779m6.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r8a779m7.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r8a779m8.dtsi     |  17 -
> >  arch/arm64/boot/dts/renesas/r8a779mb.dtsi     |  12 -
> >  arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi  |  25 --
> >  arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi  |  18 -
> >  arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi  |  18 -
> >  arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi |  18 -
> 
> I'll let Geert comment on this :-)
> 
> >  14 files changed, 614 deletions(-)
> >  delete mode 100644 arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
> >  delete mode 100644 arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m0.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m2.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m4.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m6.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m7.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779m8.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r8a779mb.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
> >  delete mode 100644 arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
> 
> -- 
> Regards,
> 
> Laurent Pinchart
> 

-- 
Kind Regards,
Niklas Söderlund