[PATCH 00/38] arm: dts: qcom: Format with dtfmt

[PATCH 00/38] arm: dts: qcom: Format with dtfmt

[Waqar Hameed]

I was looking through the notes for the last Devicetree microconference
at Linux Plumbers (LPC) 2025 [1], and realized that the tool I have been
working on might be of interest here: dtfmt - a Devicetree Source
formatter [2].

dtfmt uses a parser with lexical grammar (tree-sitter) directly ported
from the actual Device Tree Compiler's (dtc) lexer/parser, also
(hand)written by me [3]. It is primarily designed for the Linux kernel
and currently parses all 6872 devicetree files in the tree successfully
(found with `find . -iregex ".+\.dts[io]?$"`).

This tool was developed out of frustration when reviewing devicetree
patches internally at my company Axis. I would have published it earlier
if I was aware of the on-going discussions at LPC 2025. It only caught
my attention since I'm planning to attend LPC this year (especially the
Devicetree microconference and would therefore love to discuss this more
in-person!).

This patch series is only a small demonstration of dtfmt, primarily to
get Konrad Dybcio in the "To:"-field (who did the presentation on LPC
2025). Obviously one can run dtfmt for all files and generate a much
bigger diff. There have been patches sent for
`arch/arm64/boot/dts/exynos/` previously [4].

Please note that since we don't currently have a _definitive_ style
guide for Devictree Source files, dtfmt tries to be pragmatic and try to
"guess" the intended style (e.g. alignment of arrays and comma separated
property data). Some things have been left out (for now) or manually
edited, because it's too hard to implement without any concrete style
decisions in place (e.g. alignment of comments, long line wrapping
etc.). dtfmt will continue to develop and will always align with the
Linux kernel. Feedback (and collaboration) is therefore greatly
appreciated!

[1] https://lpc.events/event/19/sessions/237/attachments/1713/4459/Notes%20-%20Devicetree%20MC%20-%20LPC%202025.pdf
[2] https://codeberg.org/whame/dtfmt
    Mirror: https://github.com/whame/dtfmt
[3] https://codeberg.org/whame/tree-sitter-devicetree
    Mirror: https://github.com/whame/tree-sitter-devicetree
[4] https://lore.kernel.org/lkml/cover.1790629313.git.waqar.hameed@axis.com/

Waqar Hameed (38):
  arm: dts: qcom: pm8226: Add newline between subnodes
  arm: dts: qcom: pm8841: Remove superfluous empty lines
  arm: dts: qcom: pm8921: Remove superfluous empty lines
  arm: dts: qcom: pm8941: Remove superfluous empty lines
  arm: dts: qcom: pm8084: Remove superfluous empty line
  arm: dts: qcom: apq8026: Add newline between subnodes
  arm: dts: qcom: apq8026: Add space between OR-operator
  arm: dts: qcom: apq8060: Add newline between subnodes
  arm: dts: qcom: apq8060: Align multiple property data
  arm: dts: qcom: apq8060: Indent property data with tab
  arm: dts: qcom: apq8064: Add newline between subnodes
  arm: dts: qcom: apq8064: Remove empty line at end of node
  arm: dts: qcom: apq8064: Add space before line comments
  arm: dts: qcom: apq8064: Align multiple property data
  arm: dts: qcom: apq8064: Remove tab after property name
  arm: dts: qcom: ipq4018: Indent closing brace properly
  arm: dts: qcom: ipq4019: Add newline between subnodes
  arm: dts: qcom: ipq4019: Align multiple property data
  arm: dts: qcom: ipq4019: Break long property array data
  arm: dts: qcom: ipq8064: Add newline between subnodes
  arm: dts: qcom: ipq8064: Align multiple property data
  arm: dts: qcom: ipq8064: Break long property array data
  arm: dts: qcom: msm8226: Add space between OR-operator
  arm: dts: qcom: msm8226: Add newline between subnodes
  arm: dts: qcom: ipq8064: Break long property array data
  arm: dts: qcom: msm8226: Align multiple property data
  arm: dts: qcom: msm8660: Remove empty line at end of node
  arm: dts: qcom: msm8660: Remove space inside array property
  arm: dts: qcom: msm8660: Align closing array bracket
  arm: dts: qcom: msm8916: Add newline between subnodes
  arm: dts: qcom: msm8960: Properly indent subnodes
  arm: dts: qcom: msm8660: Align closing array bracket
  arm: dts: qcom: msm8974: Align multiple property data
  arm: dts: qcom: msm8974: Align closing brace properly
  arm: dts: qcom: msm8974: Add newline between subnodes
  arm: dts: qcom: sdx55: Properly indent subnode
  arm: dts: qcom: sdx65: Remove empty line at end of node
  arm: dts: qcom: sdx65: Align multiple property data

 arch/arm/boot/dts/qcom/pm8226.dtsi            |   5 +
 arch/arm/boot/dts/qcom/pm8841.dtsi            |   2 -
 arch/arm/boot/dts/qcom/pm8921.dtsi            |   2 -
 arch/arm/boot/dts/qcom/pm8941.dtsi            |   3 -
 arch/arm/boot/dts/qcom/pma8084.dtsi           |   1 -
 .../dts/qcom/qcom-apq8026-asus-sparrow.dts    |  32 +++++
 .../boot/dts/qcom/qcom-apq8026-lg-lenok.dts   |  27 +++++
 .../qcom/qcom-apq8026-samsung-milletwifi.dts  |   4 +-
 .../dts/qcom/qcom-apq8060-dragonboard.dts     |  54 ++++++---
 .../dts/qcom/qcom-apq8064-asus-nexus7-flo.dts |   7 +-
 .../qcom-apq8064-sony-xperia-lagan-yuga.dts   |   1 -
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi      | 114 +++++++++---------
 .../boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi |   3 +-
 .../boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi |   1 +
 .../boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi |  11 +-
 .../dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts    |   3 +-
 arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi      |  16 ++-
 arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi |   1 +
 arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi |   1 +
 arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi      |  29 +++--
 .../qcom-msm8226-samsung-matisse-common.dtsi  |   4 +-
 arch/arm/boot/dts/qcom/qcom-msm8226.dtsi      |  18 ++-
 arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts  |   2 +-
 arch/arm/boot/dts/qcom/qcom-msm8660.dtsi      |   6 +-
 arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi  |   3 +
 arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts   |  18 +--
 arch/arm/boot/dts/qcom/qcom-msm8974.dtsi      |  24 +++-
 arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts     |  10 +-
 arch/arm/boot/dts/qcom/qcom-sdx65.dtsi        |  14 +--
 29 files changed, 264 insertions(+), 152 deletions(-)


base-commit: 6c2cb8b8b843d216ab549b678a0d8831c43153e0
-- 
2.43.0

[PATCH 02/38] arm: dts: qcom: pm8841: Remove superfluous empty lines

[Waqar Hameed]

There shouldn't be a newline at the start of node definition or multiple
empty lines. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/pm8841.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/pm8841.dtsi b/arch/arm/boot/dts/qcom/pm8841.dtsi
index 3bf2ce5c86a64..49316e971fd3a 100644
--- a/arch/arm/boot/dts/qcom/pm8841.dtsi
+++ b/arch/arm/boot/dts/qcom/pm8841.dtsi
@@ -2,7 +2,6 @@
 #include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/spmi/spmi.h>
 
