mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: barebox@lists.infradead.org
Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>
Subject: [PATCH v1 10/20] ARM: dts: zii: use phandle-relative paths for extending nodes
Date: Fri, 17 Feb 2023 18:30:47 +0100	[thread overview]
Message-ID: <20230217173057.1839835-11-a.fatoum@pengutronix.de> (raw)
In-Reply-To: <20230217173057.1839835-1-a.fatoum@pengutronix.de>

Upstream will soon switch the rave-sp nodes to be called mcu
instead, which will break our usage. Anticipate this by using
references.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 arch/arm/dts/imx51-zii-rdu1.dts    | 151 +++++++++++++----------------
 arch/arm/dts/imx6qdl-zii-rdu2.dtsi |  59 +++++------
 arch/arm/dts/imx7d-zii-rmu2.dts    |  49 ++++------
 arch/arm/dts/imx8mq-zii-ultra.dtsi |  38 ++++----
 4 files changed, 127 insertions(+), 170 deletions(-)

diff --git a/arch/arm/dts/imx51-zii-rdu1.dts b/arch/arm/dts/imx51-zii-rdu1.dts
index 7f6a2962b4f0..72f5484bc0fd 100644
--- a/arch/arm/dts/imx51-zii-rdu1.dts
+++ b/arch/arm/dts/imx51-zii-rdu1.dts
@@ -53,33 +53,31 @@
                  * the switch shared DT node with it, so we use that
                  * fact to create a desirable naming
 		 */
-		switch-eeprom = &switch;
-		microwire-eeprom = &microwire_eeprom;
+		switch-eeprom = &{mdio_gpio/switch@0};
+		microwire-eeprom = &{spi_gpio/eeprom@0};
 	};
 };
 
-&ecspi1 {
-	spinor: flash@1 {
-		partition@0 {
-			/*
-			 * Do not change the size of this
-			 * partition. RDU1's BBU code relies on
-			 * "barebox" partition starting at 1024 byte
-			 * mark to function properly
-			 */
-			label = "config";
-			reg = <0x0 0x400>;
-		};
+spinor: &{ecspi1/flash@1} {
+	partition@0 {
+		/*
+		 * Do not change the size of this
+		 * partition. RDU1's BBU code relies on
+		 * "barebox" partition starting at 1024 byte
+		 * mark to function properly
+		 */
+		label = "config";
+		reg = <0x0 0x400>;
+	};
 
-		partition@400 {
-			label = "barebox";
-			reg = <0x400 0xdfc00>;
-		};
+	partition@400 {
+		label = "barebox";
+		reg = <0x400 0xdfc00>;
+	};
 
-		env_spinor: partition@e0000 {
-			label = "barebox-environment";
-			reg = <0xe0000 0x20000>;
-		};
+	env_spinor: partition@e0000 {
+		label = "barebox-environment";
+		reg = <0xe0000 0x20000>;
 	};
 };
 
@@ -99,33 +97,21 @@
 	};
 };
 
