Skip to contentSkip to conversation

arm: dts: socfpga: Fix SPI aliases

PATCH New Marek Vasut 1 participant 2 messages Jul 27, 2015 – Aug 3, 2015 Arm (32-bit)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 arm: dts: socfpga: Fix SPI aliases

The SPI aliases are completely wrong. First, they point to non-existing /spi at .* nodes instead of the correct /soc/spi at .* nodes. Second, the use ad-hoc string instead of a handle. Furthermore, they are copied multiple times in each board DTS.

So fix it such that we move these into socfpga.dtsi and make them use the usual handles.

---
 arch/arm/dts/socfpga.dtsi                  | 3 +++
 arch/arm/dts/socfpga_arria5_socdk.dts      | 8 ++------
 arch/arm/dts/socfpga_cyclone5_socdk.dts    | 4 ----
 arch/arm/dts/socfpga_cyclone5_socrates.dts | 6 ------
 4 files changed, 5 insertions(+), 16 deletions(-)
arch/arm/dts/socfpga.dtsi +3 -0
index 9b12420..3a3fb4a 100644
@@ -20,6 +20,9 @@
 		timer1 = &timer1;
 		timer2 = &timer2;
 		timer3 = &timer3;
+		spi0 = &qspi;
+		spi1 = &spi0;
+		spi2 = &spi1;
 	};
 
 	cpus {
arch/arm/dts/socfpga_arria5_socdk.dts +2 -6
index 1b86897..f2b5963 100644
@@ -22,13 +22,9 @@
 
 	aliases {
 		/* this allow the ethaddr uboot environmnet variable contents
-		* to be added to the gmac1 device tree blob.
-		*/
+		 * to be added to the gmac1 device tree blob.
+		 */
 		ethernet0 = &gmac1;
-
-		spi0 = "/spi at ff705000";		/* QSPI */
-		spi1 = "/spi at fff00000";
-		spi2 = "/spi at fff01000";
 	};
 
 	regulator_3_3v: 3-3-v-regulator {
arch/arm/dts/socfpga_cyclone5_socdk.dts +0 -4
index 0b300b9..9650eb0 100644
@@ -25,10 +25,6 @@
 		 * to be added to the gmac1 device tree blob.
 		 */
 		ethernet0 = &gmac1;
-
-		spi0 = "/spi at ff705000";		/* QSPI */
-		spi1 = "/spi at fff00000";
-		spi2 = "/spi at fff01000";
 	};
 
 	regulator_3_3v: 3-3-v-regulator {
arch/arm/dts/socfpga_cyclone5_socrates.dts +0 -6
index ea30483..00b1830 100644
@@ -14,12 +14,6 @@
 		bootargs = "console=ttyS0,115200";
 	};
 
- 	aliases {
-		spi0 = "/spi at ff705000";		/* QSPI */
-		spi1 = "/spi at fff00000";
-		spi2 = "/spi at fff01000";
- 	};
-
 	memory {
 		name = "memory";
 		device_type = "memory";
Signature · 1 line
2.1.4
#1 lore raw

PATCH 1/1 arm: dts: socfpga: Fix SPI aliases

Quoted 10 lines · Marek Vasut

The SPI aliases are completely wrong. First, they point to non-existing /spi at .* nodes instead of the correct /soc/spi at .* nodes. Second, the use ad-hoc string instead of a handle. Furthermore, they are copied multiple times in each board DTS.

So fix it such that we move these into socfpga.dtsi and make them use the usual handles.

Applied to u-boot-socfpga/master .

Best regards,
Marek Vasut