* [PATCH v2] arm64: boot: dts: qcom: sm8150: add SPI nodes
@ 2021-04-15 10:41 Felipe Balbi
2021-04-15 18:58 ` Konrad Dybcio
0 siblings, 1 reply; 3+ messages in thread
From: Felipe Balbi @ 2021-04-15 10:41 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson
Cc: Rob Herring, linux-arm-msm, devicetree, Caleb Connolly, Felipe Balbi
From: Felipe Balbi <felipe.balbi@microsoft.com>
Add missing SPI nodes for SM8150.
Signed-off-by: Felipe Balbi <felipe.balbi@microsoft.com>
---
Changes since v1:
- avoid modifying drive strength on i2c. No functional
changes, just keeping 0x02 instead of 2
Tested on Microsoft Surface Duo (DTS will be sent after -rc1). This
patch also depends on i2c nodes patch by Caleb Connolly
arch/arm64/boot/dts/qcom/sm8150.dtsi | 597 +++++++++++++++++++++++++++
1 file changed, 597 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
index 7207a3689d9d..ee43c4337f68 100644
--- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
@@ -602,6 +602,21 @@ i2c0: i2c@880000 {
status = "disabled";
};
+ spi0: spi@880000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x880000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S0_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi0_default>;
+ interrupts = <GIC_SPI 601 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c1: i2c@884000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00884000 0 0x4000>;
@@ -615,6 +630,21 @@ i2c1: i2c@884000 {
status = "disabled";
};
+ spi1: spi@884000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x884000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S1_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi1_default>;
+ interrupts = <GIC_SPI 602 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c2: i2c@888000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00888000 0 0x4000>;
@@ -628,6 +658,21 @@ i2c2: i2c@888000 {
status = "disabled";
};
+ spi2: spi@888000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x888000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S2_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi2_default>;
+ interrupts = <GIC_SPI 603 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c3: i2c@88c000 {
compatible = "qcom,geni-i2c";
reg = <0 0x0088c000 0 0x4000>;
@@ -641,6 +686,21 @@ i2c3: i2c@88c000 {
status = "disabled";
};
+ spi3: spi@88c000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x88c000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S3_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi3_default>;
+ interrupts = <GIC_SPI 604 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c4: i2c@890000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00890000 0 0x4000>;
@@ -654,6 +714,21 @@ i2c4: i2c@890000 {
status = "disabled";
};
+ spi4: spi@890000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x890000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S4_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi4_default>;
+ interrupts = <GIC_SPI 605 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c5: i2c@894000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00894000 0 0x4000>;
@@ -667,6 +742,21 @@ i2c5: i2c@894000 {
status = "disabled";
};
+ spi5: spi@894000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x894000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S5_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi5_default>;
+ interrupts = <GIC_SPI 606 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c6: i2c@898000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00898000 0 0x4000>;
@@ -680,6 +770,21 @@ i2c6: i2c@898000 {
status = "disabled";
};
+ spi6: spi@898000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x898000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S6_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi6_default>;
+ interrupts = <GIC_SPI 607 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c7: i2c@89c000 {
compatible = "qcom,geni-i2c";
reg = <0 0x0089c000 0 0x4000>;
@@ -693,6 +798,20 @@ i2c7: i2c@89c000 {
status = "disabled";
};
+ spi7: spi@89c000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0x89c000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP0_S7_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi7_default>;
+ interrupts = <GIC_SPI 608 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
};
qupv3_id_1: geniqup@ac0000 {
@@ -720,6 +839,21 @@ i2c8: i2c@a80000 {
status = "disabled";
};
+ spi8: spi@a80000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa80000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S0_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi8_default>;
+ interrupts = <GIC_SPI 353 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c9: i2c@a84000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00a84000 0 0x4000>;
@@ -733,6 +867,21 @@ i2c9: i2c@a84000 {
status = "disabled";
};
+ spi9: spi@a84000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa84000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S1_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi9_default>;
+ interrupts = <GIC_SPI 354 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c10: i2c@a88000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00a88000 0 0x4000>;
@@ -746,6 +895,21 @@ i2c10: i2c@a88000 {
status = "disabled";
};
+ spi10: spi@a88000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa88000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S2_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi10_default>;
+ interrupts = <GIC_SPI 355 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c11: i2c@a8c000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00a8c000 0 0x4000>;
@@ -759,6 +923,21 @@ i2c11: i2c@a8c000 {
status = "disabled";
};
+ spi11: spi@a8c000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa8c000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S3_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi11_default>;
+ interrupts = <GIC_SPI 356 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
uart2: serial@a90000 {
compatible = "qcom,geni-debug-uart";
reg = <0x0 0x00a90000 0x0 0x4000>;
@@ -781,6 +960,21 @@ i2c12: i2c@a90000 {
status = "disabled";
};
+ spi12: spi@a90000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa90000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S4_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi12_default>;
+ interrupts = <GIC_SPI 357 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c16: i2c@94000 {
compatible = "qcom,geni-i2c";
reg = <0 0x0094000 0 0x4000>;
@@ -793,6 +987,21 @@ i2c16: i2c@94000 {
#size-cells = <0>;
status = "disabled";
};
+
+ spi16: spi@a94000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xa94000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP1_S5_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi16_default>;
+ interrupts = <GIC_SPI 358 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
};
qupv3_id_2: geniqup@cc0000 {
@@ -821,6 +1030,21 @@ i2c17: i2c@c80000 {
status = "disabled";
};
+ spi17: spi@c80000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc80000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi17_default>;
+ interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c18: i2c@c84000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00c84000 0 0x4000>;
@@ -834,6 +1058,21 @@ i2c18: i2c@c84000 {
status = "disabled";
};
+ spi18: spi@c84000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc84000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S1_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi18_default>;
+ interrupts = <GIC_SPI 583 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c19: i2c@c88000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00c88000 0 0x4000>;
@@ -847,6 +1086,21 @@ i2c19: i2c@c88000 {
status = "disabled";
};
+ spi19: spi@c88000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc88000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S2_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi19_default>;
+ interrupts = <GIC_SPI 584 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c13: i2c@c8c000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00c8c000 0 0x4000>;
@@ -860,6 +1114,21 @@ i2c13: i2c@c8c000 {
status = "disabled";
};
+ spi13: spi@c8c000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc8c000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S3_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi13_default>;
+ interrupts = <GIC_SPI 585 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c14: i2c@c90000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00c90000 0 0x4000>;
@@ -873,6 +1142,21 @@ i2c14: i2c@c90000 {
status = "disabled";
};
+ spi14: spi@c90000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc90000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S4_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi14_default>;
+ interrupts = <GIC_SPI 586 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
+
i2c15: i2c@c94000 {
compatible = "qcom,geni-i2c";
reg = <0 0x00c94000 0 0x4000>;
@@ -885,6 +1169,21 @@ i2c15: i2c@c94000 {
#size-cells = <0>;
status = "disabled";
};
+
+ spi15: spi@c94000 {
+ compatible = "qcom,geni-spi";
+ reg = <0 0xc94000 0 0x4000>;
+ reg-names = "se";
+ clock-names = "se";
+ clocks = <&gcc GCC_QUPV3_WRAP2_S5_CLK>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&qup_spi15_default>;
+ interrupts = <GIC_SPI 587 IRQ_TYPE_LEVEL_HIGH>;
+ spi-max-frequency = <50000000>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "disabled";
+ };
};
config_noc: interconnect@1500000 {
@@ -1222,6 +1521,19 @@ config {
};
};
+ qup_spi0_default: qup-spi0-default {
+ mux {
+ pins = "gpio0", "gpio1", "gpio2", "gpio3";
+ function = "qup0";
+ };
+
+ config {
+ pins = "gpio0", "gpio1", "gpio2", "gpio3";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c1_default: qup-i2c1-default {
mux {
pins = "gpio114", "gpio115";
@@ -1235,6 +1547,21 @@ config {
};
};
+ qup_spi1_default: qup-spi1-default {
+ mux {
+ pins = "gpio114", "gpio115", "gpio116",
+ "gpio117";
+ function = "qup1";
+ };
+
+ config {
+ pins = "gpio114", "gpio115", "gpio116",
+ "gpio117";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c2_default: qup-i2c2-default {
mux {
pins = "gpio126", "gpio127";
@@ -1248,6 +1575,21 @@ config {
};
};
+ qup_spi2_default: qup-spi2-default {
+ mux {
+ pins = "gpio126", "gpio127", "gpio128",
+ "gpio129";
+ function = "qup2";
+ };
+
+ config {
+ pins = "gpio126", "gpio127", "gpio128",
+ "gpio129";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c3_default: qup-i2c3-default {
mux {
pins = "gpio144", "gpio145";
@@ -1261,6 +1603,21 @@ config {
};
};
+ qup_spi3_default: qup-spi3-default {
+ mux {
+ pins = "gpio144", "gpio145", "gpio146",
+ "gpio147";
+ function = "qup3";
+ };
+
+ config {
+ pins = "gpio144", "gpio145", "gpio146",
+ "gpio147";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c4_default: qup-i2c4-default {
mux {
pins = "gpio51", "gpio52";
@@ -1274,6 +1631,21 @@ config {
};
};
+ qup_spi4_default: qup-spi4-default {
+ mux {
+ pins = "gpio51", "gpio52", "gpio53",
+ "gpio54";
+ function = "qup4";
+ };
+
+ config {
+ pins = "gpio51", "gpio52", "gpio53",
+ "gpio54";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c5_default: qup-i2c5-default {
mux {
pins = "gpio121", "gpio122";
@@ -1287,6 +1659,21 @@ config {
};
};
+ qup_spi5_default: qup-spi5-default {
+ mux {
+ pins = "gpio119", "gpio120", "gpio121",
+ "gpio122";
+ function = "qup5";
+ };
+
+ config {
+ pins = "gpio119", "gpio120", "gpio121",
+ "gpio122";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c6_default: qup-i2c6-default {
mux {
pins = "gpio6", "gpio7";
@@ -1300,6 +1687,21 @@ config {
};
};
+ qup_spi6_default: qup-spi6_default {
+ mux {
+ pins = "gpio4", "gpio5", "gpio6",
+ "gpio7";
+ function = "qup6";
+ };
+
+ config {
+ pins = "gpio4", "gpio5", "gpio6",
+ "gpio7";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c7_default: qup-i2c7-default {
mux {
pins = "gpio98", "gpio99";
@@ -1313,6 +1715,21 @@ config {
};
};
+ qup_spi7_default: qup-spi7_default {
+ mux {
+ pins = "gpio98", "gpio99", "gpio100",
+ "gpio101";
+ function = "qup7";
+ };
+
+ config {
+ pins = "gpio98", "gpio99", "gpio100",
+ "gpio101";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c8_default: qup-i2c8-default {
mux {
pins = "gpio88", "gpio89";
@@ -1326,6 +1743,21 @@ config {
};
};
+ qup_spi8_default: qup-spi8-default {
+ mux {
+ pins = "gpio88", "gpio89", "gpio90",
+ "gpio91";
+ function = "qup8";
+ };
+
+ config {
+ pins = "gpio88", "gpio89", "gpio90",
+ "gpio91";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c9_default: qup-i2c9-default {
mux {
pins = "gpio39", "gpio40";
@@ -1339,6 +1771,21 @@ config {
};
};
+ qup_spi9_default: qup-spi9-default {
+ mux {
+ pins = "gpio39", "gpio40", "gpio41",
+ "gpio42";
+ function = "qup9";
+ };
+
+ config {
+ pins = "gpio39", "gpio40", "gpio41",
+ "gpio42";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c10_default: qup-i2c10-default {
mux {
pins = "gpio9", "gpio10";
@@ -1352,6 +1799,21 @@ config {
};
};
+ qup_spi10_default: qup-spi10-default {
+ mux {
+ pins = "gpio9", "gpio10", "gpio11",
+ "gpio12";
+ function = "qup10";
+ };
+
+ config {
+ pins = "gpio9", "gpio10", "gpio11",
+ "gpio12";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c11_default: qup-i2c11-default {
mux {
pins = "gpio94", "gpio95";
@@ -1365,6 +1827,21 @@ config {
};
};
+ qup_spi11_default: qup-spi11-default {
+ mux {
+ pins = "gpio92", "gpio93", "gpio94",
+ "gpio95";
+ function = "qup11";
+ };
+
+ config {
+ pins = "gpio92", "gpio93", "gpio94",
+ "gpio95";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c12_default: qup-i2c12-default {
mux {
pins = "gpio83", "gpio84";
@@ -1378,6 +1855,21 @@ config {
};
};
+ qup_spi12_default: qup-spi12-default {
+ mux {
+ pins = "gpio83", "gpio84", "gpio85",
+ "gpio86";
+ function = "qup12";
+ };
+
+ config {
+ pins = "gpio83", "gpio84", "gpio85",
+ "gpio86";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c13_default: qup-i2c13-default {
mux {
pins = "gpio43", "gpio44";
@@ -1391,6 +1883,21 @@ config {
};
};
+ qup_spi13_default: qup-spi13-default {
+ mux {
+ pins = "gpio43", "gpio44", "gpio45",
+ "gpio46";
+ function = "qup13";
+ };
+
+ config {
+ pins = "gpio43", "gpio44", "gpio45",
+ "gpio46";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c14_default: qup-i2c14-default {
mux {
pins = "gpio47", "gpio48";
@@ -1404,6 +1911,21 @@ config {
};
};
+ qup_spi14_default: qup-spi14-default {
+ mux {
+ pins = "gpio47", "gpio48", "gpio49",
+ "gpio50";
+ function = "qup14";
+ };
+
+ config {
+ pins = "gpio47", "gpio48", "gpio49",
+ "gpio50";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c15_default: qup-i2c15-default {
mux {
pins = "gpio27", "gpio28";
@@ -1417,6 +1939,21 @@ config {
};
};
+ qup_spi15_default: qup-spi15-default {
+ mux {
+ pins = "gpio27", "gpio28", "gpio29",
+ "gpio30";
+ function = "qup15";
+ };
+
+ config {
+ pins = "gpio27", "gpio28", "gpio29",
+ "gpio30";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c16_default: qup-i2c16-default {
mux {
pins = "gpio86", "gpio85";
@@ -1430,6 +1967,21 @@ config {
};
};
+ qup_spi16_default: qup-spi16-default {
+ mux {
+ pins = "gpio83", "gpio84", "gpio85",
+ "gpio86";
+ function = "qup16";
+ };
+
+ config {
+ pins = "gpio83", "gpio84", "gpio85",
+ "gpio86";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c17_default: qup-i2c17-default {
mux {
pins = "gpio55", "gpio56";
@@ -1443,6 +1995,21 @@ config {
};
};
+ qup_spi17_default: qup-spi17-default {
+ mux {
+ pins = "gpio55", "gpio56", "gpio57",
+ "gpio58";
+ function = "qup17";
+ };
+
+ config {
+ pins = "gpio55", "gpio56", "gpio57",
+ "gpio58";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c18_default: qup-i2c18-default {
mux {
pins = "gpio23", "gpio24";
@@ -1456,6 +2023,21 @@ config {
};
};
+ qup_spi18_default: qup-spi18-default {
+ mux {
+ pins = "gpio23", "gpio24", "gpio25",
+ "gpio26";
+ function = "qup18";
+ };
+
+ config {
+ pins = "gpio23", "gpio24", "gpio25",
+ "gpio26";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
+
qup_i2c19_default: qup-i2c19-default {
mux {
pins = "gpio57", "gpio58";
@@ -1468,6 +2050,21 @@ config {
bias-disable;
};
};
+
+ qup_spi19_default: qup-spi19-default {
+ mux {
+ pins = "gpio55", "gpio56", "gpio57",
+ "gpio58";
+ function = "qup19";
+ };
+
+ config {
+ pins = "gpio55", "gpio56", "gpio57",
+ "gpio58";
+ drive-strength = <6>;
+ bias-disable;
+ };
+ };
};
remoteproc_mpss: remoteproc@4080000 {
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] arm64: boot: dts: qcom: sm8150: add SPI nodes
2021-04-15 10:41 [PATCH v2] arm64: boot: dts: qcom: sm8150: add SPI nodes Felipe Balbi
@ 2021-04-15 18:58 ` Konrad Dybcio
2021-04-16 10:31 ` Felipe Balbi
0 siblings, 1 reply; 3+ messages in thread
From: Konrad Dybcio @ 2021-04-15 18:58 UTC (permalink / raw)
To: Felipe Balbi, Andy Gross, Bjorn Andersson
Cc: Rob Herring, linux-arm-msm, devicetree, Caleb Connolly, Felipe Balbi
Hi!
> Tested on Microsoft Surface Duo (DTS will be sent after -rc1).
Cool, along with the Lumias we'll now have all the MSFT arm64 qcom devices supported :)
>
> + qup_spi0_default: qup-spi0-default {
> + mux {
> + pins = "gpio0", "gpio1", "gpio2", "gpio3";
> + function = "qup0";
> + };
> +
> + config {
> + pins = "gpio0", "gpio1", "gpio2", "gpio3";
> + drive-strength = <6>;
> + bias-disable;
> + };
> + };
> +
This format - while technically still correct - is now considered obsolete. Please move the pin definitions into a more concise one, like so:
qup_spi0_default: qup-spi0-default {
pins = "gpio0", "gpio1", "gpio2", "gpio3";
function = "qup0";
drive-strength = <6>;
bias-disable;
};
(hopefully Thunderbird didn't mess up newlines here..)
Overall, the patch - while on the large side - looks correct to me. Thanks!
Konrad
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] arm64: boot: dts: qcom: sm8150: add SPI nodes
2021-04-15 18:58 ` Konrad Dybcio
@ 2021-04-16 10:31 ` Felipe Balbi
0 siblings, 0 replies; 3+ messages in thread
From: Felipe Balbi @ 2021-04-16 10:31 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson
Cc: Rob Herring, linux-arm-msm, devicetree, Caleb Connolly, Felipe Balbi
[-- Attachment #1: Type: text/plain, Size: 1135 bytes --]
Hi,
Konrad Dybcio <konrad.dybcio@somainline.org> writes:
> Hi!
>
>
>> Tested on Microsoft Surface Duo (DTS will be sent after -rc1).
>
> Cool, along with the Lumias we'll now have all the MSFT arm64 qcom devices supported :)
:)
>> + qup_spi0_default: qup-spi0-default {
>> + mux {
>> + pins = "gpio0", "gpio1", "gpio2", "gpio3";
>> + function = "qup0";
>> + };
>> +
>> + config {
>> + pins = "gpio0", "gpio1", "gpio2", "gpio3";
>> + drive-strength = <6>;
>> + bias-disable;
>> + };
>> + };
>> +
>
> This format - while technically still correct - is now considered
> obsolete. Please move the pin definitions into a more concise one,
> like so:
>
>
> qup_spi0_default: qup-spi0-default {
> pins = "gpio0", "gpio1", "gpio2", "gpio3";
> function = "qup0";
> drive-strength = <6>;
> bias-disable;
> };
aaa, pretty cool. I'll send v3 shortly.
> (hopefully Thunderbird didn't mess up newlines here..)
heh, it did :)
> Overall, the patch - while on the large side - looks correct to
> me. Thanks!
thanks
--
balbi
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 857 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-04-16 10:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-15 10:41 [PATCH v2] arm64: boot: dts: qcom: sm8150: add SPI nodes Felipe Balbi
2021-04-15 18:58 ` Konrad Dybcio
2021-04-16 10:31 ` Felipe Balbi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).