| Message ID | 20221212182314.1902632-5-bmasney@redhat.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | arm64: dts: qcom: sc8280xp: add i2c and spi nodes | expand |
On 12/12/2022 11:53 PM, Brian Masney wrote: > Add the missing nodes for the spi buses that's present on this SoC. > > This work was derived from various patches that Qualcomm delivered > to Red Hat in a downstream kernel. > > Signed-off-by: Brian Masney <bmasney@redhat.com> > --- > arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 384 +++++++++++++++++++++++++ > 1 file changed, 384 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > index 392a1509f0be..b50db09feae2 100644 > --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > @@ -829,6 +829,22 @@ qup2_i2c16: i2c@880000 { > status = "disabled"; > }; > > + qup2_spi16: spi@880000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00880000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; This is device property not host and same applicable for all below spi nodes. Also FYI let's enable below SPI for Qdrive usecases once spidev compatible name is confirmed. SE9 0x00A84000 SE22 0x00898000 -Shazad > + status = "disabled"; > + }; > + > qup2_i2c17: i2c@884000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00884000 0 0x4000>; > @@ -845,6 +861,22 @@ qup2_i2c17: i2c@884000 { > status = "disabled"; > }; > > + qup2_spi17: spi@884000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00884000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S1_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 583 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_uart17: serial@884000 { > compatible = "qcom,geni-uart"; > reg = <0 0x00884000 0 0x4000>; > @@ -875,6 +907,22 @@ qup2_i2c18: i2c@888000 { > status = "disabled"; > }; > > + qup2_spi18: spi@888000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00888000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S2_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 584 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_i2c19: i2c@88c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x0088c000 0 0x4000>; > @@ -891,6 +939,22 @@ qup2_i2c19: i2c@88c000 { > status = "disabled"; > }; > > + qup2_spi19: spi@88c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x0088c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S3_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 585 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_i2c20: i2c@890000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00890000 0 0x4000>; > @@ -907,6 +971,22 @@ qup2_i2c20: i2c@890000 { > status = "disabled"; > }; > > + qup2_spi20: spi@890000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00890000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S4_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 586 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_i2c21: i2c@894000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00894000 0 0x4000>; > @@ -923,6 +1003,22 @@ qup2_i2c21: i2c@894000 { > status = "disabled"; > }; > > + qup2_spi21: spi@894000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00894000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S5_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 587 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_i2c22: i2c@898000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00898000 0 0x4000>; > @@ -939,6 +1035,22 @@ qup2_i2c22: i2c@898000 { > status = "disabled"; > }; > > + qup2_spi22: spi@898000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00898000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S6_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 833 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup2_i2c23: i2c@89c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x0089c000 0 0x4000>; > @@ -954,6 +1066,22 @@ qup2_i2c23: i2c@89c000 { > interconnect-names = "qup-core", "qup-config", "qup-memory"; > status = "disabled"; > }; > + > + qup2_spi23: spi@89c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x0089c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S7_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 834 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > }; > > qup0: geniqup@9c0000 { > @@ -986,6 +1114,22 @@ qup0_i2c0: i2c@980000 { > status = "disabled"; > }; > > + qup0_spi0: spi@980000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00980000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 601 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c1: i2c@984000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00984000 0 0x4000>; > @@ -1002,6 +1146,22 @@ qup0_i2c1: i2c@984000 { > status = "disabled"; > }; > > + qup0_spi1: spi@984000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00984000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S1_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 602 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c2: i2c@988000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00988000 0 0x4000>; > @@ -1018,6 +1178,22 @@ qup0_i2c2: i2c@988000 { > status = "disabled"; > }; > > + qup0_spi2: spi@988000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00988000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S2_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 603 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c3: i2c@98c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x0098c000 0 0x4000>; > @@ -1034,6 +1210,22 @@ qup0_i2c3: i2c@98c000 { > status = "disabled"; > }; > > + qup0_spi3: spi@98c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x0098c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S3_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 604 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c4: i2c@990000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00990000 0 0x4000>; > @@ -1050,6 +1242,22 @@ qup0_i2c4: i2c@990000 { > status = "disabled"; > }; > > + qup0_spi4: spi@990000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00990000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S4_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 605 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c5: i2c@994000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00994000 0 0x4000>; > @@ -1066,6 +1274,22 @@ qup0_i2c5: i2c@994000 { > status = "disabled"; > }; > > + qup0_spi5: spi@994000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00994000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S5_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 606 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c6: i2c@998000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00998000 0 0x4000>; > @@ -1082,6 +1306,22 @@ qup0_i2c6: i2c@998000 { > status = "disabled"; > }; > > + qup0_spi6: spi@998000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00998000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S6_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 607 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup0_i2c7: i2c@99c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x0099c000 0 0x4000>; > @@ -1097,6 +1337,22 @@ qup0_i2c7: i2c@99c000 { > interconnect-names = "qup-core", "qup-config", "qup-memory"; > status = "disabled"; > }; > + > + qup0_spi7: spi@99c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x0099c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP2_S7_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 608 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, > + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > }; > > qup1: geniqup@ac0000 { > @@ -1129,6 +1385,22 @@ qup1_i2c8: i2c@a80000 { > status = "disabled"; > }; > > + qup1_spi8: spi@a80000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a80000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S0_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 353 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c9: i2c@a84000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a84000 0 0x4000>; > @@ -1145,6 +1417,22 @@ qup1_i2c9: i2c@a84000 { > status = "disabled"; > }; > > + qup1_spi9: spi@a84000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a84000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S1_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 354 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c10: i2c@a88000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a88000 0 0x4000>; > @@ -1161,6 +1449,22 @@ qup1_i2c10: i2c@a88000 { > status = "disabled"; > }; > > + qup1_spi10: spi@a88000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a88000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S2_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 355 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c11: i2c@a8c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a8c000 0 0x4000>; > @@ -1177,6 +1481,22 @@ qup1_i2c11: i2c@a8c000 { > status = "disabled"; > }; > > + qup1_spi11: spi@a8c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a8c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S3_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 356 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c12: i2c@a90000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a90000 0 0x4000>; > @@ -1193,6 +1513,22 @@ qup1_i2c12: i2c@a90000 { > status = "disabled"; > }; > > + qup1_spi12: spi@a90000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a90000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S4_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 357 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c13: i2c@a94000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a94000 0 0x4000>; > @@ -1209,6 +1545,22 @@ qup1_i2c13: i2c@a94000 { > status = "disabled"; > }; > > + qup1_spi13: spi@a94000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a94000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S5_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 358 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c14: i2c@a98000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a98000 0 0x4000>; > @@ -1225,6 +1577,22 @@ qup1_i2c14: i2c@a98000 { > status = "disabled"; > }; > > + qup1_spi14: spi@a98000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a98000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S6_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 835 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > + > qup1_i2c15: i2c@a9c000 { > compatible = "qcom,geni-i2c"; > reg = <0 0x00a9c000 0 0x4000>; > @@ -1240,6 +1608,22 @@ qup1_i2c15: i2c@a9c000 { > interconnect-names = "qup-core", "qup-config", "qup-memory"; > status = "disabled"; > }; > + > + qup1_spi15: spi@a9c000 { > + compatible = "qcom,geni-spi"; > + reg = <0 0x00a9c000 0 0x4000>; > + clocks = <&gcc GCC_QUPV3_WRAP1_S7_CLK>; > + clock-names = "se"; > + interrupts = <GIC_SPI 836 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, > + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > + spi-max-frequency = <50000000>; > + status = "disabled"; > + }; > }; > > pcie4: pcie@1c00000 {
+ Mark Brown and linux-spi list On Tue, Dec 13, 2022 at 12:46:18PM +0530, Shazad Hussain wrote: > On 12/12/2022 11:53 PM, Brian Masney wrote: > > Add the missing nodes for the spi buses that's present on this SoC. > > > > This work was derived from various patches that Qualcomm delivered > > to Red Hat in a downstream kernel. > > > > Signed-off-by: Brian Masney <bmasney@redhat.com> > > --- > > arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 384 +++++++++++++++++++++++++ > > 1 file changed, 384 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > > index 392a1509f0be..b50db09feae2 100644 > > --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > > +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi > > @@ -829,6 +829,22 @@ qup2_i2c16: i2c@880000 { > > status = "disabled"; > > }; > > + qup2_spi16: spi@880000 { > > + compatible = "qcom,geni-spi"; > > + reg = <0 0x00880000 0 0x4000>; > > + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; > > + clock-names = "se"; > > + interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, > > + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, > > + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; > > + interconnect-names = "qup-core", "qup-config", "qup-memory"; > > + spi-max-frequency = <50000000>; > > This is device property not host and same applicable for all below spi > nodes. > Also FYI let's enable below SPI for Qdrive usecases once spidev compatible > name is confirmed. > SE9 0x00A84000 > SE22 0x00898000 I talked to Javier Martinez Canillas yesterday about the spidev driver and I think I now have a better understanding of what we need to do. Quick background for Mark. For this automotive program, Qualcomm will be delivering to Red Hat and our customers parts of the media stack in userspace. This will be closed source, proprietary code that parts of it will need to interface with SPI. We can't add a generic qcom,spidev compatible to the spidev driver since this is just a software abstraction. Instead, each type of device will need to have it's own compatible that uniquely describes the type of device. So you might have a compatible like qcom,spi-video-codec. There will need to be a DT binding added that describes the hardware. I suspect that a SPI device can simply be added to trivial-devices.yaml. Once the DT binding is accepted, the compatible can be added to the spidev.c driver. If an in-kernel driver is written in the future, then the compatible can be moved from spidev to the new driver. Mark: Is my understanding above correct? If so, will it be a problem to get a compatible added to spidev.c if the corresponding userspace code is closed source and proprietary? Brian
On Tue, Dec 13, 2022 at 07:27:01AM -0500, Brian Masney wrote: > We can't add a generic qcom,spidev compatible to the spidev driver since > this is just a software abstraction. Instead, each type of device will > need to have it's own compatible that uniquely describes the type of > device. So you might have a compatible like qcom,spi-video-codec. There > will need to be a DT binding added that describes the hardware. I suspect > that a SPI device can simply be added to trivial-devices.yaml. Once the > DT binding is accepted, the compatible can be added to the spidev.c > driver. If an in-kernel driver is written in the future, then the > compatible can be moved from spidev to the new driver. > Mark: Is my understanding above correct? If so, will it be a problem to > get a compatible added to spidev.c if the corresponding userspace code is > closed source and proprietary? No restriction on what the userspace is.
On 13/12/2022 13:27, Brian Masney wrote: > + Mark Brown and linux-spi list > > On Tue, Dec 13, 2022 at 12:46:18PM +0530, Shazad Hussain wrote: >> On 12/12/2022 11:53 PM, Brian Masney wrote: >>> Add the missing nodes for the spi buses that's present on this SoC. >>> >>> This work was derived from various patches that Qualcomm delivered >>> to Red Hat in a downstream kernel. >>> >>> Signed-off-by: Brian Masney <bmasney@redhat.com> >>> --- >>> arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 384 +++++++++++++++++++++++++ >>> 1 file changed, 384 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >>> index 392a1509f0be..b50db09feae2 100644 >>> --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >>> +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >>> @@ -829,6 +829,22 @@ qup2_i2c16: i2c@880000 { >>> status = "disabled"; >>> }; >>> + qup2_spi16: spi@880000 { >>> + compatible = "qcom,geni-spi"; >>> + reg = <0 0x00880000 0 0x4000>; >>> + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; >>> + clock-names = "se"; >>> + interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>; >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, >>> + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, >>> + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; >>> + interconnect-names = "qup-core", "qup-config", "qup-memory"; >>> + spi-max-frequency = <50000000>; >> >> This is device property not host and same applicable for all below spi >> nodes. >> Also FYI let's enable below SPI for Qdrive usecases once spidev compatible >> name is confirmed. >> SE9 0x00A84000 >> SE22 0x00898000 > > I talked to Javier Martinez Canillas yesterday about the spidev driver > and I think I now have a better understanding of what we need to do. > Quick background for Mark. For this automotive program, Qualcomm will be > delivering to Red Hat and our customers parts of the media stack in > userspace. This will be closed source, proprietary code that parts of it > will need to interface with SPI. > > We can't add a generic qcom,spidev compatible to the spidev driver since > this is just a software abstraction. Instead, each type of device will > need to have it's own compatible that uniquely describes the type of > device. So you might have a compatible like qcom,spi-video-codec. There > will need to be a DT binding added that describes the hardware. I suspect > that a SPI device can simply be added to trivial-devices.yaml. Once the > DT binding is accepted, the compatible can be added to the spidev.c > driver. If an in-kernel driver is written in the future, then the > compatible can be moved from spidev to the new driver. > > Mark: Is my understanding above correct? If so, will it be a problem to > get a compatible added to spidev.c if the corresponding userspace code is > closed source and proprietary? qcom,spi-video-codec is still not specific enough. You need to describe real device behind spidev. To be clear - you do not describe userspace, but the device. Best regards, Krzysztof
Hello Krzysztof, Long time no see :) On 12/13/22 14:02, Krzysztof Kozlowski wrote: > On 13/12/2022 13:27, Brian Masney wrote: [...] > > qcom,spi-video-codec is still not specific enough. You need to describe > real device behind spidev. To be clear - you do not describe userspace, > but the device. > Agree. I believe Brian just used "qcom,spi-video-codec" as an example but is only a make up name to illustrate the concept. QC needs to determine what would be the correct <vendor,device> tuple for the IP block that the user-space driver will drive.
On Tue, Dec 13, 2022 at 02:08:47PM +0100, Javier Martinez Canillas wrote: > On 12/13/22 14:02, Krzysztof Kozlowski wrote: > > On 13/12/2022 13:27, Brian Masney wrote: > > qcom,spi-video-codec is still not specific enough. You need to describe > > real device behind spidev. To be clear - you do not describe userspace, > > but the device. > > > > Agree. > > I believe Brian just used "qcom,spi-video-codec" as an example but is only > a make up name to illustrate the concept. QC needs to determine what would > be the correct <vendor,device> tuple for the IP block that the user-space > driver will drive. Yes, that was just an example. Shazad: Is this thread clear about what QC needs for spidev? I'll let QC take care of sending patch(es) to add the various compatibles since I'm not sure what hardware will be backed by spidev. I'll take care of making sure that sc8280xp.dtsi gets the spi controller nodes added. Brian
On 12/13/2022 8:06 PM, Brian Masney wrote: > On Tue, Dec 13, 2022 at 02:08:47PM +0100, Javier Martinez Canillas wrote: >> On 12/13/22 14:02, Krzysztof Kozlowski wrote: >>> On 13/12/2022 13:27, Brian Masney wrote: >>> qcom,spi-video-codec is still not specific enough. You need to describe >>> real device behind spidev. To be clear - you do not describe userspace, >>> but the device. >>> >> >> Agree. >> >> I believe Brian just used "qcom,spi-video-codec" as an example but is only >> a make up name to illustrate the concept. QC needs to determine what would >> be the correct <vendor,device> tuple for the IP block that the user-space >> driver will drive. > > Yes, that was just an example. > > Shazad: Is this thread clear about what QC needs for spidev? I'll let QC > take care of sending patch(es) to add the various compatibles since I'm > not sure what hardware will be backed by spidev. > I think for qup2_spi22 we can use qcom,spi-msm-codec-slave as compatible. As this is what used in downstream. > I'll take care of making sure that sc8280xp.dtsi gets the spi controller > nodes added. Yes, for qup1_spi9 we can add it later when needed. This is for display/touch 2nd. Shazad > > Brian >
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi index 392a1509f0be..b50db09feae2 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi @@ -829,6 +829,22 @@ qup2_i2c16: i2c@880000 { status = "disabled"; }; + qup2_spi16: spi@880000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00880000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c17: i2c@884000 { compatible = "qcom,geni-i2c"; reg = <0 0x00884000 0 0x4000>; @@ -845,6 +861,22 @@ qup2_i2c17: i2c@884000 { status = "disabled"; }; + qup2_spi17: spi@884000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00884000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S1_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 583 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_uart17: serial@884000 { compatible = "qcom,geni-uart"; reg = <0 0x00884000 0 0x4000>; @@ -875,6 +907,22 @@ qup2_i2c18: i2c@888000 { status = "disabled"; }; + qup2_spi18: spi@888000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00888000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S2_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 584 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c19: i2c@88c000 { compatible = "qcom,geni-i2c"; reg = <0 0x0088c000 0 0x4000>; @@ -891,6 +939,22 @@ qup2_i2c19: i2c@88c000 { status = "disabled"; }; + qup2_spi19: spi@88c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x0088c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S3_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 585 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c20: i2c@890000 { compatible = "qcom,geni-i2c"; reg = <0 0x00890000 0 0x4000>; @@ -907,6 +971,22 @@ qup2_i2c20: i2c@890000 { status = "disabled"; }; + qup2_spi20: spi@890000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00890000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S4_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 586 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c21: i2c@894000 { compatible = "qcom,geni-i2c"; reg = <0 0x00894000 0 0x4000>; @@ -923,6 +1003,22 @@ qup2_i2c21: i2c@894000 { status = "disabled"; }; + qup2_spi21: spi@894000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00894000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S5_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 587 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c22: i2c@898000 { compatible = "qcom,geni-i2c"; reg = <0 0x00898000 0 0x4000>; @@ -939,6 +1035,22 @@ qup2_i2c22: i2c@898000 { status = "disabled"; }; + qup2_spi22: spi@898000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00898000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S6_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 833 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup2_i2c23: i2c@89c000 { compatible = "qcom,geni-i2c"; reg = <0 0x0089c000 0 0x4000>; @@ -954,6 +1066,22 @@ qup2_i2c23: i2c@89c000 { interconnect-names = "qup-core", "qup-config", "qup-memory"; status = "disabled"; }; + + qup2_spi23: spi@89c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x0089c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S7_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 834 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_2 0 &clk_virt SLAVE_QUP_CORE_2 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_2 0>, + <&aggre1_noc MASTER_QUP_2 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; }; qup0: geniqup@9c0000 { @@ -986,6 +1114,22 @@ qup0_i2c0: i2c@980000 { status = "disabled"; }; + qup0_spi0: spi@980000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00980000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S0_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 601 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c1: i2c@984000 { compatible = "qcom,geni-i2c"; reg = <0 0x00984000 0 0x4000>; @@ -1002,6 +1146,22 @@ qup0_i2c1: i2c@984000 { status = "disabled"; }; + qup0_spi1: spi@984000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00984000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S1_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 602 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c2: i2c@988000 { compatible = "qcom,geni-i2c"; reg = <0 0x00988000 0 0x4000>; @@ -1018,6 +1178,22 @@ qup0_i2c2: i2c@988000 { status = "disabled"; }; + qup0_spi2: spi@988000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00988000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S2_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 603 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c3: i2c@98c000 { compatible = "qcom,geni-i2c"; reg = <0 0x0098c000 0 0x4000>; @@ -1034,6 +1210,22 @@ qup0_i2c3: i2c@98c000 { status = "disabled"; }; + qup0_spi3: spi@98c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x0098c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S3_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 604 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c4: i2c@990000 { compatible = "qcom,geni-i2c"; reg = <0 0x00990000 0 0x4000>; @@ -1050,6 +1242,22 @@ qup0_i2c4: i2c@990000 { status = "disabled"; }; + qup0_spi4: spi@990000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00990000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S4_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 605 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c5: i2c@994000 { compatible = "qcom,geni-i2c"; reg = <0 0x00994000 0 0x4000>; @@ -1066,6 +1274,22 @@ qup0_i2c5: i2c@994000 { status = "disabled"; }; + qup0_spi5: spi@994000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00994000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S5_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 606 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c6: i2c@998000 { compatible = "qcom,geni-i2c"; reg = <0 0x00998000 0 0x4000>; @@ -1082,6 +1306,22 @@ qup0_i2c6: i2c@998000 { status = "disabled"; }; + qup0_spi6: spi@998000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00998000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S6_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 607 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup0_i2c7: i2c@99c000 { compatible = "qcom,geni-i2c"; reg = <0 0x0099c000 0 0x4000>; @@ -1097,6 +1337,22 @@ qup0_i2c7: i2c@99c000 { interconnect-names = "qup-core", "qup-config", "qup-memory"; status = "disabled"; }; + + qup0_spi7: spi@99c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x0099c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP2_S7_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 608 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_0 0 &clk_virt SLAVE_QUP_CORE_0 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_0 0>, + <&aggre1_noc MASTER_QUP_0 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; }; qup1: geniqup@ac0000 { @@ -1129,6 +1385,22 @@ qup1_i2c8: i2c@a80000 { status = "disabled"; }; + qup1_spi8: spi@a80000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a80000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S0_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 353 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c9: i2c@a84000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a84000 0 0x4000>; @@ -1145,6 +1417,22 @@ qup1_i2c9: i2c@a84000 { status = "disabled"; }; + qup1_spi9: spi@a84000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a84000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S1_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 354 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c10: i2c@a88000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a88000 0 0x4000>; @@ -1161,6 +1449,22 @@ qup1_i2c10: i2c@a88000 { status = "disabled"; }; + qup1_spi10: spi@a88000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a88000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S2_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 355 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c11: i2c@a8c000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a8c000 0 0x4000>; @@ -1177,6 +1481,22 @@ qup1_i2c11: i2c@a8c000 { status = "disabled"; }; + qup1_spi11: spi@a8c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a8c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S3_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 356 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c12: i2c@a90000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a90000 0 0x4000>; @@ -1193,6 +1513,22 @@ qup1_i2c12: i2c@a90000 { status = "disabled"; }; + qup1_spi12: spi@a90000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a90000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S4_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 357 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c13: i2c@a94000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a94000 0 0x4000>; @@ -1209,6 +1545,22 @@ qup1_i2c13: i2c@a94000 { status = "disabled"; }; + qup1_spi13: spi@a94000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a94000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S5_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 358 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c14: i2c@a98000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a98000 0 0x4000>; @@ -1225,6 +1577,22 @@ qup1_i2c14: i2c@a98000 { status = "disabled"; }; + qup1_spi14: spi@a98000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a98000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S6_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 835 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; + qup1_i2c15: i2c@a9c000 { compatible = "qcom,geni-i2c"; reg = <0 0x00a9c000 0 0x4000>; @@ -1240,6 +1608,22 @@ qup1_i2c15: i2c@a9c000 { interconnect-names = "qup-core", "qup-config", "qup-memory"; status = "disabled"; }; + + qup1_spi15: spi@a9c000 { + compatible = "qcom,geni-spi"; + reg = <0 0x00a9c000 0 0x4000>; + clocks = <&gcc GCC_QUPV3_WRAP1_S7_CLK>; + clock-names = "se"; + interrupts = <GIC_SPI 836 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + interconnects = <&clk_virt MASTER_QUP_CORE_1 0 &clk_virt SLAVE_QUP_CORE_1 0>, + <&gem_noc MASTER_APPSS_PROC 0 &config_noc SLAVE_QUP_1 0>, + <&aggre1_noc MASTER_QUP_1 0 &mc_virt SLAVE_EBI1 0>; + interconnect-names = "qup-core", "qup-config", "qup-memory"; + spi-max-frequency = <50000000>; + status = "disabled"; + }; }; pcie4: pcie@1c00000 {
Add the missing nodes for the spi buses that's present on this SoC. This work was derived from various patches that Qualcomm delivered to Red Hat in a downstream kernel. Signed-off-by: Brian Masney <bmasney@redhat.com> --- arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 384 +++++++++++++++++++++++++ 1 file changed, 384 insertions(+)