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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20251213071203.GA681376@ragnatech.se>
Date: Sat, 13 Dec 2025 08:12:03 +0100
From: Niklas Söderlund <niklas.soderlund@...natech.se>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
Cc: "Rob Herring (Arm)" <robh@...nel.org>,
	Geert Uytterhoeven <geert+renesas@...der.be>,
	Magnus Damm <magnus.damm@...il.com>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Jacopo Mondi <jacopo.mondi@...asonboard.com>
Subject: Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi

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@...nel.org>
> > ---
> >  .../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