DT: Fix spelling of standard properties

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 arch/arm/boot/dts/omap36xx.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

arch/arm/boot/dts/omap36xx.dtsi +0 -0

index ce1e242d4dc07ea8..8b797915300894d8 100644

arch/arm/boot/dts/omap36xx.dtsi +0 -1

                        #size-cells = <0>;
                        reg = <0x483072f0 0x8>, <0x48306818 0x4>;
                        reg-names = "base-address", "int-address";
--

1.9.1

Quoted 10 lines · Geert Uytterhoeven

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 arch/arm/boot/dts/omap36xx.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

arch/arm/boot/dts/omap36xx.dtsi +0 -0

index ce1e242d4dc07ea8..8b797915300894d8 100644

arch/arm/boot/dts/omap36xx.dtsi line 47

Quoted 5 lines

arch/arm/boot/dts/omap36xx.dtsi +1 -1

@@ -44,7 +44,7 @@
                abb_mpu_iva: regulator-abb-mpu {
                        compatible = "ti,abb-v1";
                        regulator-name = "abb_mpu_iva";
-                       #address-cell = <0>;
+                       #address-cells = <0>;

dtc should flag this if it was really needed. However, it looks like it isn't as there are no child nodes (unless the board files add them).

A nice dtc check would be to flag unnecessary #address-cells or #size-cells. That would only help here after your fix though.

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/sound/davinci-mcbsp.txt | 2 +-

 drivers/dma/bcm2835-dma.c                                 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Documentation/devicetree/bindings/sound/davinci-mcbsp.txt +1 -1

index 55b53e1fd72c9d6e..e0b6165c9cfcec19 100644
@@ -43,7 +43,7 @@ mcbsp0: mcbsp@1d10000 {
 		<0x00310000 0x1000>;
 	reg-names = "mpu", "dat";
 	interrupts = <97 98>;
-	interrupts-names = "rx", "tx";
+	interrupt-names = "rx", "tx";
 	dmas = <&edma0 3 1
 		&edma0 2 1>;
 	dma-names = "tx", "rx";

drivers/dma/bcm2835-dma.c +1 -1

index 974015193b93cdb3..d724393e904e9a41 100644
@@ -974,7 +974,7 @@ static int bcm2835_dma_probe(struct platform_device *pdev)
 
 		/* legacy device tree case handling */
 		dev_warn_once(&pdev->dev,
-			      "missing interrupts-names property in device tree - legacy interpretation is used");
+			      "missing interrupt-names property in device tree - legacy interpretation is used");
 		/*
 		 * in case of channel >= 11
 		 * use the 11th interrupt and that is shared

Signature · 1 line

1.9.1

Quoted 3 lines · Geert Uytterhoeven

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/sound/davinci-mcbsp.txt | 2 +-

This change does not apply for me, can you please split it up and send the sound ones thru sound tree

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 .../devicetree/bindings/input/touchscreen/brcm,iproc-touchscreen.txt    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Documentation/devicetree/bindings/input/touchscreen/brcm,iproc-touchscreen.txt +1 -1

index 25541f30e38701f2..ac5dff412e25266f 100644
@@ -7,7 +7,7 @@ Required properties:
   If this property is selected please make sure MFD_SYSCON config
   is enabled in the defconfig file.
 - clocks:  The clock provided by the SOC to driver the tsc
-- clock-name:  name for the clock
+- clock-names:  name for the clock
 - interrupts: The touchscreen controller's interrupt
 - address-cells: Specify the number of u32 entries needed in child nodes.
   Should set to 1.

Signature · 6 lines

1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/sram/sram.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Documentation/devicetree/bindings/sram/sram.txt +1 -1

index 227e3a341af1e2b5..add48f09015e212e 100644
@@ -51,7 +51,7 @@ sram: sram at 5c000000 {
 	compatible = "mmio-sram";
 	reg = <0x5c000000 0x40000>; /* 256 KiB SRAM at address 0x5c000000 */
 
-	#adress-cells = <1>;
+	#address-cells = <1>;
 	#size-cells = <1>;
 	ranges = <0 0x5c000000 0x40000>;
 

Signature · 1 line

1.9.1

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/pci/hisilicon-pcie.txt | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Documentation/devicetree/bindings/pci/hisilicon-pcie.txt +6 -6

index b721beacfe4dae6c..59c2f47aa303ae24 100644
@@ -34,11 +34,11 @@ Hip05 Example (note that Hip06 is the same except compatible):
 		ranges = <0x82000000 0 0x00000000 0x220 0x00000000 0 0x10000000>;
 		num-lanes = <8>;
 		port-id = <1>;
-		#interrupts-cells = <1>;
-		interrupts-map-mask = <0xf800 0 0 7>;
-		interrupts-map = <0x0 0 0 1 &mbigen_pcie 1 10
-				  0x0 0 0 2 &mbigen_pcie 2 11
-				  0x0 0 0 3 &mbigen_pcie 3 12
-				  0x0 0 0 4 &mbigen_pcie 4 13>;
+		#interrupt-cells = <1>;
+		interrupt-map-mask = <0xf800 0 0 7>;
+		interrupt-map = <0x0 0 0 1 &mbigen_pcie 1 10
+				 0x0 0 0 2 &mbigen_pcie 2 11
+				 0x0 0 0 3 &mbigen_pcie 3 12
+				 0x0 0 0 4 &mbigen_pcie 4 13>;
 		status = "ok";
 	};