-
 / {
 	thermal-zones {
 		pm8841-thermal {
@@ -34,7 +33,6 @@ crit {
 };
 
 &spmi_bus {
-
 	pm8841_0: pm8841@4 {
 		compatible = "qcom,pm8841", "qcom,spmi-pmic";
 		reg = <0x4 SPMI_USID>;
-- 
2.43.0

[PATCH 01/38] arm: dts: qcom: pm8226: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/pm8226.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/pm8226.dtsi b/arch/arm/boot/dts/qcom/pm8226.dtsi
index 2fd4f135ed84c..abab3c6e253c2 100644
--- a/arch/arm/boot/dts/qcom/pm8226.dtsi
+++ b/arch/arm/boot/dts/qcom/pm8226.dtsi
@@ -109,21 +109,26 @@ channel@7 {
 				qcom,pre-scaling = <1 3>;
 				label = "vph_pwr";
 			};
+
 			channel@8 {
 				reg = <VADC_DIE_TEMP>;
 				label = "die_temp";
 			};
+
 			channel@9 {
 				reg = <VADC_REF_625MV>;
 				label = "ref_625mv";
 			};
+
 			channel@a {
 				reg = <VADC_REF_1250MV>;
 				label = "ref_1250mv";
 			};
+
 			channel@e {
 				reg = <VADC_GND_REF>;
 			};
+
 			channel@f {
 				reg = <VADC_VDD_VADC>;
 			};
-- 
2.43.0

[PATCH 03/38] arm: dts: qcom: pm8921: Remove superfluous empty lines

[Waqar Hameed]

There shouldn't be a newline at the start or end of node definition.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/pm8921.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/pm8921.dtsi b/arch/arm/boot/dts/qcom/pm8921.dtsi
index 15246f4bd2672..7b177b65c6b61 100644
--- a/arch/arm/boot/dts/qcom/pm8921.dtsi
+++ b/arch/arm/boot/dts/qcom/pm8921.dtsi
@@ -53,7 +53,6 @@ pm8921_keypad: keypad@148 {
 		};
 
 		pm8921_gpio: gpio@150 {
-
 			compatible = "qcom,pm8921-gpio",
 				     "qcom,ssbi-gpio";
 			reg = <0x150>;
@@ -62,7 +61,6 @@ pm8921_gpio: gpio@150 {
 			gpio-controller;
 			gpio-ranges = <&pm8921_gpio 0 0 44>;
 			#gpio-cells = <2>;
-
 		};
 
 		pm8921_xoadc: xoadc@197 {
-- 
2.43.0

[PATCH 04/38] arm: dts: qcom: pm8941: Remove superfluous empty lines

[Waqar Hameed]

There shouldn't be a newline at the start of node definition or multiple
empty lines. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/pm8941.dtsi | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/pm8941.dtsi b/arch/arm/boot/dts/qcom/pm8941.dtsi
index aca0052a02b75..e48803fa74333 100644
--- a/arch/arm/boot/dts/qcom/pm8941.dtsi
+++ b/arch/arm/boot/dts/qcom/pm8941.dtsi
@@ -3,7 +3,6 @@
 #include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/spmi/spmi.h>
 
-
 / {
 	thermal-zones {
 		pm8941-thermal {
@@ -35,7 +34,6 @@ crit {
 };
 
 &spmi_bus {
-
 	pm8941_0: pm8941@0 {
 		compatible = "qcom,pm8941", "qcom,spmi-pmic";
 		reg = <0x0 SPMI_USID>;
@@ -146,7 +144,6 @@ pm8941_vadc: adc@3100 {
 			#size-cells = <0>;
 			#io-channel-cells = <1>;
 
-
 			channel@6 {
 				reg = <VADC_VBAT_SNS>;
 			};
-- 
2.43.0

[PATCH 05/38] arm: dts: qcom: pm8084: Remove superfluous empty line

[Waqar Hameed]

There shouldn't be a newline at the start of node definition.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/pma8084.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/pma8084.dtsi b/arch/arm/boot/dts/qcom/pma8084.dtsi
index 309f5256754bb..f0a0bc65653fe 100644
--- a/arch/arm/boot/dts/qcom/pma8084.dtsi
+++ b/arch/arm/boot/dts/qcom/pma8084.dtsi
@@ -5,7 +5,6 @@
 #include <dt-bindings/spmi/spmi.h>
 
 &spmi_bus {
-
 	pma8084_0: pma8084@0 {
 		compatible = "qcom,pma8084", "qcom,spmi-pmic";
 		reg = <0x0 SPMI_USID>;
-- 
2.43.0

[PATCH 06/38] arm: dts: qcom: apq8026: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 .../dts/qcom/qcom-apq8026-asus-sparrow.dts    | 32 +++++++++++++++++++
 .../boot/dts/qcom/qcom-apq8026-lg-lenok.dts   | 27 ++++++++++++++++
 2 files changed, 59 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8026-asus-sparrow.dts b/arch/arm/boot/dts/qcom/qcom-apq8026-asus-sparrow.dts
index a2ca456012f1a..4340b4581e011 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8026-asus-sparrow.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8026-asus-sparrow.dts
@@ -22,22 +22,27 @@ sbl_region: sbl@2f00000 {
 			reg = <0x02f00000 0x100000>;
 			no-map;
 		};
+
 		external_image_region: external-image@3100000 {
 			reg = <0x3100000 0x200000>;
 			no-map;
 		};
+
 		peripheral_region: peripheral@3300000 {
 			reg = <0x3300000 0x600000>;
 			no-map;
 		};
+
 		adsp_region: adsp@3900000 {
 			reg = <0x3900000 0x1400000>;
 			no-map;
 		};
+
 		modem_region: modem@4d00000 {
 			reg = <0x4d00000 0x1b00000>;
 			no-map;
 		};
+
 		modem_efs_region: modem-efs@7f00000 {
 			reg = <0x7f00000 0x100000>;
 			no-map;
@@ -94,10 +99,12 @@ pm8226_s3: s3 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1300000>;
 		};
+
 		pm8226_s4: s4 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2200000>;
 		};
+
 		pm8226_s5: s5 {
 			regulator-min-microvolt = <1150000>;
 			regulator-max-microvolt = <1150000>;
@@ -107,102 +114,127 @@ pm8226_l1: l1 {
 			regulator-min-microvolt = <1225000>;
 			regulator-max-microvolt = <1225000>;
 		};
+
 		pm8226_l2: l2 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l3: l3 {
 			regulator-min-microvolt = <750000>;
 			regulator-max-microvolt = <1337500>;
 		};
+
 		pm8226_l4: l4 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l5: l5 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l6: l6 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l7: l7 {
 			regulator-min-microvolt = <1900000>;
 			regulator-max-microvolt = <1900000>;
 		};
+
 		pm8226_l8: l8 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l9: l9 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l10: l10 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l12: l12 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l14: l14 {
 			regulator-min-microvolt = <2750000>;
 			regulator-max-microvolt = <2750000>;
 		};
+
 		pm8226_l15: l15 {
 			regulator-min-microvolt = <3000000>;
 			regulator-max-microvolt = <3000000>;
 		};
+
 		pm8226_l16: l16 {
 			regulator-min-microvolt = <3000000>;
 			regulator-max-microvolt = <3000000>;
 		};
+
 		pm8226_l17: l17 {
 			regulator-min-microvolt = <2950000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l18: l18 {
 			regulator-min-microvolt = <2950000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l19: l19 {
 			regulator-min-microvolt = <2850000>;
 			regulator-max-microvolt = <2850000>;
 		};
+
 		pm8226_l20: l20 {
 			regulator-min-microvolt = <3075000>;
 			regulator-max-microvolt = <3075000>;
 		};
+
 		pm8226_l21: l21 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l22: l22 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l23: l23 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l24: l24 {
 			regulator-min-microvolt = <1300000>;
 			regulator-max-microvolt = <1350000>;
 		};
+
 		pm8226_l25: l25 {
 			regulator-min-microvolt = <1775000>;
 			regulator-max-microvolt = <2125000>;
 		};
+
 		pm8226_l26: l26 {
 			regulator-min-microvolt = <1225000>;
 			regulator-max-microvolt = <1225000>;
 		};
+
 		pm8226_l27: l27 {
 			regulator-min-microvolt = <2050000>;
 			regulator-max-microvolt = <2050000>;
 		};
+
 		pm8226_l28: l28 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2950000>;
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8026-lg-lenok.dts b/arch/arm/boot/dts/qcom/qcom-apq8026-lg-lenok.dts
index a70de21bf139b..6184bb47a5731 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8026-lg-lenok.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8026-lg-lenok.dts
@@ -157,10 +157,12 @@ pm8226_s3: s3 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1350000>;
 		};
+
 		pm8226_s4: s4 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2200000>;
 		};
+
 		pm8226_s5: s5 {
 			regulator-min-microvolt = <1150000>;
 			regulator-max-microvolt = <1150000>;
@@ -170,102 +172,127 @@ pm8226_l1: l1 {
 			regulator-min-microvolt = <1225000>;
 			regulator-max-microvolt = <1225000>;
 		};
+
 		pm8226_l2: l2 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l3: l3 {
 			regulator-min-microvolt = <750000>;
 			regulator-max-microvolt = <1337500>;
 		};
+
 		pm8226_l4: l4 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l5: l5 {
 			regulator-min-microvolt = <1200000>;
 			regulator-max-microvolt = <1200000>;
 		};
+
 		pm8226_l6: l6 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l7: l7 {
 			regulator-min-microvolt = <1850000>;
 			regulator-max-microvolt = <1850000>;
 		};
+
 		pm8226_l8: l8 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l9: l9 {
 			regulator-min-microvolt = <2050000>;
 			regulator-max-microvolt = <2050000>;
 		};
+
 		pm8226_l10: l10 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l12: l12 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l14: l14 {
 			regulator-min-microvolt = <2750000>;
 			regulator-max-microvolt = <2750000>;
 		};
+
 		pm8226_l15: l15 {
 			regulator-min-microvolt = <3300000>;
 			regulator-max-microvolt = <3300000>;
 		};
+
 		pm8226_l16: l16 {
 			regulator-min-microvolt = <3000000>;
 			regulator-max-microvolt = <3350000>;
 		};
+
 		pm8226_l17: l17 {
 			regulator-min-microvolt = <2950000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l18: l18 {
 			regulator-min-microvolt = <3000000>;
 			regulator-max-microvolt = <3300000>;
 		};
+
 		pm8226_l19: l19 {
 			regulator-min-microvolt = <3000000>;
 			regulator-max-microvolt = <3000000>;
 		};
+
 		pm8226_l20: l20 {
 			regulator-min-microvolt = <3075000>;
 			regulator-max-microvolt = <3075000>;
 		};
+
 		pm8226_l21: l21 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l22: l22 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <1800000>;
 		};
+
 		pm8226_l23: l23 {
 			regulator-min-microvolt = <1800000>;
 			regulator-max-microvolt = <2950000>;
 		};
+
 		pm8226_l24: l24 {
 			regulator-min-microvolt = <1300000>;
 			regulator-max-microvolt = <1350000>;
 		};
+
 		pm8226_l25: l25 {
 			regulator-min-microvolt = <1775000>;
 			regulator-max-microvolt = <2125000>;
 		};
+
 		pm8226_l26: l26 {
 			regulator-min-microvolt = <1225000>;
 			regulator-max-microvolt = <1225000>;
 		};
+
 		pm8226_l27: l27 {
 			regulator-min-microvolt = <2050000>;
 			regulator-max-microvolt = <2050000>;
 		};
+
 		pm8226_l28: l28 {
 			regulator-min-microvolt = <2700000>;
 			regulator-max-microvolt = <3000000>;
-- 
2.43.0

[PATCH 07/38] arm: dts: qcom: apq8026: Add space between OR-operator

[Waqar Hameed]

There should be a space between binary operators. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8026-samsung-milletwifi.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8026-samsung-milletwifi.dts b/arch/arm/boot/dts/qcom/qcom-apq8026-samsung-milletwifi.dts
index a8543ca7b5569..da0b5969c8320 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8026-samsung-milletwifi.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8026-samsung-milletwifi.dts
@@ -86,8 +86,8 @@ key-volume-up {
 
 	i2c-backlight {
 		compatible = "i2c-gpio";
-		sda-gpios = <&tlmm 20 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
-		scl-gpios = <&tlmm 21 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
+		sda-gpios = <&tlmm 20 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+		scl-gpios = <&tlmm 21 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
 
 		pinctrl-0 = <&backlight_i2c_default_state>;
 		pinctrl-names = "default";
-- 
2.43.0

[PATCH 08/38] arm: dts: qcom: apq8060: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 .../dts/qcom/qcom-apq8060-dragonboard.dts     | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
index 009afd8212c22..fa2b372e51bf3 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
@@ -181,6 +181,7 @@ eeprom@52 {
 		vcc-supply = <&pm8058_s3>;
 		pagesize = <64>;
 	};
+
 	wm8903: wm8903@1a {
 		/* This Woolfson Micro device has an unrouted interrupt line */
 		compatible = "wlf,wm8903";
@@ -226,6 +227,7 @@ ak8975@c {
 		vid-supply = <&pm8058_lvs0>; // 1.8V
 		vdd-supply = <&pm8058_l14>; // 2.85V
 	};
+
 	bmp085@77 {
 		compatible = "bosch,bmp085";
 		reg = <0x77>;
@@ -237,6 +239,7 @@ bmp085@77 {
 		vddd-supply = <&pm8058_lvs0>; // 1.8V
 		vdda-supply = <&pm8058_l14>; // 2.85V
 	};
+
 	mpu3050@68 {
 		compatible = "invensense,mpu3050";
 		reg = <0x68>;
@@ -287,6 +290,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_bmp085_gpios: bmp085-state {
 		pinconf {
 			pins = "gpio16";
@@ -296,6 +300,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_mpu3050_gpios: mpu3050-state {
 		pinconf {
 			pins = "gpio17";
@@ -305,6 +310,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_sdcc3_gpios: sdcc3-state {
 		pinconf {
 			pins = "gpio22";
@@ -314,6 +320,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_sdcc5_gpios: sdcc5-state {
 		pinconf {
 			pins = "gpio26";
@@ -324,6 +331,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_ak8975_gpios: ak8975-state {
 		pinconf {
 			pins = "gpio33";
@@ -333,6 +341,7 @@ pinconf {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_cm3605_gpios: cm3605-state {
 		/* Pin 34 connected to the proxy IRQ */
 		gpio34-pins {
@@ -342,6 +351,7 @@ gpio34-pins {
 			bias-disable;
 			power-source = <PM8058_GPIO_S3>;
 		};
+
 		/* Pin 35 connected to ASET */
 		gpio35-pins {
 			pins = "gpio35";
@@ -351,6 +361,7 @@ gpio35-pins {
 			power-source = <PM8058_GPIO_S3>;
 		};
 	};
+
 	dragon_veth_gpios: veth-state {
 		pinconf {
 			pins = "gpio40";
@@ -521,12 +532,14 @@ pm8901_s2: s2 {
 			qcom,switch-mode-frequency = <1600000>;
 			bias-pull-down;
 		};
+
 		pm8901_s3: s3 {
 			regulator-min-microvolt = <1100000>;
 			regulator-max-microvolt = <1100000>;
 			qcom,switch-mode-frequency = <1600000>;
 			bias-pull-down;
 		};
+
 		pm8901_s4: s4 {
 			regulator-min-microvolt = <1225000>;
 			regulator-max-microvolt = <1225000>;
@@ -837,12 +850,14 @@ clk-pins {
 			drive-strength = <16>;
 			bias-disable;
 		};
+
 		cmd-pins {
 			pins = "gpio168"; /* SDC1 CMD */
 			function = "sdc1";
 			drive-strength = <10>;
 			bias-pull-up;
 		};
+
 		data-pins {
 			/* SDC1 D0 to D7 */
 			pins = "gpio159", "gpio160", "gpio161", "gpio162",
@@ -863,11 +878,13 @@ clk-pins {
 			drive-strength = <8>;
 			bias-disable;
 		};
+
 		cmd-pins {
 			pins = "sdc3_cmd";
 			drive-strength = <8>;
 			bias-pull-up;
 		};
+
 		data-pins {
 			pins = "sdc3_data";
 			drive-strength = <8>;
@@ -883,12 +900,14 @@ clk-pins {
 			drive-strength = <16>;
 			bias-disable;
 		};
+
 		cmd-pins {
 			pins = "gpio95"; /* SDC5 CMD */
 			function = "sdc5";
 			drive-strength = <10>;
 			bias-pull-up;
 		};
+
 		data-pins {
 			/* SDC5 D0 to D3 */
 			pins = "gpio96", "gpio98", "gpio99", "gpio100";
@@ -930,6 +949,7 @@ tx-pins {
 			drive-strength = <8>;
 			bias-disable;
 		};
+
 		rx-pins {
 			pins = "gpio118";
 			function = "gsbi12";
@@ -951,6 +971,7 @@ mux0-pins {
 			    "gpio134"; /* CS0_N testpoint TP29 */
 			function = "ebi2cs";
 		};
+
 		mux1-pins {
 			pins =
 			    /* EBI2_ADDR_7 downto EBI2_ADDR_0 address bus */
@@ -983,6 +1004,7 @@ reset-pins {
 			drive-strength = <6>;
 			bias-disable;
 		};
+
 		irq-pins {
 			pins = "gpio61"; /* IRQ line */
 			function = "gpio";
-- 
2.43.0

[PATCH 09/38] arm: dts: qcom: apq8060: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
index fa2b372e51bf3..4dc63032cccb6 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
@@ -99,7 +99,7 @@ ethernet@2,0 {
 		 * for network wakeup, connected to the TLMM.
 		 */
 		interrupts-extended = <&pm8058_gpio 7 IRQ_TYPE_EDGE_FALLING>,
-				    <&tlmm 29 IRQ_TYPE_EDGE_RISING>;
+				      <&tlmm 29 IRQ_TYPE_EDGE_RISING>;
 		reset-gpios = <&tlmm 30 GPIO_ACTIVE_LOW>;
 		vdd33a-supply = <&dragon_veth>;
 		vddvario-supply = <&dragon_vario>;
@@ -861,7 +861,7 @@ cmd-pins {
 		data-pins {
 			/* SDC1 D0 to D7 */
 			pins = "gpio159", "gpio160", "gpio161", "gpio162",
-			     "gpio163", "gpio164", "gpio165", "gpio166";
+			       "gpio163", "gpio164", "gpio165", "gpio166";
 			function = "sdc1";
 			drive-strength = <10>;
 			bias-pull-up;
-- 
2.43.0

[PATCH 11/38] arm: dts: qcom: apq8064: Add newline between subnodes

[Waqar Hameed]

There should be a newline (and only one) between subnodes. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 .../boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts    |  2 ++
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi              | 11 +++++++++--
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
index 9471839928505..bdd3dab530ece 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
@@ -53,6 +53,7 @@ key-volume-up {
 			gpios = <&pm8921_gpio 4 GPIO_ACTIVE_HIGH>;
 			linux,code = <KEY_VOLUMEUP>;
 		};
+
 		key-volume-down {
 			label = "Volume Down";
 			gpios = <&pm8921_gpio 38 GPIO_ACTIVE_HIGH>;
@@ -330,6 +331,7 @@ pm8921_lvs5: lvs5 {
 		pm8921_lvs6: lvs6 {
 			bias-pull-down;
 		};
+
 		/*
 		 * mipi_dsi.1-dsi1_vddio
 		 * pil_riva-pll_vdd
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
index 09062b2ad8ba5..0423b9380c812 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
@@ -116,6 +116,7 @@ cpu_alert0: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit0: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -137,6 +138,7 @@ cpu_alert1: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit1: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -158,6 +160,7 @@ cpu_alert2: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit2: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -179,6 +182,7 @@ cpu_alert3: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit3: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -707,6 +711,7 @@ qfprom: efuse@700000 {
 			tsens_calib: calib@404 {
 				reg = <0x404 0x10>;
 			};
+
 			tsens_backup: backup-calib@414 {
 				reg = <0x414 0x10>;
 			};
@@ -1177,7 +1182,6 @@ dsi0_out: endpoint {
 			};
 		};
 
-
 		dsi0_phy: phy@4700200 {
 			compatible = "qcom,dsi-phy-28nm-8960";
 			#clock-cells = <1>;
@@ -1249,7 +1253,6 @@ dsi1_out: endpoint {
 			};
 		};
 
-
 		dsi1_phy: dsi-phy@5800200 {
 			compatible = "qcom,dsi-phy-28nm-8960";
 			reg = <0x05800200 0x100>,
@@ -1596,18 +1599,21 @@ funnel_in0: endpoint {
 						remote-endpoint = <&etm0_out>;
 					};
 				};
+
 				port@1 {
 					reg = <1>;
 					funnel_in1: endpoint {
 						remote-endpoint = <&etm1_out>;
 					};
 				};
+
 				port@4 {
 					reg = <4>;
 					funnel_in4: endpoint {
 						remote-endpoint = <&etm2_out>;
 					};
 				};
+
 				port@5 {
 					reg = <5>;
 					funnel_in5: endpoint {
@@ -1698,4 +1704,5 @@ etm3_out: endpoint {
 		};
 	};
 };
+
 #include "qcom-apq8064-pins.dtsi"
-- 
2.43.0

[PATCH 10/38] arm: dts: qcom: apq8060: Indent property data with tab

[Waqar Hameed]

Multiple property data on a newline should be indented with tabs.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 .../dts/qcom/qcom-apq8060-dragonboard.dts     | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
index 4dc63032cccb6..6ede4e5af4494 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts
@@ -966,25 +966,25 @@ dragon_ebi2_pins: ebi2-state {
 		 */
 		mux0-pins {
 			pins =
-			    /* "gpio39", CS1A_N this is not good to mux */
-			    "gpio40", /* CS2A_N */
-			    "gpio134"; /* CS0_N testpoint TP29 */
+				/* "gpio39", CS1A_N this is not good to mux */
+				"gpio40", /* CS2A_N */
+				"gpio134"; /* CS0_N testpoint TP29 */
 			function = "ebi2cs";
 		};
 
 		mux1-pins {
 			pins =
-			    /* EBI2_ADDR_7 downto EBI2_ADDR_0 address bus */
-			    "gpio123", "gpio124", "gpio125", "gpio126",
-			    "gpio127", "gpio128", "gpio129", "gpio130",
-			    /* EBI2_DATA_15 downto EBI2_DATA_0 data bus */
-			    "gpio135", "gpio136", "gpio137", "gpio138",
-			    "gpio139", "gpio140", "gpio141", "gpio142",
-			    "gpio143", "gpio144", "gpio145", "gpio146",
-			    "gpio147", "gpio148", "gpio149", "gpio150",
-			    "gpio151", /* EBI2_OE_N */
-			    "gpio153", /* EBI2_ADV */
-			    "gpio157"; /* EBI2_WE_N */
+				/* EBI2_ADDR_7 downto EBI2_ADDR_0 address bus */
+				"gpio123", "gpio124", "gpio125", "gpio126",
+				"gpio127", "gpio128", "gpio129", "gpio130",
+				/* EBI2_DATA_15 downto EBI2_DATA_0 data bus */
+				"gpio135", "gpio136", "gpio137", "gpio138",
+				"gpio139", "gpio140", "gpio141", "gpio142",
+				"gpio143", "gpio144", "gpio145", "gpio146",
+				"gpio147", "gpio148", "gpio149", "gpio150",
+				"gpio151", /* EBI2_OE_N */
+				"gpio153", /* EBI2_ADV */
+				"gpio157"; /* EBI2_WE_N */
 			function = "ebi2";
 		};
 	};
-- 
2.43.0

[PATCH 12/38] arm: dts: qcom: apq8064: Remove empty line at end of node

[Waqar Hameed]

There shouldn't be an empty line at end of node definition. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts        | 1 -
 arch/arm/boot/dts/qcom/qcom-apq8064-sony-xperia-lagan-yuga.dts | 1 -
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi                       | 1 -
 3 files changed, 3 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
index bdd3dab530ece..4111834b12012 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
@@ -137,7 +137,6 @@ bq27541@55 {
 		compatible = "ti,bq27541";
 		reg = <0x55>;
 	};
-
 };
 
 &gsbi3 {
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-sony-xperia-lagan-yuga.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-sony-xperia-lagan-yuga.dts
index 7752f07973f9e..28bd88eb5df55 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064-sony-xperia-lagan-yuga.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064-sony-xperia-lagan-yuga.dts
@@ -395,7 +395,6 @@ tx-pins {
 		};
 	};
 
-
 	sdcc3_cd_pin_a: sdcc3-cd-pin-active-state {
 		pins = "gpio26";
 		function = "gpio";
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
index 0423b9380c812..150b5670e13b3 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
@@ -482,7 +482,6 @@ gsbi1_i2c: i2c@12460000 {
 				#size-cells = <0>;
 				status = "disabled";
 			};
-
 		};
 
 		gsbi2: gsbi@12480000 {
-- 
2.43.0

[PATCH 13/38] arm: dts: qcom: apq8064: Add space before line comments

[Waqar Hameed]

There should be a space before start of comment delimiter. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
index 4111834b12012..1ae9de453ccf9 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
@@ -79,7 +79,7 @@ reboot-mode {
 };
 
 &dsi0 {
-	vdda-supply = <&pm8921_l2>;/*VDD_MIPI1 to 4*/
+	vdda-supply = <&pm8921_l2>; /* VDD_MIPI1 to 4 */
 	vdd-supply = <&pm8921_l8>;
 	vddio-supply = <&pm8921_lvs7>;
 	avdd-supply = <&pm8921_l11>;
@@ -114,7 +114,7 @@ &dsi0_out {
 };
 
 &dsi0_phy {
-	vddio-supply = <&pm8921_lvs7>;/*VDD_PLL2_1 to 7*/
+	vddio-supply = <&pm8921_lvs7>; /* VDD_PLL2_1 to 7 */
 	status = "okay";
 };
 
-- 
2.43.0

[PATCH 14/38] arm: dts: qcom: apq8064: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first one
or properly indented with tabs. Found/fixed with dtfmt (Devicetree
Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi | 98 ++++++++++++------------
 1 file changed, 49 insertions(+), 49 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
index 150b5670e13b3..565f98c00b3fb 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
@@ -89,7 +89,7 @@ l2: l2-cache {
 		idle-states {
 			cpu_spc: cpu-spc {
 				compatible = "qcom,idle-state-spc",
-						"arm,idle-state";
+					     "arm,idle-state";
 				entry-latency-us = <400>;
 				exit-latency-us = <900>;
 				min-residency-us = <3000>;
@@ -1036,15 +1036,15 @@ gpu: gpu@4300000 {
 			interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-names = "kgsl_3d0_irq";
 			clock-names =
-			    "core",
-			    "iface",
-			    "mem",
-			    "mem_iface";
+				"core",
+				"iface",
+				"mem",
+				"mem_iface";
 			clocks =
-			    <&mmcc GFX3D_CLK>,
-			    <&mmcc GFX3D_AHB_CLK>,
-			    <&mmcc GFX3D_AXI_CLK>,
-			    <&mmcc MMSS_IMEM_AHB_CLK>;
+				<&mmcc GFX3D_CLK>,
+				<&mmcc GFX3D_AHB_CLK>,
+				<&mmcc GFX3D_AXI_CLK>,
+				<&mmcc MMSS_IMEM_AHB_CLK>;
 
 			iommus = <&gfx3d 0
 				  &gfx3d 1
@@ -1141,24 +1141,24 @@ dsi0: dsi@4700000 {
 			reg-names = "dsi_ctrl";
 
 			clocks = <&mmcc DSI_M_AHB_CLK>,
-				<&mmcc DSI_S_AHB_CLK>,
-				<&mmcc AMP_AHB_CLK>,
-				<&mmcc DSI_CLK>,
-				<&mmcc DSI1_BYTE_CLK>,
-				<&mmcc DSI_PIXEL_CLK>,
-				<&mmcc DSI1_ESC_CLK>;
+				 <&mmcc DSI_S_AHB_CLK>,
+				 <&mmcc AMP_AHB_CLK>,
+				 <&mmcc DSI_CLK>,
+				 <&mmcc DSI1_BYTE_CLK>,
+				 <&mmcc DSI_PIXEL_CLK>,
+				 <&mmcc DSI1_ESC_CLK>;
 			clock-names = "iface", "bus", "core_mmss",
-					"src", "byte", "pixel",
-					"core";
+				      "src", "byte", "pixel",
+				      "core";
 
 			assigned-clocks = <&mmcc DSI1_BYTE_SRC>,
-					<&mmcc DSI1_ESC_SRC>,
-					<&mmcc DSI_SRC>,
-					<&mmcc DSI_PIXEL_SRC>;
+					  <&mmcc DSI1_ESC_SRC>,
+					  <&mmcc DSI_SRC>,
+					  <&mmcc DSI_PIXEL_SRC>;
 			assigned-clock-parents = <&dsi0_phy 0>,
-						<&dsi0_phy 0>,
-						<&dsi0_phy 1>,
-						<&dsi0_phy 1>;
+						 <&dsi0_phy 0>,
+						 <&dsi0_phy 1>,
+						 <&dsi0_phy 1>;
 			syscon-sfpb = <&mmss_sfpb>;
 			phys = <&dsi0_phy>;
 			status = "disabled";
@@ -1187,8 +1187,8 @@ dsi0_phy: phy@4700200 {
 			#phy-cells = <0>;
 
 			reg = <0x04700200 0x100>,
-				<0x04700300 0x200>,
-				<0x04700500 0x5c>;
+			      <0x04700300 0x200>,
+			      <0x04700500 0x5c>;
 			reg-names = "dsi_pll", "dsi_phy", "dsi_phy_regulator";
 			clock-names = "iface", "ref";
 			clocks = <&mmcc DSI_M_AHB_CLK>,
@@ -1274,15 +1274,15 @@ mdp_port0: iommu@7500000 {
 			compatible = "qcom,apq8064-iommu";
 			#iommu-cells = <1>;
 			clock-names =
-			    "smmu_pclk",
-			    "iommu_clk";
+				"smmu_pclk",
+				"iommu_clk";
 			clocks =
-			    <&mmcc SMMU_AHB_CLK>,
-			    <&mmcc MDP_AXI_CLK>;
+				<&mmcc SMMU_AHB_CLK>,
+				<&mmcc MDP_AXI_CLK>;
 			reg = <0x07500000 0x100000>;
 			interrupts =
-			    <GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>,
-			    <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
+				<GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>,
+				<GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
 			qcom,ncb = <2>;
 		};
 
@@ -1290,15 +1290,15 @@ mdp_port1: iommu@7600000 {
 			compatible = "qcom,apq8064-iommu";
 			#iommu-cells = <1>;
 			clock-names =
-			    "smmu_pclk",
-			    "iommu_clk";
+				"smmu_pclk",
+				"iommu_clk";
 			clocks =
-			    <&mmcc SMMU_AHB_CLK>,
-			    <&mmcc MDP_AXI_CLK>;
+				<&mmcc SMMU_AHB_CLK>,
+				<&mmcc MDP_AXI_CLK>;
 			reg = <0x07600000 0x100000>;
 			interrupts =
-			    <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>,
-			    <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
+				<GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>,
+				<GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
 			qcom,ncb = <2>;
 		};
 
@@ -1306,15 +1306,15 @@ gfx3d: iommu@7c00000 {
 			compatible = "qcom,apq8064-iommu";
 			#iommu-cells = <1>;
 			clock-names =
-			    "smmu_pclk",
-			    "iommu_clk";
+				"smmu_pclk",
+				"iommu_clk";
 			clocks =
-			    <&mmcc SMMU_AHB_CLK>,
-			    <&mmcc GFX3D_AXI_CLK>;
+				<&mmcc SMMU_AHB_CLK>,
+				<&mmcc GFX3D_AXI_CLK>;
 			reg = <0x07c00000 0x100000>;
 			interrupts =
-			    <GIC_SPI 69 IRQ_TYPE_LEVEL_HIGH>,
-			    <GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
+				<GIC_SPI 69 IRQ_TYPE_LEVEL_HIGH>,
+				<GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
 			qcom,ncb = <3>;
 		};
 
@@ -1322,15 +1322,15 @@ gfx3d1: iommu@7d00000 {
 			compatible = "qcom,apq8064-iommu";
 			#iommu-cells = <1>;
 			clock-names =
-			    "smmu_pclk",
-			    "iommu_clk";
+				"smmu_pclk",
+				"iommu_clk";
 			clocks =
-			    <&mmcc SMMU_AHB_CLK>,
-			    <&mmcc GFX3D_AXI_CLK>;
+				<&mmcc SMMU_AHB_CLK>,
+				<&mmcc GFX3D_AXI_CLK>;
 			reg = <0x07d00000 0x100000>;
 			interrupts =
-			    <GIC_SPI 210 IRQ_TYPE_LEVEL_HIGH>,
-			    <GIC_SPI 211 IRQ_TYPE_LEVEL_HIGH>;
+				<GIC_SPI 210 IRQ_TYPE_LEVEL_HIGH>,
+				<GIC_SPI 211 IRQ_TYPE_LEVEL_HIGH>;
 			qcom,ncb = <3>;
 		};
 
-- 
2.43.0

[PATCH 16/38] arm: dts: qcom: ipq4018: Indent closing brace properly

[Waqar Hameed]

Closing brace of a node definition should be indented like subnode
name/reference. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi
index be76bc39ac277..d78623fd6acf3 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4018-ap120c-ac.dtsi
@@ -260,8 +260,7 @@ &usb3_hs_phy {
 
 &usb3 {
 	status = "okay";
-
-	};
+};
 
 &usb3_dwc {
 	phys = <&usb3_hs_phy>;
-- 
2.43.0

[PATCH 15/38] arm: dts: qcom: apq8064: Remove tab after property name

[Waqar Hameed]

These shouldn't have these to be consistent with rest of the style.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
index 565f98c00b3fb..3c7585ee00a7d 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
@@ -916,8 +916,8 @@ sata_phy0: phy@1b400000 {
 
 		sata0: sata@29000000 {
 			compatible = "qcom,apq8064-ahci", "generic-ahci";
-			status	 = "disabled";
-			reg	 = <0x29000000 0x180>;
+			status = "disabled";
+			reg = <0x29000000 0x180>;
 			interrupts = <GIC_SPI 209 IRQ_TYPE_LEVEL_HIGH>;
 
 			clocks = <&gcc SFAB_SATA_S_H_CLK>,
-- 
2.43.0

[PATCH 17/38] arm: dts: qcom: ipq4019: Add newline between subnodes

[Waqar Hameed]

There should be a newline (and only one) between subnodes. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi   | 1 +
 arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi   | 1 +
 arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts | 1 +
 arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi             | 3 +++
 4 files changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi
index efbe89dd47937..a85c0aad7cf83 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk01.1.dtsi
@@ -47,6 +47,7 @@ spi0-pins {
 			drive-strength = <12>;
 			bias-disable;
 		};
+
 		spi0-cs-pins {
 			pins = "gpio54";
 			function = "gpio";
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
index 91e296d2ea827..e998cb72c98db 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
@@ -43,6 +43,7 @@ spi0-pins {
 					pins = "gpio13", "gpio14", "gpio15";
 					bias-disable;
 				};
+
 				spi0-cs-pins {
 					function = "gpio";
 					pins = "gpio12";
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
index 41c5874f6f976..71c5ba00ff982 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
@@ -32,6 +32,7 @@ spi0-pins {
 					pins = "gpio13", "gpio14", "gpio15";
 					bias-disable;
 				};
+
 				spio-cs-pins {
 					function = "gpio";
 					pins = "gpio12";
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
index 8eeaab1c0be11..6ff87808e2a0f 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
@@ -111,14 +111,17 @@ opp-48000000 {
 			opp-hz = /bits/ 64 <48000000>;
 			clock-latency-ns = <256000>;
 		};
+
 		opp-200000000 {
 			opp-hz = /bits/ 64 <200000000>;
 			clock-latency-ns = <256000>;
 		};
+
 		opp-500000000 {
 			opp-hz = /bits/ 64 <500000000>;
 			clock-latency-ns = <256000>;
 		};
+
 		opp-716000000 {
 			opp-hz = /bits/ 64 <716000000>;
 			clock-latency-ns = <256000>;
-- 
2.43.0

[PATCH 19/38] arm: dts: qcom: ipq4019: Break long property array data

[Waqar Hameed]

There is an awkward line break at the last elements expression. Put all
elements on a newline instead. Found/fixed with dtfmt (Devicetree Source
formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
index 1fa1aef140a12..51721fedf7d60 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
@@ -135,8 +135,9 @@ memory {
 
 	pmu {
 		compatible = "arm,cortex-a7-pmu";
-		interrupts = <GIC_PPI 7 (GIC_CPU_MASK_SIMPLE(4) |
-					 IRQ_TYPE_LEVEL_HIGH)>;
+		interrupts = <
+			GIC_PPI 7 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)
+		>;
 	};
 
 	clocks {
-- 
2.43.0

[PATCH 18/38] arm: dts: qcom: ipq4019: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi   | 10 +++++-----
 arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts |  2 +-
 arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi             |  8 ++++----
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
index e998cb72c98db..faedcc0a76387 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk04.1.dtsi
@@ -32,7 +32,7 @@ serial_0_pins: serial0-state {
 
 			serial_1_pins: serial1-state {
 				pins = "gpio8", "gpio9",
-					"gpio10", "gpio11";
+				       "gpio10", "gpio11";
 				function = "blsp_uart1";
 				bias-disable;
 			};
@@ -60,10 +60,10 @@ i2c_0_pins: i2c-0-state {
 
 			nand_pins: nand-state {
 				pins = "gpio53", "gpio55", "gpio56",
-					"gpio57", "gpio58", "gpio59",
-					"gpio60", "gpio62", "gpio63",
-					"gpio64", "gpio65", "gpio66",
-					"gpio67", "gpio68", "gpio69";
+				       "gpio57", "gpio58", "gpio59",
+				       "gpio60", "gpio62", "gpio63",
+				       "gpio64", "gpio65", "gpio66",
+				       "gpio67", "gpio68", "gpio69";
 				function = "qpic";
 			};
 		};
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
index 71c5ba00ff982..4230d71c48b16 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019-ap.dk07.1-c1.dts
@@ -21,7 +21,7 @@ spi@78b6000 {
 		pinctrl@1000000 {
 			serial_1_pins: serial1-state {
 				pins = "gpio8", "gpio9",
-					"gpio10", "gpio11";
+				       "gpio10", "gpio11";
 				function = "blsp_uart1";
 				bias-disable;
 			};
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
index 6ff87808e2a0f..1fa1aef140a12 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
@@ -181,7 +181,7 @@ intc: interrupt-controller@b000000 {
 			#address-cells = <0>;
 			#interrupt-cells = <3>;
 			reg = <0x0b000000 0x1000>,
-			<0x0b002000 0x1000>;
+			      <0x0b002000 0x1000>;
 		};
 
 		gcc: clock-controller@1800000 {
@@ -268,7 +268,7 @@ blsp1_spi2: spi@78b6000 { /* BLSP1 QUP2 */
 			reg = <0x78b6000 0x600>;
 			interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&gcc GCC_BLSP1_QUP2_SPI_APPS_CLK>,
-				<&gcc GCC_BLSP1_AHB_CLK>;
+				 <&gcc GCC_BLSP1_AHB_CLK>;
 			clock-names = "core", "iface";
 			#address-cells = <1>;
 			#size-cells = <0>;
@@ -380,7 +380,7 @@ blsp1_uart1: serial@78af000 {
 			interrupts = <GIC_SPI 107 IRQ_TYPE_LEVEL_HIGH>;
 			status = "disabled";
 			clocks = <&gcc GCC_BLSP1_UART1_APPS_CLK>,
-				<&gcc GCC_BLSP1_AHB_CLK>;
+				 <&gcc GCC_BLSP1_AHB_CLK>;
 			clock-names = "core", "iface";
 			dmas = <&blsp_dma 0>, <&blsp_dma 1>;
 			dma-names = "tx", "rx";
@@ -392,7 +392,7 @@ blsp1_uart2: serial@78b0000 {
 			interrupts = <GIC_SPI 108 IRQ_TYPE_LEVEL_HIGH>;
 			status = "disabled";
 			clocks = <&gcc GCC_BLSP1_UART2_APPS_CLK>,
-				<&gcc GCC_BLSP1_AHB_CLK>;
+				 <&gcc GCC_BLSP1_AHB_CLK>;
 			clock-names = "core", "iface";
 			dmas = <&blsp_dma 2>, <&blsp_dma 3>;
 			dma-names = "tx", "rx";
-- 
2.43.0

[PATCH 21/38] arm: dts: qcom: ipq8064: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
index 2b3a3ec4e03d8..24632af28d52e 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
@@ -353,8 +353,8 @@ rpm: rpm@108000 {
 			qcom,ipc = <&l2cc 0x8 2>;
 
 			interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>,
-					<GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>,
-					<GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
+				     <GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-names = "ack", "err", "wakeup";
 
 			clocks = <&gcc RPM_MSG_RAM_H_CLK>;
@@ -438,7 +438,7 @@ spi_pins: spi-state {
 
 			leds_pins: leds-state {
 				pins = "gpio7", "gpio8", "gpio9",
-					"gpio26", "gpio53";
+				       "gpio26", "gpio53";
 				function = "gpio";
 				drive-strength = <2>;
 				bias-pull-down;
@@ -489,9 +489,9 @@ mdio0_pins: mdio0-state {
 
 			rgmii2_pins: rgmii2-state {
 				pins = "gpio27", "gpio28", "gpio29",
-					"gpio30", "gpio31", "gpio32",
-					"gpio51", "gpio52", "gpio59",
-					"gpio60", "gpio61", "gpio62";
+				       "gpio30", "gpio31", "gpio32",
+				       "gpio51", "gpio52", "gpio59",
+				       "gpio60", "gpio61", "gpio62";
 				function = "rgmii2";
 				drive-strength = <8>;
 				bias-disable;
@@ -1257,11 +1257,11 @@ lpass@28100000 {
 			compatible = "qcom,lpass-cpu";
 			status = "disabled";
 			clocks = <&lcc AHBIX_CLK>,
-					<&lcc MI2S_OSR_CLK>,
-					<&lcc MI2S_BIT_CLK>;
+				 <&lcc MI2S_OSR_CLK>,
+				 <&lcc MI2S_BIT_CLK>;
 			clock-names = "ahbix-clk",
-					"mi2s-osr-clk",
-					"mi2s-bit-clk";
+				      "mi2s-osr-clk",
+				      "mi2s-bit-clk";
 			interrupts = <GIC_SPI 85 IRQ_TYPE_EDGE_RISING>;
 			interrupt-names = "lpass-irq-lpaif";
 			reg = <0x28100000 0x10000>;
@@ -1280,7 +1280,7 @@ sata: sata@29000000 {
 				 <&gcc SATA_RXOOB_CLK>,
 				 <&gcc SATA_PMALIVE_CLK>;
 			clock-names = "slave_iface", "iface", "core",
-					"rxoob", "pmalive";
+				      "rxoob", "pmalive";
 
 			assigned-clocks = <&gcc SATA_RXOOB_CLK>, <&gcc SATA_PMALIVE_CLK>;
 			assigned-clock-rates = <100000000>, <100000000>;
-- 
2.43.0

[PATCH 20/38] arm: dts: qcom: ipq8064: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi | 1 +
 arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi | 1 +
 arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi      | 2 ++
 3 files changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi
index 49de9752632f6..316dbe55bec8b 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq8064-v1.0.dtsi
@@ -26,6 +26,7 @@ button-1 {
 			linux,input-type = <1>;
 			debounce-interval = <60>;
 		};
+
 		button-2 {
 			label = "wps";
 			linux,code = <KEY_WPS_BUTTON>;
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi
index 2f117d576daf3..deece92f610f3 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq8064-v2.0.dtsi
@@ -32,6 +32,7 @@ &gsbi4 {
 	serial@16340000 {
 		status = "okay";
 	};
+
 	/*
 	 * The i2c device on gsbi4 should not be enabled.
 	 * On ipq806x designs gsbi4 i2c is meant for exclusive
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
index 1a278503d3401..2b3a3ec4e03d8 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
@@ -380,9 +380,11 @@ qfprom: efuse@700000 {
 			speedbin_efuse: speedbin@c0 {
 				reg = <0xc0 0x4>;
 			};
+
 			tsens_calib: calib@400 {
 				reg = <0x400 0xb>;
 			};
+
 			tsens_calib_backup: calib-backup@410 {
 				reg = <0x410 0xb>;
 			};
-- 
2.43.0

[PATCH 22/38] arm: dts: qcom: ipq8064: Break long property array data

[Waqar Hameed]

There is an awkward line break at the last elements expression. Put all
elements on a newline instead. Found/fixed with dtfmt (Devicetree Source
formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
index 24632af28d52e..d2b576a37c47d 100644
--- a/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-ipq8064.dtsi
@@ -278,8 +278,9 @@ memory {
 
 	cpu-pmu {
 		compatible = "qcom,krait-pmu";
-		interrupts = <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(2) |
-					  IRQ_TYPE_LEVEL_HIGH)>;
+		interrupts = <
+			GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(2) | IRQ_TYPE_LEVEL_HIGH)
+		>;
 	};
 
 	reserved-memory {
-- 
2.43.0

[PATCH 23/38] arm: dts: qcom: msm8226: Add space between OR-operator

[Waqar Hameed]

There should be a space between binary operators. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 .../boot/dts/qcom/qcom-msm8226-samsung-matisse-common.dtsi    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8226-samsung-matisse-common.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8226-samsung-matisse-common.dtsi
index f1544a7e8369c..fc4d7ad8d2ab5 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8226-samsung-matisse-common.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8226-samsung-matisse-common.dtsi
@@ -82,8 +82,8 @@ key-volume-up {
 
 	i2c-backlight {
 		compatible = "i2c-gpio";
-		sda-gpios = <&tlmm 20 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
-		scl-gpios = <&tlmm 21 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
+		sda-gpios = <&tlmm 20 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+		scl-gpios = <&tlmm 21 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
 
 		pinctrl-0 = <&backlight_i2c_default_state>;
 		pinctrl-names = "default";
-- 
2.43.0

[PATCH 24/38] arm: dts: qcom: msm8226: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8226.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
index bcf14a3b13a1b..14605d7666e68 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
@@ -183,18 +183,23 @@ rpmpd_opp_table: opp-table {
 						rpmpd_opp_ret: opp1 {
 							opp-level = <1>;
 						};
+
 						rpmpd_opp_svs_krait: opp2 {
 							opp-level = <2>;
 						};
+
 						rpmpd_opp_svs_soc: opp3 {
 							opp-level = <3>;
 						};
+
 						rpmpd_opp_nom: opp4 {
 							opp-level = <4>;
 						};
+
 						rpmpd_opp_turbo: opp5 {
 							opp-level = <5>;
 						};
+
 						rpmpd_opp_super_turbo: opp6 {
 							opp-level = <6>;
 						};
-- 
2.43.0

[PATCH 25/38] arm: dts: qcom: ipq8064: Break long property array data

[Waqar Hameed]

There is an awkward line break at the last elements expression. Put all
elements on a newline instead. Found/fixed with dtfmt (Devicetree Source
formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8226.dtsi | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
index 14605d7666e68..346aef4a1d9f5 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
@@ -137,8 +137,9 @@ opp-787200000 {
 
 	pmu {
 		compatible = "arm,cortex-a7-pmu";
-		interrupts = <GIC_PPI 7 (GIC_CPU_MASK_SIMPLE(4) |
-					 IRQ_TYPE_LEVEL_HIGH)>;
+		interrupts = <
+			GIC_PPI 7 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)
+		>;
 	};
 
 	rpm: remoteproc {
-- 
2.43.0

[PATCH 26/38] arm: dts: qcom: msm8226: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8226.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
index 346aef4a1d9f5..ac18744b50924 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8226.dtsi
@@ -1483,12 +1483,12 @@ cpu_crit1: trip1 {
 	timer {
 		compatible = "arm,armv7-timer";
 		interrupts = <GIC_PPI 2
-				(GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
+			      (GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
 			     <GIC_PPI 3
-				(GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
+			      (GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
 			     <GIC_PPI 4
-				(GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
+			      (GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>,
 			     <GIC_PPI 1
-				(GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>;
+			      (GIC_CPU_MASK_RAW(15) | IRQ_TYPE_LEVEL_LOW)>;
 	};
 };
-- 
2.43.0

[PATCH 28/38] arm: dts: qcom: msm8660: Remove space inside array property

[Waqar Hameed]

There shouldn't be spaces at the beginning and end of an property array.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8660.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
index 7440ea621df59..5af3de24b21ef 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
@@ -83,8 +83,8 @@ intc: interrupt-controller@2080000 {
 			compatible = "qcom,msm-8660-qgic";
 			interrupt-controller;
 			#interrupt-cells = <3>;
-			reg = < 0x02080000 0x1000 >,
-			      < 0x02081000 0x1000 >;
+			reg = <0x02080000 0x1000>,
+			      <0x02081000 0x1000>;
 		};
 
 		timer@2000000 {
-- 
2.43.0

[PATCH 27/38] arm: dts: qcom: msm8660: Remove empty line at end of node

[Waqar Hameed]

There shouldn't be an empty line at end of node definition. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8660.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
index 3f69b98d0041e..7440ea621df59 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8660.dtsi
@@ -107,7 +107,6 @@ tlmm: pinctrl@800000 {
 			interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-controller;
 			#interrupt-cells = <2>;
-
 		};
 
 		gcc: clock-controller@900000 {
@@ -430,5 +429,4 @@ tcsr: syscon@1a400000 {
 			reg = <0x1a400000 0x100>;
 		};
 	};
-
 };
-- 
2.43.0

[PATCH 29/38] arm: dts: qcom: msm8660: Align closing array bracket

[Waqar Hameed]

Closing angle bracket of an array should be indented like property name.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts b/arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts
index 69fe651f564d2..c58d5ffdc678c 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts
+++ b/arch/arm/boot/dts/qcom/qcom-msm8660-surf.dts
@@ -60,7 +60,7 @@ MATRIX_KEY(5, 1, KEY_RIGHT)
 		MATRIX_KEY(5, 2, KEY_DOWN)
 		MATRIX_KEY(5, 3, KEY_BACK)
 		MATRIX_KEY(5, 4, KEY_MENU)
-		>;
+	>;
 	keypad,num-rows = <6>;
 	keypad,num-columns = <5>;
 };
-- 
2.43.0

[PATCH 30/38] arm: dts: qcom: msm8916: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
index 94b7694eeeffa..f286679a6af07 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
@@ -5,12 +5,15 @@ cpus {
 		cpu@0 {
 			enable-method = "qcom,msm8916-smp";
 		};
+
 		cpu@1 {
 			enable-method = "qcom,msm8916-smp";
 		};
+
 		cpu@2 {
 			enable-method = "qcom,msm8916-smp";
 		};
+
 		cpu@3 {
 			enable-method = "qcom,msm8916-smp";
 		};
-- 
2.43.0

[PATCH 31/38] arm: dts: qcom: msm8960: Properly indent subnodes

[Waqar Hameed]

Subnodes should be indented with tabs only. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts b/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
index 1df078d7d89b8..bd0665727277a 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
+++ b/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
@@ -58,34 +58,34 @@ &gsbi5_serial {
 
 &tlmm {
 	spi1_default: spi1-default-state {
-		 mosi-pins {
+		mosi-pins {
 			pins = "gpio6";
 			function = "gsbi1";
 			drive-strength = <12>;
 			bias-disable;
-		 };
+		};
 
-		 miso-pins {
+		miso-pins {
 			pins = "gpio7";
 			function = "gsbi1";
 			drive-strength = <12>;
 			bias-disable;
-		 };
+		};
 
-		 cs-pins {
+		cs-pins {
 			pins = "gpio8";
 			function = "gsbi1";
 			drive-strength = <12>;
 			bias-disable;
 			output-low;
-		 };
+		};
 
-		 clk-pins {
+		clk-pins {
 			pins = "gpio9";
 			function = "gsbi1";
 			drive-strength = <12>;
 			bias-disable;
-		 };
+		};
 	};
 };
 
-- 
2.43.0

[PATCH 32/38] arm: dts: qcom: msm8660: Align closing array bracket

[Waqar Hameed]

Closing angle bracket of an array should be indented like property name.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts b/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
index bd0665727277a..174af88fd7a5d 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
+++ b/arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts
@@ -99,7 +99,7 @@ MATRIX_KEY(0, 0, KEY_VOLUMEUP)
 		MATRIX_KEY(0, 1, KEY_VOLUMEDOWN)
 		MATRIX_KEY(0, 2, KEY_CAMERA_FOCUS)
 		MATRIX_KEY(0, 3, KEY_CAMERA)
-		>;
+	>;
 	keypad,num-rows = <1>;
 	keypad,num-columns = <5>;
 
-- 
2.43.0

[PATCH 34/38] arm: dts: qcom: msm8974: Align closing brace properly

[Waqar Hameed]

Closing brace of a node definition should be indented like subnode name.
Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
index ccf66f45ea7c3..4c2730b34cd1d 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
@@ -867,7 +867,7 @@ port {
 					tpiu_in: endpoint {
 						remote-endpoint = <&replicator_out1>;
 					};
-				 };
+				};
 			};
 		};
 
-- 
2.43.0

[PATCH 33/38] arm: dts: qcom: msm8974: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
index f300de74c47d4..ccf66f45ea7c3 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
@@ -92,7 +92,7 @@ l2: l2-cache {
 		idle-states {
 			cpu_spc: cpu-spc {
 				compatible = "qcom,idle-state-spc",
-						"arm,idle-state";
+					     "arm,idle-state";
 				entry-latency-us = <150>;
 				exit-latency-us = <200>;
 				min-residency-us = <2000>;
@@ -2228,10 +2228,10 @@ remoteproc_adsp: remoteproc@fe200000 {
 			reg = <0xfe200000 0x100>;
 
 			interrupts-extended = <&intc GIC_SPI 162 IRQ_TYPE_EDGE_RISING>,
-					       <&adsp_smp2p_in 0 IRQ_TYPE_EDGE_RISING>,
-					       <&adsp_smp2p_in 1 IRQ_TYPE_EDGE_RISING>,
-					       <&adsp_smp2p_in 2 IRQ_TYPE_EDGE_RISING>,
-					       <&adsp_smp2p_in 3 IRQ_TYPE_EDGE_RISING>;
+					      <&adsp_smp2p_in 0 IRQ_TYPE_EDGE_RISING>,
+					      <&adsp_smp2p_in 1 IRQ_TYPE_EDGE_RISING>,
+					      <&adsp_smp2p_in 2 IRQ_TYPE_EDGE_RISING>,
+					      <&adsp_smp2p_in 3 IRQ_TYPE_EDGE_RISING>;
 			interrupt-names = "wdog", "fatal", "ready", "handover", "stop-ack";
 
 			clocks = <&xo_board>;
-- 
2.43.0

[PATCH 35/38] arm: dts: qcom: msm8974: Add newline between subnodes

[Waqar Hameed]

There should be a newline between subnodes. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
index 4c2730b34cd1d..1a2f0cc9f8a03 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
@@ -960,6 +960,7 @@ replicator_out0: endpoint {
 						remote-endpoint = <&etr_in>;
 					};
 				};
+
 				port@1 {
 					reg = <1>;
 					replicator_out1: endpoint {
@@ -1083,18 +1084,21 @@ kpss_in0: endpoint {
 						remote-endpoint = <&etm0_out>;
 					};
 				};
+
 				port@1 {
 					reg = <1>;
 					kpss_in1: endpoint {
 						remote-endpoint = <&etm1_out>;
 					};
 				};
+
 				port@2 {
 					reg = <2>;
 					kpss_in2: endpoint {
 						remote-endpoint = <&etm2_out>;
 					};
 				};
+
 				port@3 {
 					reg = <3>;
 					kpss_in3: endpoint {
@@ -1785,6 +1789,7 @@ blsp1_i2c6_sleep: blsp1-i2c6-sleep-state {
 				drive-strength = <2>;
 				bias-pull-up;
 			};
+
 			/* 6 interfaces per QUP, BLSP2 indexes are numbered (n)+6 */
 
 			/* BLSP2_I2C1 info is missing */
@@ -1872,14 +1877,17 @@ mosi-pins {
 					pins = "gpio45";
 					function = "blsp_spi8";
 				};
+
 				miso-pins {
 					pins = "gpio46";
 					function = "blsp_spi8";
 				};
+
 				cs-pins {
 					pins = "gpio47";
 					function = "blsp_spi8";
 				};
+
 				clk-pins {
 					pins = "gpio48";
 					function = "blsp_spi8";
@@ -2280,6 +2288,7 @@ cpu_alert0: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit0: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -2300,6 +2309,7 @@ cpu_alert1: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit1: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -2320,6 +2330,7 @@ cpu_alert2: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit2: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
@@ -2340,6 +2351,7 @@ cpu_alert3: trip0 {
 					hysteresis = <2000>;
 					type = "passive";
 				};
+
 				cpu_crit3: trip1 {
 					temperature = <110000>;
 					hysteresis = <2000>;
-- 
2.43.0

[PATCH 36/38] arm: dts: qcom: sdx55: Properly indent subnode

[Waqar Hameed]

Subnodes should be indented with tabs only. Found/fixed with
dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts b/arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts
index 0443a86899a9c..cd57292e3bfd3 100644
--- a/arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts
+++ b/arch/arm/boot/dts/qcom/qcom-sdx55-t55.dts
@@ -297,11 +297,11 @@ perst-pins {
 		};
 
 		wake-pins {
-		       pins = "gpio53";
-		       function = "gpio";
-		       drive-strength = <2>;
-		       bias-pull-up;
-	       };
+			pins = "gpio53";
+			function = "gpio";
+			drive-strength = <2>;
+			bias-pull-up;
+		};
 	};
 };
 
-- 
2.43.0

[PATCH 37/38] arm: dts: qcom: sdx65: Remove empty line at end of node

[Waqar Hameed]

There shouldn't be an empty line at end of node definition. Found/fixed
with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-sdx65.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
index c8e312dcd26b0..c1c5b574424dd 100644
--- a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
@@ -261,7 +261,6 @@ usb_qmpphy: phy@ff6000 {
 				      "phy_phy";
 
 			status = "disabled";
-
 		};
 
 		system_noc: interconnect@1620000 {
@@ -807,7 +806,6 @@ rpmhpd_opp_turbo_l1: opp10 {
 			apps_bcm_voter: bcm-voter {
 				compatible = "qcom,bcm-voter";
 			};
-
 		};
 	};
 
-- 
2.43.0

[PATCH 38/38] arm: dts: qcom: sdx65: Align multiple property data

[Waqar Hameed]

Multiple property data on a newline should be aligned with the first
one. Found/fixed with dtfmt (Devicetree Source formatter).

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
---
 arch/arm/boot/dts/qcom/qcom-sdx65.dtsi | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
index c1c5b574424dd..9814f20e6e167 100644
--- a/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-sdx65.dtsi
@@ -544,10 +544,10 @@ restart@c264000 {
 		spmi_bus: spmi@c440000 {
 			compatible = "qcom,spmi-pmic-arb";
 			reg = <0xc440000 0xd00>,
-				<0xc600000 0x2000000>,
-				<0xe600000 0x100000>,
-				<0xe700000 0xa0000>,
-				<0xc40a000 0x26000>;
+			      <0xc600000 0x2000000>,
+			      <0xe600000 0x100000>,
+			      <0xe700000 0xa0000>,
+			      <0xc40a000 0x26000>;
 			reg-names = "core", "chnls", "obsrvr", "intr", "cnfg";
 			interrupts-extended = <&pdc 1 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-names = "periph_irq";
@@ -735,10 +735,10 @@ apps_rsc: rsc@17830000 {
 			label = "apps_rsc";
 			compatible = "qcom,rpmh-rsc";
 			reg = <0x17830000 0x10000>,
-			    <0x17840000 0x10000>;
+			      <0x17840000 0x10000>;
 			reg-names = "drv-0", "drv-1";
 			interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>,
-				   <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
+				     <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
 			qcom,tcs-offset = <0xd00>;
 			qcom,drv-id = <1>;
 			qcom,tcs-config = <ACTIVE_TCS  2>,
-- 
2.43.0