linux-renesas-soc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
@ 2025-12-12 20:32 Rob Herring (Arm)
  2025-12-13  6:20 ` Laurent Pinchart
  2025-12-14  7:03 ` Chris Paterson
  0 siblings, 2 replies; 8+ messages in thread
From: Rob Herring (Arm) @ 2025-12-12 20:32 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-renesas-soc, devicetree, linux-kernel

These .dtsi files are not included anywhere in the tree and can't be
tested.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../renesas/gr-peach-audiocamerashield.dtsi   |  75 ----
 .../dts/renesas/r8a77xx-aa121td01-panel.dtsi  |  39 --
 arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi | 332 ------------------
 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 -
 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

diff --git a/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi b/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
deleted file mode 100644
index 8d77579807ec..000000000000
--- a/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
+++ /dev/null
@@ -1,75 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-/*
- * Device Tree Source for the GR-Peach audiocamera shield expansion board
- *
- * Copyright (C) 2017 Jacopo Mondi <jacopo+renesas@jmondi.org>
- */
-
-#include "r7s72100.dtsi"
-#include <dt-bindings/gpio/gpio.h>
-#include <dt-bindings/pinctrl/r7s72100-pinctrl.h>
-
-/ {
-	/* On-board camera clock. */
-	camera_clk: camera_clk {
-		compatible = "fixed-clock";
-		#clock-cells = <0>;
-		clock-frequency = <27000000>;
-	};
-};
-
-&pinctrl {
-	i2c1_pins: i2c1 {
-		/* P1_2 as SCL; P1_3 as SDA */
-		pinmux = <RZA1_PINMUX(1, 2, 1)>, <RZA1_PINMUX(1, 3, 1)>;
-	};
-
-	vio_pins: vio {
-		/* CEU pins: VIO_D[0-10], VIO_VD, VIO_HD, VIO_CLK */
-		pinmux = <RZA1_PINMUX(1, 0, 5)>, /* VIO_VD */
-			 <RZA1_PINMUX(1, 1, 5)>, /* VIO_HD */
-			 <RZA1_PINMUX(2, 0, 7)>, /* VIO_D0 */
-			 <RZA1_PINMUX(2, 1, 7)>, /* VIO_D1 */
-			 <RZA1_PINMUX(2, 2, 7)>, /* VIO_D2 */
-			 <RZA1_PINMUX(2, 3, 7)>, /* VIO_D3 */
-			 <RZA1_PINMUX(2, 4, 7)>, /* VIO_D4 */
-			 <RZA1_PINMUX(2, 5, 7)>, /* VIO_D5 */
-			 <RZA1_PINMUX(2, 6, 7)>, /* VIO_D6 */
-			 <RZA1_PINMUX(2, 7, 7)>, /* VIO_D7 */
-			 <RZA1_PINMUX(10, 0, 6)>; /* VIO_CLK */
-	};
-};
-
-&i2c1 {
-	pinctrl-names = "default";
-	pinctrl-0 = <&i2c1_pins>;
-
-	status = "okay";
-	clock-frequency = <100000>;
-
-	camera@48 {
-		compatible = "aptina,mt9v111";
-		reg = <0x48>;
-
-		clocks = <&camera_clk>;
-
-		port {
-			mt9v111_out: endpoint {
-				remote-endpoint = <&ceu_in>;
-			};
-		};
-	};
-};
-
-&ceu {
-	pinctrl-names = "default";
-	pinctrl-0 = <&vio_pins>;
-
-	status = "okay";
-
-	port {
-		ceu_in: endpoint {
-			remote-endpoint = <&mt9v111_out>;
-		};
-	};
-};
diff --git a/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi b/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
deleted file mode 100644
index 6e7589ea7562..000000000000
--- a/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
+++ /dev/null
@@ -1,39 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-/*
- * Common file for the AA121TD01 panel connected to Renesas R-Car boards
- *
- * Copyright (C) 2015 Renesas Electronics Corp.
- */
-
-/ {
-	panel {
-		compatible = "mitsubishi,aa121td01", "panel-lvds";
-
-		width-mm = <261>;
-		height-mm = <163>;
-		data-mapping = "jeida-18";
-
-		panel-timing {
-			/* 1280x800 @60Hz */
-			clock-frequency = <71000000>;
-			hactive = <1280>;
-			vactive = <800>;
-			hsync-len = <70>;
-			hfront-porch = <20>;
-			hback-porch = <70>;
-			vsync-len = <5>;
-			vfront-porch = <3>;
-			vback-porch = <15>;
-		};
-
-		port {
-			panel_in: endpoint {
-				remote-endpoint = <&lvds_connector>;
-			};
-		};
-	};
-};
-
-&lvds_connector {
-	remote-endpoint = <&panel_in>;
-};
diff --git a/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi b/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
deleted file mode 100644
index e0930d1ba3aa..000000000000
--- a/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
+++ /dev/null
@@ -1,332 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * Copyright (C) 2017 Ideas on Board <kieran.bingham@ideasonboard.com>
- * Copyright (C) 2021 Jacopo Mondi <jacopo+renesas@jmondi.org>
- *
- * Device Tree Source (overlay) that describes GMSL camera connected to
- * Fakra connectors for the Eagle V3M and Condor V3H (and compatible) boards.
- *
- * The following cameras are currently supported: RDACM20 and RDACM21.
- *
- * The board .dts file that include this has to select which cameras are in use
- * by specifying the camera model with:
- *
- * #define GMSL_CAMERA_RDACM20
- * or
- * #define GMSL_CAMERA_RDACM21
- *
- * And which cameras are connected to the board by defining:
- * for GMSL channel 0:
- *	   #define GMSL_CAMERA_0
- *	   #define GMSL_CAMERA_1
- *	   #define GMSL_CAMERA_2
- *	   #define GMSL_CAMERA_3
- *
- * for GMSL channel 1:
- *	   #define GMSL_CAMERA_4
- *	   #define GMSL_CAMERA_5
- *	   #define GMSL_CAMERA_6
- *	   #define GMSL_CAMERA_7
- */
-
-#include <dt-bindings/gpio/gpio.h>
-
-/* Validate the board file settings. */
-#if !defined(GMSL_CAMERA_RDACM20) && !defined(GMSL_CAMERA_RDACM21)
-#error "Camera model should be defined by the board file"
-#endif
-
-#if defined(GMSL_CAMERA_RDACM20) && defined(GMSL_CAMERA_RDACM21)
-#error "A single camera model should be selected"
-#endif
-
-#if !defined(GMSL_CAMERA_0) && !defined(GMSL_CAMERA_1) && \
-    !defined(GMSL_CAMERA_2) && !defined(GMSL_CAMERA_3) && \
-    !defined(GMSL_CAMERA_4) && !defined(GMSL_CAMERA_5) && \
-    !defined(GMSL_CAMERA_6) && !defined(GMSL_CAMERA_7)
-#error "At least one camera should be selected"
-#endif
-
-/* Deduce from the enabled cameras which GMSL channels are active. */
-#if defined(GMSL_CAMERA_0) || defined(GMSL_CAMERA_1) || \
-    defined(GMSL_CAMERA_2) || defined(GMSL_CAMERA_3)
-#define GMSL_0
-#endif
-
-#if defined(GMSL_CAMERA_4) || defined(GMSL_CAMERA_5) || \
-    defined(GMSL_CAMERA_6) || defined(GMSL_CAMERA_7)
-#define GMSL_1
-#endif
-
-/* Deduce the camera model compatible string. */
-#if defined(GMSL_CAMERA_RDACM20)
-#define GMSL_CAMERA_MODEL "imi,rdacm20"
-#elif defined(GMSL_CAMERA_RDACM21)
-#define GMSL_CAMERA_MODEL "imi,rdacm21"
-#endif
-
-#ifdef GMSL_0
-&vin0 {
-	status = "okay";
-};
-
-&vin1 {
-	status = "okay";
-};
-
-&vin2 {
-	status = "okay";
-};
-
-&vin3 {
-	status = "okay";
-};
-
-&gmsl0 {
-	status = "okay";
-
-#if defined(GMSL_CAMERA_RDACM21)
-	maxim,reverse-channel-microvolt = <100000>;
-#endif
-
-	ports {
-#ifdef GMSL_CAMERA_0
-		port@0 {
-			max9286_in0: endpoint {
-				remote-endpoint = <&fakra_con0>;
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_1
-		port@1 {
-			max9286_in1: endpoint {
-				remote-endpoint = <&fakra_con1>;
-			};
-
-		};
-#endif
-
-#ifdef GMSL_CAMERA_2
-		port@2 {
-			max9286_in2: endpoint {
-				remote-endpoint = <&fakra_con2>;
-			};
-
-		};
-#endif
-
-#ifdef GMSL_CAMERA_3
-		port@3 {
-			max9286_in3: endpoint {
-				remote-endpoint = <&fakra_con3>;
-			};
-
-		};
-#endif
-	};
-
-	i2c-mux {
-#ifdef GMSL_CAMERA_0
-		i2c@0 {
-			status = "okay";
-
-			camera@51 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x51>, <0x61>;
-
-				port {
-					fakra_con0: endpoint {
-						remote-endpoint = <&max9286_in0>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_1
-		i2c@1 {
-			status = "okay";
-
-			camera@52 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x52>, <0x62>;
-
-				port {
-					fakra_con1: endpoint {
-						remote-endpoint = <&max9286_in1>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_2
-		i2c@2 {
-			status = "okay";
-
-			camera@53 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x53>, <0x63>;
-
-				port {
-					fakra_con2: endpoint {
-						remote-endpoint = <&max9286_in2>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_3
-		i2c@3 {
-			status = "okay";
-
-			camera@54 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x54>, <0x64>;
-
-				port {
-					fakra_con3: endpoint {
-						remote-endpoint = <&max9286_in3>;
-					};
-				};
-			};
-		};
-#endif
-	};
-};
-#endif /* ifdef GMSL_0 */
-
-#ifdef GMSL_1
-&vin4 {
-	status = "okay";
-};
-
-&vin5 {
-	status = "okay";
-};
-
-&vin6 {
-	status = "okay";
-};
-
-&vin7 {
-	status = "okay";
-};
-
-&gmsl1 {
-	status = "okay";
-
-#if defined(GMSL_CAMERA_RDACM21)
-	maxim,reverse-channel-microvolt = <100000>;
-#endif
-
-	ports {
-#ifdef GMSL_CAMERA_4
-		port@0 {
-			max9286_in4: endpoint {
-				remote-endpoint = <&fakra_con4>;
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_5
-		port@1 {
-			max9286_in5: endpoint {
-				remote-endpoint = <&fakra_con5>;
-			};
-
-		};
-#endif
-
-#ifdef GMSL_CAMERA_6
-		port@2 {
-			max9286_in6: endpoint {
-				remote-endpoint = <&fakra_con6>;
-			};
-
-		};
-#endif
-
-#ifdef GMSL_CAMERA_7
-		port@3 {
-			max9286_in7: endpoint {
-				remote-endpoint = <&fakra_con7>;
-			};
-
-		};
-#endif
-	};
-
-	i2c-mux {
-#ifdef GMSL_CAMERA_4
-		i2c@0 {
-			status = "okay";
-
-			camera@55 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x55>, <0x65>;
-
-				port {
-					fakra_con4: endpoint {
-						remote-endpoint = <&max9286_in4>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_5
-		i2c@1 {
-			status = "okay";
-
-			camera@56 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x56>, <0x66>;
-
-				port {
-					fakra_con5: endpoint {
-						remote-endpoint = <&max9286_in5>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_6
-		i2c@2 {
-			status = "okay";
-
-			camera@57 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x57>, <0x67>;
-
-				port {
-					fakra_con6: endpoint {
-						remote-endpoint = <&max9286_in6>;
-					};
-				};
-			};
-		};
-#endif
-
-#ifdef GMSL_CAMERA_7
-		i2c@3 {
-			status = "okay";
-
-			camera@58 {
-				compatible = GMSL_CAMERA_MODEL;
-				reg = <0x58>, <0x68>;
-
-				port {
-					fakra_con7: endpoint {
-						remote-endpoint = <&max9286_in7>;
-					};
-				};
-			};
-		};
-#endif
-	};
-};
-#endif /* ifdef GMSL_1 */
diff --git a/arch/arm64/boot/dts/renesas/r8a779m0.dtsi b/arch/arm64/boot/dts/renesas/r8a779m0.dtsi
deleted file mode 100644
index 38978360e722..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m0.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car H3e (R8A779M0) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77951.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m0", "renesas,r8a7795";
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779m2.dtsi b/arch/arm64/boot/dts/renesas/r8a779m2.dtsi
deleted file mode 100644
index bced12764c69..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m2.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car M3e (R8A779M2) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77961.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m2", "renesas,r8a77961";
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779m4.dtsi b/arch/arm64/boot/dts/renesas/r8a779m4.dtsi
deleted file mode 100644
index ae8486056962..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m4.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car M3Ne (R8A779M4) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77965.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m4", "renesas,r8a77965";
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779m6.dtsi b/arch/arm64/boot/dts/renesas/r8a779m6.dtsi
deleted file mode 100644
index 94d6a6cf503e..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m6.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car E3e (R8A779M6) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77990.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m6", "renesas,r8a77990";
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779m7.dtsi b/arch/arm64/boot/dts/renesas/r8a779m7.dtsi
deleted file mode 100644
index 0580fa614034..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m7.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car D3e (R8A779M7) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77995.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m7", "renesas,r8a77995";
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779m8.dtsi b/arch/arm64/boot/dts/renesas/r8a779m8.dtsi
deleted file mode 100644
index dfccc080fb3e..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779m8.dtsi
+++ /dev/null
@@ -1,17 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car H3Ne (R8A779M8) SoC
- *
- * Copyright (C) 2021 Glider bv
- */
-
-#include "r8a77951.dtsi"
-
-/ {
-	compatible = "renesas,r8a779m8", "renesas,r8a7795";
-};
-
-&cluster0_opp {
-	/delete-node/ opp-1600000000;
-	/delete-node/ opp-1700000000;
-};
diff --git a/arch/arm64/boot/dts/renesas/r8a779mb.dtsi b/arch/arm64/boot/dts/renesas/r8a779mb.dtsi
deleted file mode 100644
index 181b737c91cd..000000000000
--- a/arch/arm64/boot/dts/renesas/r8a779mb.dtsi
+++ /dev/null
@@ -1,12 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0 OR MIT)
-/*
- * Device Tree Source for the R-Car H3Ne-1.7G (R8A779MB) SoC
- *
- * Copyright (C) 2022 Glider bv
- */
-
-#include "r8a77951.dtsi"
-
-/ {
-	compatible = "renesas,r8a779mb", "renesas,r8a7795";
-};
diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi b/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
deleted file mode 100644
index 56a979e82c4f..000000000000
--- a/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
+++ /dev/null
@@ -1,25 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-/*
- * Device Tree Source for the RZ/G2LC R9A07G044C1 SoC specific parts
- *
- * Copyright (C) 2021 Renesas Electronics Corp.
- */
-
-/dts-v1/;
-#include "r9a07g044.dtsi"
-
-/ {
-	compatible = "renesas,r9a07g044c1", "renesas,r9a07g044";
-
-	cpus {
-		/delete-node/ cpu-map;
-		/delete-node/ cpu@100;
-	};
-};
-
-&soc {
-	/delete-node/ ssi@1004a800;
-	/delete-node/ serial@1004c800;
-	/delete-node/ adc@10059000;
-	/delete-node/ ethernet@11c30000;
-};
diff --git a/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi b/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
deleted file mode 100644
index 9cf27ca9f1d2..000000000000
--- a/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
+++ /dev/null
@@ -1,18 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-/*
- * Device Tree Source for the RZ/G2L R9A07G044L1 SoC specific parts
- *
- * Copyright (C) 2021 Renesas Electronics Corp.
- */
-
-/dts-v1/;
-#include "r9a07g044.dtsi"
-
-/ {
-	compatible = "renesas,r9a07g044l1", "renesas,r9a07g044";
-
-	cpus {
-		/delete-node/ cpu-map;
-		/delete-node/ cpu@100;
-	};
-};
diff --git a/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi b/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
deleted file mode 100644
index d85a6ac0f024..000000000000
--- a/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
+++ /dev/null
@@ -1,18 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-/*
- * Device Tree Source for the RZ/V2L R9A07G054L1 SoC specific parts
- *
- * Copyright (C) 2021 Renesas Electronics Corp.
- */
-
-/dts-v1/;
-#include "r9a07g054.dtsi"
-
-/ {
-	compatible = "renesas,r9a07g054l1", "renesas,r9a07g054";
-
-	cpus {
-		/delete-node/ cpu-map;
-		/delete-node/ cpu@100;
-	};
-};
diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi b/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
deleted file mode 100644
index e50d9159e832..000000000000
--- a/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
+++ /dev/null
@@ -1,18 +0,0 @@
-// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-/*
- * Device Tree Source for the RZ/G3E R9A09G047E37 SoC specific parts
- *
- * Copyright (C) 2024 Renesas Electronics Corp.
- */
-
-/dts-v1/;
-#include "r9a09g047.dtsi"
-
-/ {
-	compatible = "renesas,r9a09g047e37", "renesas,r9a09g047";
-
-	cpus {
-		/delete-node/ cpu@200;
-		/delete-node/ cpu@300;
-	};
-};
-- 
2.51.0


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-12 20:32 [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi Rob Herring (Arm)
@ 2025-12-13  6:20 ` Laurent Pinchart
  2025-12-13  7:12   ` Niklas Söderlund
                     ` (2 more replies)
  2025-12-14  7:03 ` Chris Paterson
  1 sibling, 3 replies; 8+ messages in thread
From: Laurent Pinchart @ 2025-12-13  6:20 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski,
	Conor Dooley, linux-renesas-soc, devicetree, linux-kernel,
	Jacopo Mondi

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@kernel.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.

>  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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-13  6:20 ` Laurent Pinchart
@ 2025-12-13  7:12   ` Niklas Söderlund
  2025-12-15  8:13   ` Jacopo Mondi
  2025-12-15  9:02   ` Geert Uytterhoeven
  2 siblings, 0 replies; 8+ messages in thread
From: Niklas Söderlund @ 2025-12-13  7:12 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: Rob Herring (Arm), Geert Uytterhoeven, Magnus Damm,
	Krzysztof Kozlowski, Conor Dooley, linux-renesas-soc, devicetree,
	linux-kernel, Jacopo Mondi

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@kernel.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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-12 20:32 [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi Rob Herring (Arm)
  2025-12-13  6:20 ` Laurent Pinchart
@ 2025-12-14  7:03 ` Chris Paterson
  2025-12-15  9:09   ` Geert Uytterhoeven
  1 sibling, 1 reply; 8+ messages in thread
From: Chris Paterson @ 2025-12-14  7:03 UTC (permalink / raw)
  To: Rob Herring (Arm), Geert Uytterhoeven, magnus.damm,
	Krzysztof Kozlowski, Conor Dooley
  Cc: linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org

Hi Rob,

> From: Rob Herring (Arm) <robh@kernel.org>
> Sent: 13 December 2025 05:32
> 
> These .dtsi files are not included anywhere in the tree and can't be
> tested.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../renesas/gr-peach-audiocamerashield.dtsi   |  75 ----
>  .../dts/renesas/r8a77xx-aa121td01-panel.dtsi  |  39 --
>  arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi | 332 ------------------
>  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 -
>  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

The lot of the above are for SoCs that exist and there could well be users using them who haven't upstreamed a dts.
I'd convert them to overlays but they all make use of /delete-node/ which as far as I understand isn't supported in overlays.

Is there another option rather than removing these dtsi files?

Kind regards, Chris

> 
> diff --git a/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
> b/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
> deleted file mode 100644
> index 8d77579807ec..000000000000
> --- a/arch/arm/boot/dts/renesas/gr-peach-audiocamerashield.dtsi
> +++ /dev/null
> @@ -1,75 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0
> -/*
> - * Device Tree Source for the GR-Peach audiocamera shield expansion board
> - *
> - * Copyright (C) 2017 Jacopo Mondi <jacopo+renesas@jmondi.org>
> - */
> -
> -#include "r7s72100.dtsi"
> -#include <dt-bindings/gpio/gpio.h>
> -#include <dt-bindings/pinctrl/r7s72100-pinctrl.h>
> -
> -/ {
> -	/* On-board camera clock. */
> -	camera_clk: camera_clk {
> -		compatible = "fixed-clock";
> -		#clock-cells = <0>;
> -		clock-frequency = <27000000>;
> -	};
> -};
> -
> -&pinctrl {
> -	i2c1_pins: i2c1 {
> -		/* P1_2 as SCL; P1_3 as SDA */
> -		pinmux = <RZA1_PINMUX(1, 2, 1)>, <RZA1_PINMUX(1, 3, 1)>;
> -	};
> -
> -	vio_pins: vio {
> -		/* CEU pins: VIO_D[0-10], VIO_VD, VIO_HD, VIO_CLK */
> -		pinmux = <RZA1_PINMUX(1, 0, 5)>, /* VIO_VD */
> -			 <RZA1_PINMUX(1, 1, 5)>, /* VIO_HD */
> -			 <RZA1_PINMUX(2, 0, 7)>, /* VIO_D0 */
> -			 <RZA1_PINMUX(2, 1, 7)>, /* VIO_D1 */
> -			 <RZA1_PINMUX(2, 2, 7)>, /* VIO_D2 */
> -			 <RZA1_PINMUX(2, 3, 7)>, /* VIO_D3 */
> -			 <RZA1_PINMUX(2, 4, 7)>, /* VIO_D4 */
> -			 <RZA1_PINMUX(2, 5, 7)>, /* VIO_D5 */
> -			 <RZA1_PINMUX(2, 6, 7)>, /* VIO_D6 */
> -			 <RZA1_PINMUX(2, 7, 7)>, /* VIO_D7 */
> -			 <RZA1_PINMUX(10, 0, 6)>; /* VIO_CLK */
> -	};
> -};
> -
> -&i2c1 {
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&i2c1_pins>;
> -
> -	status = "okay";
> -	clock-frequency = <100000>;
> -
> -	camera@48 {
> -		compatible = "aptina,mt9v111";
> -		reg = <0x48>;
> -
> -		clocks = <&camera_clk>;
> -
> -		port {
> -			mt9v111_out: endpoint {
> -				remote-endpoint = <&ceu_in>;
> -			};
> -		};
> -	};
> -};
> -
> -&ceu {
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&vio_pins>;
> -
> -	status = "okay";
> -
> -	port {
> -		ceu_in: endpoint {
> -			remote-endpoint = <&mt9v111_out>;
> -		};
> -	};
> -};
> diff --git a/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
> b/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
> deleted file mode 100644
> index 6e7589ea7562..000000000000
> --- a/arch/arm/boot/dts/renesas/r8a77xx-aa121td01-panel.dtsi
> +++ /dev/null
> @@ -1,39 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0
> -/*
> - * Common file for the AA121TD01 panel connected to Renesas R-Car boards
> - *
> - * Copyright (C) 2015 Renesas Electronics Corp.
> - */
> -
> -/ {
> -	panel {
> -		compatible = "mitsubishi,aa121td01", "panel-lvds";
> -
> -		width-mm = <261>;
> -		height-mm = <163>;
> -		data-mapping = "jeida-18";
> -
> -		panel-timing {
> -			/* 1280x800 @60Hz */
> -			clock-frequency = <71000000>;
> -			hactive = <1280>;
> -			vactive = <800>;
> -			hsync-len = <70>;
> -			hfront-porch = <20>;
> -			hback-porch = <70>;
> -			vsync-len = <5>;
> -			vfront-porch = <3>;
> -			vback-porch = <15>;
> -		};
> -
> -		port {
> -			panel_in: endpoint {
> -				remote-endpoint = <&lvds_connector>;
> -			};
> -		};
> -	};
> -};
> -
> -&lvds_connector {
> -	remote-endpoint = <&panel_in>;
> -};
> diff --git a/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
> b/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
> deleted file mode 100644
> index e0930d1ba3aa..000000000000
> --- a/arch/arm64/boot/dts/renesas/gmsl-cameras.dtsi
> +++ /dev/null
> @@ -1,332 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * Copyright (C) 2017 Ideas on Board <kieran.bingham@ideasonboard.com>
> - * Copyright (C) 2021 Jacopo Mondi <jacopo+renesas@jmondi.org>
> - *
> - * Device Tree Source (overlay) that describes GMSL camera connected to
> - * Fakra connectors for the Eagle V3M and Condor V3H (and compatible)
> boards.
> - *
> - * The following cameras are currently supported: RDACM20 and RDACM21.
> - *
> - * The board .dts file that include this has to select which cameras are
> in use
> - * by specifying the camera model with:
> - *
> - * #define GMSL_CAMERA_RDACM20
> - * or
> - * #define GMSL_CAMERA_RDACM21
> - *
> - * And which cameras are connected to the board by defining:
> - * for GMSL channel 0:
> - *	   #define GMSL_CAMERA_0
> - *	   #define GMSL_CAMERA_1
> - *	   #define GMSL_CAMERA_2
> - *	   #define GMSL_CAMERA_3
> - *
> - * for GMSL channel 1:
> - *	   #define GMSL_CAMERA_4
> - *	   #define GMSL_CAMERA_5
> - *	   #define GMSL_CAMERA_6
> - *	   #define GMSL_CAMERA_7
> - */
> -
> -#include <dt-bindings/gpio/gpio.h>
> -
> -/* Validate the board file settings. */
> -#if !defined(GMSL_CAMERA_RDACM20) && !defined(GMSL_CAMERA_RDACM21)
> -#error "Camera model should be defined by the board file"
> -#endif
> -
> -#if defined(GMSL_CAMERA_RDACM20) && defined(GMSL_CAMERA_RDACM21)
> -#error "A single camera model should be selected"
> -#endif
> -
> -#if !defined(GMSL_CAMERA_0) && !defined(GMSL_CAMERA_1) && \
> -    !defined(GMSL_CAMERA_2) && !defined(GMSL_CAMERA_3) && \
> -    !defined(GMSL_CAMERA_4) && !defined(GMSL_CAMERA_5) && \
> -    !defined(GMSL_CAMERA_6) && !defined(GMSL_CAMERA_7)
> -#error "At least one camera should be selected"
> -#endif
> -
> -/* Deduce from the enabled cameras which GMSL channels are active. */
> -#if defined(GMSL_CAMERA_0) || defined(GMSL_CAMERA_1) || \
> -    defined(GMSL_CAMERA_2) || defined(GMSL_CAMERA_3)
> -#define GMSL_0
> -#endif
> -
> -#if defined(GMSL_CAMERA_4) || defined(GMSL_CAMERA_5) || \
> -    defined(GMSL_CAMERA_6) || defined(GMSL_CAMERA_7)
> -#define GMSL_1
> -#endif
> -
> -/* Deduce the camera model compatible string. */
> -#if defined(GMSL_CAMERA_RDACM20)
> -#define GMSL_CAMERA_MODEL "imi,rdacm20"
> -#elif defined(GMSL_CAMERA_RDACM21)
> -#define GMSL_CAMERA_MODEL "imi,rdacm21"
> -#endif
> -
> -#ifdef GMSL_0
> -&vin0 {
> -	status = "okay";
> -};
> -
> -&vin1 {
> -	status = "okay";
> -};
> -
> -&vin2 {
> -	status = "okay";
> -};
> -
> -&vin3 {
> -	status = "okay";
> -};
> -
> -&gmsl0 {
> -	status = "okay";
> -
> -#if defined(GMSL_CAMERA_RDACM21)
> -	maxim,reverse-channel-microvolt = <100000>;
> -#endif
> -
> -	ports {
> -#ifdef GMSL_CAMERA_0
> -		port@0 {
> -			max9286_in0: endpoint {
> -				remote-endpoint = <&fakra_con0>;
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_1
> -		port@1 {
> -			max9286_in1: endpoint {
> -				remote-endpoint = <&fakra_con1>;
> -			};
> -
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_2
> -		port@2 {
> -			max9286_in2: endpoint {
> -				remote-endpoint = <&fakra_con2>;
> -			};
> -
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_3
> -		port@3 {
> -			max9286_in3: endpoint {
> -				remote-endpoint = <&fakra_con3>;
> -			};
> -
> -		};
> -#endif
> -	};
> -
> -	i2c-mux {
> -#ifdef GMSL_CAMERA_0
> -		i2c@0 {
> -			status = "okay";
> -
> -			camera@51 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x51>, <0x61>;
> -
> -				port {
> -					fakra_con0: endpoint {
> -						remote-endpoint = <&max9286_in0>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_1
> -		i2c@1 {
> -			status = "okay";
> -
> -			camera@52 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x52>, <0x62>;
> -
> -				port {
> -					fakra_con1: endpoint {
> -						remote-endpoint = <&max9286_in1>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_2
> -		i2c@2 {
> -			status = "okay";
> -
> -			camera@53 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x53>, <0x63>;
> -
> -				port {
> -					fakra_con2: endpoint {
> -						remote-endpoint = <&max9286_in2>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_3
> -		i2c@3 {
> -			status = "okay";
> -
> -			camera@54 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x54>, <0x64>;
> -
> -				port {
> -					fakra_con3: endpoint {
> -						remote-endpoint = <&max9286_in3>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -	};
> -};
> -#endif /* ifdef GMSL_0 */
> -
> -#ifdef GMSL_1
> -&vin4 {
> -	status = "okay";
> -};
> -
> -&vin5 {
> -	status = "okay";
> -};
> -
> -&vin6 {
> -	status = "okay";
> -};
> -
> -&vin7 {
> -	status = "okay";
> -};
> -
> -&gmsl1 {
> -	status = "okay";
> -
> -#if defined(GMSL_CAMERA_RDACM21)
> -	maxim,reverse-channel-microvolt = <100000>;
> -#endif
> -
> -	ports {
> -#ifdef GMSL_CAMERA_4
> -		port@0 {
> -			max9286_in4: endpoint {
> -				remote-endpoint = <&fakra_con4>;
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_5
> -		port@1 {
> -			max9286_in5: endpoint {
> -				remote-endpoint = <&fakra_con5>;
> -			};
> -
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_6
> -		port@2 {
> -			max9286_in6: endpoint {
> -				remote-endpoint = <&fakra_con6>;
> -			};
> -
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_7
> -		port@3 {
> -			max9286_in7: endpoint {
> -				remote-endpoint = <&fakra_con7>;
> -			};
> -
> -		};
> -#endif
> -	};
> -
> -	i2c-mux {
> -#ifdef GMSL_CAMERA_4
> -		i2c@0 {
> -			status = "okay";
> -
> -			camera@55 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x55>, <0x65>;
> -
> -				port {
> -					fakra_con4: endpoint {
> -						remote-endpoint = <&max9286_in4>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_5
> -		i2c@1 {
> -			status = "okay";
> -
> -			camera@56 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x56>, <0x66>;
> -
> -				port {
> -					fakra_con5: endpoint {
> -						remote-endpoint = <&max9286_in5>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_6
> -		i2c@2 {
> -			status = "okay";
> -
> -			camera@57 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x57>, <0x67>;
> -
> -				port {
> -					fakra_con6: endpoint {
> -						remote-endpoint = <&max9286_in6>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -
> -#ifdef GMSL_CAMERA_7
> -		i2c@3 {
> -			status = "okay";
> -
> -			camera@58 {
> -				compatible = GMSL_CAMERA_MODEL;
> -				reg = <0x58>, <0x68>;
> -
> -				port {
> -					fakra_con7: endpoint {
> -						remote-endpoint = <&max9286_in7>;
> -					};
> -				};
> -			};
> -		};
> -#endif
> -	};
> -};
> -#endif /* ifdef GMSL_1 */
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m0.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m0.dtsi
> deleted file mode 100644
> index 38978360e722..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m0.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car H3e (R8A779M0) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77951.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m0", "renesas,r8a7795";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m2.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m2.dtsi
> deleted file mode 100644
> index bced12764c69..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m2.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car M3e (R8A779M2) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77961.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m2", "renesas,r8a77961";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m4.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m4.dtsi
> deleted file mode 100644
> index ae8486056962..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m4.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car M3Ne (R8A779M4) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77965.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m4", "renesas,r8a77965";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m6.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m6.dtsi
> deleted file mode 100644
> index 94d6a6cf503e..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m6.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car E3e (R8A779M6) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77990.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m6", "renesas,r8a77990";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m7.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m7.dtsi
> deleted file mode 100644
> index 0580fa614034..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m7.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car D3e (R8A779M7) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77995.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m7", "renesas,r8a77995";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779m8.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779m8.dtsi
> deleted file mode 100644
> index dfccc080fb3e..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779m8.dtsi
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car H3Ne (R8A779M8) SoC
> - *
> - * Copyright (C) 2021 Glider bv
> - */
> -
> -#include "r8a77951.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779m8", "renesas,r8a7795";
> -};
> -
> -&cluster0_opp {
> -	/delete-node/ opp-1600000000;
> -	/delete-node/ opp-1700000000;
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r8a779mb.dtsi
> b/arch/arm64/boot/dts/renesas/r8a779mb.dtsi
> deleted file mode 100644
> index 181b737c91cd..000000000000
> --- a/arch/arm64/boot/dts/renesas/r8a779mb.dtsi
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -/*
> - * Device Tree Source for the R-Car H3Ne-1.7G (R8A779MB) SoC
> - *
> - * Copyright (C) 2022 Glider bv
> - */
> -
> -#include "r8a77951.dtsi"
> -
> -/ {
> -	compatible = "renesas,r8a779mb", "renesas,r8a7795";
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
> b/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
> deleted file mode 100644
> index 56a979e82c4f..000000000000
> --- a/arch/arm64/boot/dts/renesas/r9a07g044c1.dtsi
> +++ /dev/null
> @@ -1,25 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> -/*
> - * Device Tree Source for the RZ/G2LC R9A07G044C1 SoC specific parts
> - *
> - * Copyright (C) 2021 Renesas Electronics Corp.
> - */
> -
> -/dts-v1/;
> -#include "r9a07g044.dtsi"
> -
> -/ {
> -	compatible = "renesas,r9a07g044c1", "renesas,r9a07g044";
> -
> -	cpus {
> -		/delete-node/ cpu-map;
> -		/delete-node/ cpu@100;
> -	};
> -};
> -
> -&soc {
> -	/delete-node/ ssi@1004a800;
> -	/delete-node/ serial@1004c800;
> -	/delete-node/ adc@10059000;
> -	/delete-node/ ethernet@11c30000;
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
> b/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
> deleted file mode 100644
> index 9cf27ca9f1d2..000000000000
> --- a/arch/arm64/boot/dts/renesas/r9a07g044l1.dtsi
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> -/*
> - * Device Tree Source for the RZ/G2L R9A07G044L1 SoC specific parts
> - *
> - * Copyright (C) 2021 Renesas Electronics Corp.
> - */
> -
> -/dts-v1/;
> -#include "r9a07g044.dtsi"
> -
> -/ {
> -	compatible = "renesas,r9a07g044l1", "renesas,r9a07g044";
> -
> -	cpus {
> -		/delete-node/ cpu-map;
> -		/delete-node/ cpu@100;
> -	};
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
> b/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
> deleted file mode 100644
> index d85a6ac0f024..000000000000
> --- a/arch/arm64/boot/dts/renesas/r9a07g054l1.dtsi
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> -/*
> - * Device Tree Source for the RZ/V2L R9A07G054L1 SoC specific parts
> - *
> - * Copyright (C) 2021 Renesas Electronics Corp.
> - */
> -
> -/dts-v1/;
> -#include "r9a07g054.dtsi"
> -
> -/ {
> -	compatible = "renesas,r9a07g054l1", "renesas,r9a07g054";
> -
> -	cpus {
> -		/delete-node/ cpu-map;
> -		/delete-node/ cpu@100;
> -	};
> -};
> diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
> b/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
> deleted file mode 100644
> index e50d9159e832..000000000000
> --- a/arch/arm64/boot/dts/renesas/r9a09g047e37.dtsi
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> -/*
> - * Device Tree Source for the RZ/G3E R9A09G047E37 SoC specific parts
> - *
> - * Copyright (C) 2024 Renesas Electronics Corp.
> - */
> -
> -/dts-v1/;
> -#include "r9a09g047.dtsi"
> -
> -/ {
> -	compatible = "renesas,r9a09g047e37", "renesas,r9a09g047";
> -
> -	cpus {
> -		/delete-node/ cpu@200;
> -		/delete-node/ cpu@300;
> -	};
> -};
> --
> 2.51.0
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-13  6:20 ` Laurent Pinchart
  2025-12-13  7:12   ` Niklas Söderlund
@ 2025-12-15  8:13   ` Jacopo Mondi
  2025-12-15  9:02   ` Geert Uytterhoeven
  2 siblings, 0 replies; 8+ messages in thread
From: Jacopo Mondi @ 2025-12-15  8:13 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: Rob Herring (Arm), Geert Uytterhoeven, Magnus Damm,
	Krzysztof Kozlowski, Conor Dooley, linux-renesas-soc, devicetree,
	linux-kernel, Jacopo Mondi

Hi Laurent, Rob

On Sat, Dec 13, 2025 at 03:20:37PM +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@kernel.org>
> > ---
> >  .../renesas/gr-peach-audiocamerashield.dtsi   |  75 ----
>
> Jacopo, any opinion on this ? Should it be conevrted to an overlay, or
> dropped ?
>

Running Linux in the 10MB SRAM of the GR-Peach using XIP was
challenging enough and I don't have an easy access to that setup
anymore.

I think we can drop this one, and resubmit as a .dtso eventually, if
anyone ever needs it

Acked-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>

Thanks
  j


> >  .../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.
>
> >  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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-13  6:20 ` Laurent Pinchart
  2025-12-13  7:12   ` Niklas Söderlund
  2025-12-15  8:13   ` Jacopo Mondi
@ 2025-12-15  9:02   ` Geert Uytterhoeven
  2025-12-15 13:19     ` Rob Herring
  2 siblings, 1 reply; 8+ messages in thread
From: Geert Uytterhoeven @ 2025-12-15  9:02 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Laurent Pinchart, Magnus Damm, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc, devicetree, linux-kernel, Jacopo Mondi

Hi Rob,

On Sat, 13 Dec 2025 at 07:20, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> 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@kernel.org>

Thanks for your patch!

> >  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 :-)

While I do understand your point, these might be used by customers of
the SoCs.  Hence they can be considered part of the "stable DT ABI",
and removing them can cause issues downstream.

At least for the r8a779m* parts, I do have local patches in my tree,
so I would notice any future build breakages (most files are rather
simple, so unlikely to break, though).
Shall we just upstream .dts files using these .dtsi files (cfr. the
existing arch/arm64/boot/dts/renesas/r8a779m*dts)? AFAIK none of
the missing board/SoC-combos actually exist as products, but someone
might create one locally by replacing the SoC on an existing board,
as they are pin-compatible variants.

There is a similar story for the r9a0* parts: they are variants with
less CPU cores, which may end up in actual products.  We could add
.dts files using them, to make sure no build breakage is introduced.

Please let me know how you would like to proceed.
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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-14  7:03 ` Chris Paterson
@ 2025-12-15  9:09   ` Geert Uytterhoeven
  0 siblings, 0 replies; 8+ messages in thread
From: Geert Uytterhoeven @ 2025-12-15  9:09 UTC (permalink / raw)
  To: Chris Paterson
  Cc: Rob Herring (Arm), magnus.damm, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org

Hi Chris,

On Sun, 14 Dec 2025 at 08:04, Chris Paterson
<Chris.Paterson2@renesas.com> wrote:
> > From: Rob Herring (Arm) <robh@kernel.org>
> > These .dtsi files are not included anywhere in the tree and can't be
> > tested.
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

> >  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
>
> The lot of the above are for SoCs that exist and there could well be users using them who haven't upstreamed a dts.
> I'd convert them to overlays but they all make use of /delete-node/ which as far as I understand isn't supported in overlays.

Converting to overlays is not suitable here, as these do represent base
SoC functionality, not an add-on.

Besides, overlays also have the "must have at least one user" rule.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi
  2025-12-15  9:02   ` Geert Uytterhoeven
@ 2025-12-15 13:19     ` Rob Herring
  0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2025-12-15 13:19 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Laurent Pinchart, Magnus Damm, Krzysztof Kozlowski, Conor Dooley,
	linux-renesas-soc, devicetree, linux-kernel, Jacopo Mondi

On Mon, Dec 15, 2025 at 3:02 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Rob,
>
> On Sat, 13 Dec 2025 at 07:20, Laurent Pinchart
> <laurent.pinchart@ideasonboard.com> wrote:
> > 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@kernel.org>
>
> Thanks for your patch!
>
> > >  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 :-)
>
> While I do understand your point, these might be used by customers of
> the SoCs.  Hence they can be considered part of the "stable DT ABI",
> and removing them can cause issues downstream.

DTBs are the ABI, not .dtsi. The compatible in these is not validated
and isn't valid. We could simply restructure .dtsi files in a way that
breaks downstream and wouldn't give it a second thought.

> At least for the r8a779m* parts, I do have local patches in my tree,
> so I would notice any future build breakages (most files are rather
> simple, so unlikely to break, though).
> Shall we just upstream .dts files using these .dtsi files (cfr. the
> existing arch/arm64/boot/dts/renesas/r8a779m*dts)? AFAIK none of
> the missing board/SoC-combos actually exist as products, but someone
> might create one locally by replacing the SoC on an existing board,
> as they are pin-compatible variants.

You probably could restructure these where you include the subset
.dtsi and then add nodes rather than delete them. For the ones that
just override the compatible with an invalid value that has to be
overridden by the board dts anyways, there's little value to them.
Just drop them or add a proper board.

> There is a similar story for the r9a0* parts: they are variants with
> less CPU cores, which may end up in actual products.  We could add
> .dts files using them, to make sure no build breakage is introduced.

If these are all just binned parts, I would not have separate SoC
compatibles for them.

Rob

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2025-12-15 13:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-12 20:32 [PATCH] arm/arm64: dts: renesas: Drop unused .dtsi Rob Herring (Arm)
2025-12-13  6:20 ` Laurent Pinchart
2025-12-13  7:12   ` Niklas Söderlund
2025-12-15  8:13   ` Jacopo Mondi
2025-12-15  9:02   ` Geert Uytterhoeven
2025-12-15 13:19     ` Rob Herring
2025-12-14  7:03 ` Chris Paterson
2025-12-15  9:09   ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).