Signature · 1 line

1.9.1

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/phy/phy-stih41x-usb.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Documentation/devicetree/bindings/phy/phy-stih41x-usb.txt +1 -1

index 00944a05ee6b4998..744b4809542edd3b 100644
@@ -17,7 +17,7 @@ Example:
 
 usb2_phy: usb2phy at 0 {
 	compatible	= "st,stih416-usb-phy";
-	#phy-cell	= <0>;
+	#phy-cells	= <0>;
 	st,syscfg	= <&syscfg_rear>;
 	clocks		= <&clk_sysin>;
 	clock-names	= "osc_phy";

Signature · 1 line

1.9.1

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 Documentation/devicetree/bindings/rtc/sa1100-rtc.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Signed-off-by Geert Uytterhoeven <[email protected]>

---
 .../devicetree/bindings/regulator/ti-abb-regulator.txt         | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Documentation/devicetree/bindings/regulator/ti-abb-regulator.txt +0 -0

index c58db75f959e601f..c3f6546ebac77742 100644

This patch series fixes misspellings of various standard DT properties in DT binding documentation, DTS files, and error messages.

Hi all,

This patch series fixes misspellings of various standard DT properties in DT binding documentation, DTS files, and error messages. While most of these are harmless, some of them may cause hard-to-debug failures.

Please apply where appropriate.

Thanks!

P.S. I used the following to detect misspellings:

    words="(address|clock|cooling|dma|gpio|index|interrupt|mbox|msi|nvmem|phy|phys|power-domain|pwm|reset|size|sleep|sound-dai|thermal-sensor)"

    git grep -Ew "${words}s-names"
    git grep -E "[^-]\<${words}-name\>[^-]"
    git grep -Ew "#${words}s-cells"             # false positive phys-cells
    git grep -E "#${words}-cell\>[^-]"

    git grep -w adress-cells
    git grep -Ew "interrupts-(map|parent)"

How can we prevent adding more of these?

One simple option is to add the offenders to scripts/spelling.txt. Alternatively, we may want to do something smarter and more DT specific?

Quoted 7 lines · Geert Uytterhoeven

Hi all,

This patch series fixes misspellings of various standard DT properties in DT binding documentation, DTS files, and error messages. While most of these are harmless, some of them may cause hard-to-debug failures.

Quoted 1 line

Please apply where appropriate.

I'll happily take the whole series if you like, though I've not reviewed it all yet.