-&mdio_gpio {
-	switch: switch@0 {};
+&{uart3/rave-sp/watchdog} {
+	nvmem-cells = <&boot_source>;
+	nvmem-cell-names = "boot-source";
 };
 
-&spi_gpio {
-	microwire_eeprom: eeprom@0 {};
-};
-
-&uart3 {
-	rave-sp {
-		watchdog {
-			nvmem-cells = <&boot_source>;
-			nvmem-cell-names = "boot-source";
-		};
+&{uart3/rave-sp/eeprom@a4} {
+	nvmem-cells = <&shadow_config>;
+	nvmem-cell-names = "shadow-config";
 
-		eeprom@a4 {
-			nvmem-cells = <&shadow_config>;
-			nvmem-cell-names = "shadow-config";
-
-			boot_source: boot-source@83 {
-				reg = <0x83 1>;
-			};
+	boot_source: boot-source@83 {
+		reg = <0x83 1>;
+	};
 
-			shadow_config: shadow-config@1000 {
-				reg = <0x1000 0x400>;
-			};
-		};
+	shadow_config: shadow-config@1000 {
+		reg = <0x1000 0x400>;
 	};
 };
 
@@ -133,44 +119,41 @@
 	status = "disabled";
 };
 
-&iomuxc {
-	pinctrl_usbh1: usbh1grp {
-
-		/*
-		 * Overwrite upstream USBH1,2 iomux settings to match
-		 * the setting U-Boot would set these to. Remove this
-		 * once this is fixed upstream.
-		 */
-		fsl,pins = <
-			MX51_PAD_USBH1_STP__USBH1_STP		0x1e5
-			MX51_PAD_USBH1_CLK__USBH1_CLK		0x1e5
-			MX51_PAD_USBH1_DIR__USBH1_DIR		0x1e5
-			MX51_PAD_USBH1_NXT__USBH1_NXT		0x1e5
-			MX51_PAD_USBH1_DATA0__USBH1_DATA0	0x1e5
-			MX51_PAD_USBH1_DATA1__USBH1_DATA1	0x1e5
-			MX51_PAD_USBH1_DATA2__USBH1_DATA2	0x1e5
-			MX51_PAD_USBH1_DATA3__USBH1_DATA3	0x1e5
-			MX51_PAD_USBH1_DATA4__USBH1_DATA4	0x1e5
-			MX51_PAD_USBH1_DATA5__USBH1_DATA5	0x1e5
-			MX51_PAD_USBH1_DATA6__USBH1_DATA6	0x1e5
-			MX51_PAD_USBH1_DATA7__USBH1_DATA7	0x1e5
-		>;
-	};
+&pinctrl_usbh1 {
+	/*
+	 * Overwrite upstream USBH1,2 iomux settings to match
+	 * the setting U-Boot would set these to. Remove this
+	 * once this is fixed upstream.
+	 */
+	fsl,pins = <
+		MX51_PAD_USBH1_STP__USBH1_STP		0x1e5
+		MX51_PAD_USBH1_CLK__USBH1_CLK		0x1e5
+		MX51_PAD_USBH1_DIR__USBH1_DIR		0x1e5
+		MX51_PAD_USBH1_NXT__USBH1_NXT		0x1e5
+		MX51_PAD_USBH1_DATA0__USBH1_DATA0	0x1e5
+		MX51_PAD_USBH1_DATA1__USBH1_DATA1	0x1e5
+		MX51_PAD_USBH1_DATA2__USBH1_DATA2	0x1e5
+		MX51_PAD_USBH1_DATA3__USBH1_DATA3	0x1e5
+		MX51_PAD_USBH1_DATA4__USBH1_DATA4	0x1e5
+		MX51_PAD_USBH1_DATA5__USBH1_DATA5	0x1e5
+		MX51_PAD_USBH1_DATA6__USBH1_DATA6	0x1e5
+		MX51_PAD_USBH1_DATA7__USBH1_DATA7	0x1e5
+	>;
+};
 
-	pinctrl_usbh2: usbh2grp {
-		fsl,pins = <
-			MX51_PAD_EIM_A26__USBH2_STP		0x1e5
-			MX51_PAD_EIM_A24__USBH2_CLK		0x1e5
-			MX51_PAD_EIM_A25__USBH2_DIR		0x1e5
-			MX51_PAD_EIM_A27__USBH2_NXT		0x1e5
-			MX51_PAD_EIM_D16__USBH2_DATA0		0x1e5
-			MX51_PAD_EIM_D17__USBH2_DATA1		0x1e5
-			MX51_PAD_EIM_D18__USBH2_DATA2		0x1e5
-			MX51_PAD_EIM_D19__USBH2_DATA3		0x1e5
-			MX51_PAD_EIM_D20__USBH2_DATA4		0x1e5
-			MX51_PAD_EIM_D21__USBH2_DATA5		0x1e5
-			MX51_PAD_EIM_D22__USBH2_DATA6		0x1e5
-			MX51_PAD_EIM_D23__USBH2_DATA7		0x1e5
-		>;
-	};
+&pinctrl_usbh2 {
+	fsl,pins = <
+		MX51_PAD_EIM_A26__USBH2_STP		0x1e5
+		MX51_PAD_EIM_A24__USBH2_CLK		0x1e5
+		MX51_PAD_EIM_A25__USBH2_DIR		0x1e5
+		MX51_PAD_EIM_A27__USBH2_NXT		0x1e5
+		MX51_PAD_EIM_D16__USBH2_DATA0		0x1e5
+		MX51_PAD_EIM_D17__USBH2_DATA1		0x1e5
+		MX51_PAD_EIM_D18__USBH2_DATA2		0x1e5
+		MX51_PAD_EIM_D19__USBH2_DATA3		0x1e5
+		MX51_PAD_EIM_D20__USBH2_DATA4		0x1e5
+		MX51_PAD_EIM_D21__USBH2_DATA5		0x1e5
+		MX51_PAD_EIM_D22__USBH2_DATA6		0x1e5
+		MX51_PAD_EIM_D23__USBH2_DATA7		0x1e5
+	>;
 };
diff --git a/arch/arm/dts/imx6qdl-zii-rdu2.dtsi b/arch/arm/dts/imx6qdl-zii-rdu2.dtsi
index 994e43b53b64..2e6a74dde019 100644
--- a/arch/arm/dts/imx6qdl-zii-rdu2.dtsi
+++ b/arch/arm/dts/imx6qdl-zii-rdu2.dtsi
@@ -178,34 +178,27 @@
 	};
 };
 
-&uart4 {
-	rave-sp {
-		#address-cells = <1>;
-		#size-cells = <1>;
-
-		watchdog {
-			nvmem-cells = <&boot_source>;
-			nvmem-cell-names = "boot-source";
-		};
+&{uart4/rave-sp/watchdog} {
+	nvmem-cells = <&boot_source>;
+	nvmem-cell-names = "boot-source";
+};
 
-		eeprom@a4 {
-			lru_part_number: lru-part-number@21 {
-				reg = <0x21 15>;
-				read-only;
-			};
+&{uart4/rave-sp/eeprom@a4} {
+	lru_part_number: lru-part-number@21 {
+		reg = <0x21 15>;
+		read-only;
+	};
 
-			boot_source: boot-source@83 {
-				reg = <0x83 1>;
-			};
+	boot_source: boot-source@83 {
+		reg = <0x83 1>;
+	};
 
-			mac_address_0: mac-address@180 {
-				reg = <0x180 6>;
-			};
+	mac_address_0: mac-address@180 {
+		reg = <0x180 6>;
+	};
 
-			mac_address_1: mac-address@190 {
-				reg = <0x190 6>;
-			};
-		};
+	mac_address_1: mac-address@190 {
+		reg = <0x190 6>;
 	};
 };
 
@@ -231,22 +224,16 @@
 	nvmem-cell-names = "mac-address";
 };
 
-&i2c1 {
-	edp-bridge@68 {
-		pinctrl-0 = <&pinctrl_tc358767>, <&pinctrl_disp0>;
-	};
+&{i2c1/edp-bridge@68} {
+	pinctrl-0 = <&pinctrl_tc358767>, <&pinctrl_disp0>;
 };
 
-&i2c2 {
-	temp-sense@48 {
-		barebox,sensor-name = "Temp Sensor 1";
-	};
+&{i2c2/temp-sense@48} {
+	barebox,sensor-name = "Temp Sensor 1";
 };
 
-&ldb {
-	lvds-channel@0 {
-		fsl,data-width = <24>;
-	};
+&{ldb/lvds-channel@0} {
+	fsl,data-width = <24>;
 };
 
 &i210 {
diff --git a/arch/arm/dts/imx7d-zii-rmu2.dts b/arch/arm/dts/imx7d-zii-rmu2.dts
index 41566f54a112..17d625b83e23 100644
--- a/arch/arm/dts/imx7d-zii-rmu2.dts
+++ b/arch/arm/dts/imx7d-zii-rmu2.dts
@@ -18,37 +18,28 @@
 	/delete-property/ assigned-clock-parents;
 };
 
-&ecspi1 {
-	nor_flash: flash@0 {
-		#address-cells = <1>;
-		#size-cells = <1>;
-
-		partition@0 {
-			label = "barebox";
-			reg = <0x0 0xc0000>;
-		};
-
-		partition@c0000 {
-			label = "barebox-environment";
-			reg = <0xc0000 0x40000>;
-		};
+nor_flash: &{ecspi1/flash@0} {
+	#address-cells = <1>;
+	#size-cells = <1>;
+
+	partition@0 {
+		label = "barebox";
+		reg = <0x0 0xc0000>;
+	};
+
+	partition@c0000 {
+		label = "barebox-environment";
+		reg = <0xc0000 0x40000>;
 	};
 };
 
-&uart4 {
-	rave-sp {
-		#address-cells = <1>;
-		#size-cells = <1>;
-
-		watchdog {
-			nvmem-cells = <&boot_source>;
-			nvmem-cell-names = "boot-source";
-		};
-
-		eeprom@a3 {
-			boot_source: boot-source@83 {
-				reg = <0x83 1>;
-			};
-		};
+&{uart4/rave-sp/watchdog} {
+	nvmem-cells = <&boot_source>;
+	nvmem-cell-names = "boot-source";
+};
+
+&{uart4/rave-sp/eeprom@a3} {
+	boot_source: boot-source@83 {
+		reg = <0x83 1>;
 	};
 };
diff --git a/arch/arm/dts/imx8mq-zii-ultra.dtsi b/arch/arm/dts/imx8mq-zii-ultra.dtsi
index dc6d743f29cd..27ffebc154f4 100644
--- a/arch/arm/dts/imx8mq-zii-ultra.dtsi
+++ b/arch/arm/dts/imx8mq-zii-ultra.dtsi
@@ -64,31 +64,27 @@
 	nvmem-cell-names = "mac-address";
 };
 
-&uart2 {
-	rave-sp {
-		watchdog {
-			nvmem-cells = <&boot_source>;
-			nvmem-cell-names = "boot-source";
-		};
+&{uart2/rave-sp/watchdog} {
+	nvmem-cells = <&boot_source>;
+	nvmem-cell-names = "boot-source";
+};
 
-		eeprom@a4 {
-			lru_part_number: lru-part-number@21 {
-				reg = <0x21 15>;
-				read-only;
-			};
+&{uart2/rave-sp/eeprom@a4} {
+	lru_part_number: lru-part-number@21 {
+		reg = <0x21 15>;
+		read-only;
+	};
 
-			boot_source: boot-source@83 {
-				reg = <0x83 1>;
-			};
+	boot_source: boot-source@83 {
+		reg = <0x83 1>;
+	};
 
-			mac_address_0: mac-address@180 {
-				reg = <0x180 6>;
-			};
+	mac_address_0: mac-address@180 {
+		reg = <0x180 6>;
+	};
 
-			mac_address_1: mac-address@190 {
-				reg = <0x190 6>;
-			};
-		};
+	mac_address_1: mac-address@190 {
+		reg = <0x190 6>;
 	};
 };
 
-- 
2.30.2




  parent reply	other threads:[~2023-02-17 17:32 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-17 17:30 [PATCH v1 00/20] dts: avoid DT breakage new fancy DTC v1.7.0 syntax Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 01/20] scripts/dtc: Update to upstream version v1.7.0 Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 02/20] scripts/dtc: update-dts-source.sh: don't fail if libfdt/ exists Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 03/20] Documentation: devicetree: describe new label-relative-path syntax Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 04/20] ARM: dts: use DT labels instead of paths where possible Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 05/20] MIPS: dts: ath79: ar9331: use path references when extending nodes Ahmad Fatoum
2023-02-22 10:25   ` Sascha Hauer
2023-02-17 17:30 ` [PATCH v1 06/20] ARM: dts: tegra: switch to path and label references Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 07/20] ARM: i.MX51: ccmx51: support newer device trees Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 08/20] ARM: i.MX6: gw54xx: rename watchdog nodes Ahmad Fatoum
2023-02-17 18:08   ` Jules Maselbas
2023-02-17 17:30 ` [PATCH v1 09/20] ARM: i.MX6: karo-tx6x: fix now renamed DT nodes Ahmad Fatoum
2023-02-17 17:30 ` Ahmad Fatoum [this message]
2023-02-17 17:30 ` [PATCH v1 11/20] ARM: dts: AT91: use label-relative paths for extending nodes Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 12/20] ARM: dts: Layerscape: drop unneeded EEPROM node overriding Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 13/20] ARM: dts: Layerscape: use label-relative paths references Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 14/20] ARM: dts: vf610-zii-cfu1: remove duplicate LED Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 15/20] ARM: dts: i.MX53: ccxmx53: remove unused /memory Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 16/20] ARM: dts: i.MX: use label-relative references Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 17/20] ARM: dts: i.MX: delete now renamed memory nodes as well Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 18/20] ARM: dts: socfpga: cut down on NAND node duplication Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 19/20] ARM: dts: i.MX8MN: evk: use upstream flash node definition Ahmad Fatoum
2023-02-17 17:30 ` [PATCH v1 20/20] ARM: dts: i.MX8MN: evk: reduce duplication in barebox-added nodes Ahmad Fatoum
2023-02-20  8:48 ` [PATCH v1 00/20] dts: avoid DT breakage new fancy DTC v1.7.0 syntax Marco Felsch
2023-02-20  8:51   ` Ahmad Fatoum
2023-02-21 10:25 ` Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230217173057.1839835-11-a.fatoum@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox