Using overlay sugar syntax makes the DTS overlay files easier to read
(and write).

Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be>
---
Why are there two separate fragments for spi0? Can't they be combined?
Why do you need the spidev@1 entry?
---
 .../devicetree/pi433-overlay.dts              | 79 ++++++++-----------
 1 file changed, 35 insertions(+), 44 deletions(-)

diff --git a/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts 
b/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts
index b584180d78d019aa..096137fcd5cc0131 100644
--- a/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts
+++ b/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts
@@ -4,54 +4,45 @@
 
 / {
        compatible = "brcm,bcm2835", "brcm,bcm2708", "brcm,bcm2709";
+};
+
+&spi0 {
+       #address-cells = <1>;
+       #size-cells = <0>;
+       status = "okay";
+
+       spidev@0{
+               reg = <0>;
+               status = "disabled";
+       };
 
-       fragment@0 {
-               target = <&spi0>;
-               __overlay__ {
-                       #address-cells = <1>;
-                       #size-cells = <0>;
-                       status = "okay";
-
-                       spidev@0{
-                               reg = <0>;
-                               status = "disabled";
-                       };
-
-                       spidev@1{
-                               reg = <1>;
-                               status = "disabled";
-                       };
-               };
+       spidev@1{
+               reg = <1>;
+               status = "disabled";
        };
+};
 
-       fragment@1 {
-               target = <&gpio>;
-               __overlay__ {
-                       pi433_pins: pi433_pins {
-                               brcm,pins = <7 25 24>;
-                               brcm,function = <0 0 0>; // in in in
-                       };
-               };
+&gpio {
+       pi433_pins: pi433_pins {
+               brcm,pins = <7 25 24>;
+               brcm,function = <0 0 0>; // in in in
        };
+};
 
-       fragment@2 {
-               target = <&spi0>;
-               __overlay__ {
-                       #address-cells = <1>;
-                       #size-cells = <0>;
-                       status = "okay";
-
-                       pi433: pi433@0 {
-                               compatible = "Smarthome-Wolf,pi433";
-                               reg = <0>;
-                               spi-max-frequency = <10000000>;
-                               status = "okay";
-
-                               pinctrl-0 = <&pi433_pins>;
-                               DIO0-gpio = <&gpio 24 0>;
-                               DIO1-gpio = <&gpio 25 0>;
-                               DIO2-gpio = <&gpio  7 0>;
-                       };
-               };
+&spi0 {
+       #address-cells = <1>;
+       #size-cells = <0>;
+       status = "okay";
+
+       pi433: pi433@0 {
+               compatible = "Smarthome-Wolf,pi433";
+               reg = <0>;
+               spi-max-frequency = <10000000>;
+               status = "okay";
+
+               pinctrl-0 = <&pi433_pins>;
+               DIO0-gpio = <&gpio 24 0>;
+               DIO1-gpio = <&gpio 25 0>;
+               DIO2-gpio = <&gpio  7 0>;
        };
 };
-- 
2.17.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to