* [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The first patch of this series adds a compatible for rk3568-spi to the DT bindings. The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes were lifted from the downstream vendor kernel's devicetree, and were double-checked for correctness. The third patch sets up the broken-out SPI pins on the Quartz64 Model A; they use the "m1" set of the pins, not the "m0" set. I assume the "m" stands for "mux". I've tested both patches by connecting an MCP2515 SPI CAN bus controller to the spi pins, which initialised fine. Regards, Nicolas Frattaroli Nicolas Frattaroli (3): dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible arm64: dts: rockchip: Add spi nodes on rk356x arm64: dts: rockchip: Add spi1 pins on Quartz64 A .../devicetree/bindings/spi/spi-rockchip.yaml | 1 + .../boot/dts/rockchip/rk3566-quartz64-a.dts | 5 ++ arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 +++++++++++++++++++ 3 files changed, 74 insertions(+) -- 2.34.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The first patch of this series adds a compatible for rk3568-spi to the DT bindings. The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes were lifted from the downstream vendor kernel's devicetree, and were double-checked for correctness. The third patch sets up the broken-out SPI pins on the Quartz64 Model A; they use the "m1" set of the pins, not the "m0" set. I assume the "m" stands for "mux". I've tested both patches by connecting an MCP2515 SPI CAN bus controller to the spi pins, which initialised fine. Regards, Nicolas Frattaroli Nicolas Frattaroli (3): dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible arm64: dts: rockchip: Add spi nodes on rk356x arm64: dts: rockchip: Add spi1 pins on Quartz64 A .../devicetree/bindings/spi/spi-rockchip.yaml | 1 + .../boot/dts/rockchip/rk3566-quartz64-a.dts | 5 ++ arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 +++++++++++++++++++ 3 files changed, 74 insertions(+) -- 2.34.0 ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The first patch of this series adds a compatible for rk3568-spi to the DT bindings. The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes were lifted from the downstream vendor kernel's devicetree, and were double-checked for correctness. The third patch sets up the broken-out SPI pins on the Quartz64 Model A; they use the "m1" set of the pins, not the "m0" set. I assume the "m" stands for "mux". I've tested both patches by connecting an MCP2515 SPI CAN bus controller to the spi pins, which initialised fine. Regards, Nicolas Frattaroli Nicolas Frattaroli (3): dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible arm64: dts: rockchip: Add spi nodes on rk356x arm64: dts: rockchip: Add spi1 pins on Quartz64 A .../devicetree/bindings/spi/spi-rockchip.yaml | 1 + .../boot/dts/rockchip/rk3566-quartz64-a.dts | 5 ++ arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 +++++++++++++++++++ 3 files changed, 74 insertions(+) -- 2.34.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2021-11-26 15:43 ` Nicolas Frattaroli -1 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Mark Brown, Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, linux-spi, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds a compatible string for the SPI controller found on the RK3566 and RK3568 SoCs. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- Documentation/devicetree/bindings/spi/spi-rockchip.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml index 7f987e79337c..52a78a2e362e 100644 --- a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml +++ b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml @@ -33,6 +33,7 @@ properties: - rockchip,rk3328-spi - rockchip,rk3368-spi - rockchip,rk3399-spi + - rockchip,rk3568-spi - rockchip,rv1126-spi - const: rockchip,rk3066-spi -- 2.34.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Mark Brown, Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, linux-spi, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds a compatible string for the SPI controller found on the RK3566 and RK3568 SoCs. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- Documentation/devicetree/bindings/spi/spi-rockchip.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml index 7f987e79337c..52a78a2e362e 100644 --- a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml +++ b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml @@ -33,6 +33,7 @@ properties: - rockchip,rk3328-spi - rockchip,rk3368-spi - rockchip,rk3399-spi + - rockchip,rk3568-spi - rockchip,rv1126-spi - const: rockchip,rk3066-spi -- 2.34.0 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Mark Brown, Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, linux-spi, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds a compatible string for the SPI controller found on the RK3566 and RK3568 SoCs. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- Documentation/devicetree/bindings/spi/spi-rockchip.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml index 7f987e79337c..52a78a2e362e 100644 --- a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml +++ b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml @@ -33,6 +33,7 @@ properties: - rockchip,rk3328-spi - rockchip,rk3368-spi - rockchip,rk3399-spi + - rockchip,rk3568-spi - rockchip,rv1126-spi - const: rockchip,rk3066-spi -- 2.34.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2021-11-26 15:43 ` Nicolas Frattaroli -1 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds the four spi nodes (spi0, spi1, spi2, spi3) to the rk356x dtsi. These are from the downstream device tree, though I have double-checked that their interrupts and DMA numbers are correct. I have also tested spi1 with an SPI device. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi index 46d9552f6028..57c0197cc65a 100644 --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi @@ -39,6 +39,10 @@ aliases { serial7 = &uart7; serial8 = &uart8; serial9 = &uart9; + spi0 = &spi0; + spi1 = &spi1; + spi2 = &spi2; + spi3 = &spi3; }; cpus { @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { clock-names = "tclk", "pclk"; }; + spi0: spi@fe610000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe610000 0x0 0x1000>; + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 20>, <&dmac0 21>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; + status = "disabled"; + }; + + spi1: spi@fe620000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe620000 0x0 0x1000>; + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 22>, <&dmac0 23>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; + status = "disabled"; + }; + + spi2: spi@fe630000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe630000 0x0 0x1000>; + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 24>, <&dmac0 25>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; + status = "disabled"; + }; + + spi3: spi@fe640000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe640000 0x0 0x1000>; + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 26>, <&dmac0 27>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; + status = "disabled"; + }; + uart1: serial@fe650000 { compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; reg = <0x0 0xfe650000 0x0 0x100>; -- 2.34.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds the four spi nodes (spi0, spi1, spi2, spi3) to the rk356x dtsi. These are from the downstream device tree, though I have double-checked that their interrupts and DMA numbers are correct. I have also tested spi1 with an SPI device. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi index 46d9552f6028..57c0197cc65a 100644 --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi @@ -39,6 +39,10 @@ aliases { serial7 = &uart7; serial8 = &uart8; serial9 = &uart9; + spi0 = &spi0; + spi1 = &spi1; + spi2 = &spi2; + spi3 = &spi3; }; cpus { @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { clock-names = "tclk", "pclk"; }; + spi0: spi@fe610000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe610000 0x0 0x1000>; + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 20>, <&dmac0 21>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; + status = "disabled"; + }; + + spi1: spi@fe620000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe620000 0x0 0x1000>; + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 22>, <&dmac0 23>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; + status = "disabled"; + }; + + spi2: spi@fe630000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe630000 0x0 0x1000>; + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 24>, <&dmac0 25>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; + status = "disabled"; + }; + + spi3: spi@fe640000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe640000 0x0 0x1000>; + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 26>, <&dmac0 27>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; + status = "disabled"; + }; + uart1: serial@fe650000 { compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; reg = <0x0 0xfe650000 0x0 0x100>; -- 2.34.0 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel This adds the four spi nodes (spi0, spi1, spi2, spi3) to the rk356x dtsi. These are from the downstream device tree, though I have double-checked that their interrupts and DMA numbers are correct. I have also tested spi1 with an SPI device. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi index 46d9552f6028..57c0197cc65a 100644 --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi @@ -39,6 +39,10 @@ aliases { serial7 = &uart7; serial8 = &uart8; serial9 = &uart9; + spi0 = &spi0; + spi1 = &spi1; + spi2 = &spi2; + spi3 = &spi3; }; cpus { @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { clock-names = "tclk", "pclk"; }; + spi0: spi@fe610000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe610000 0x0 0x1000>; + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 20>, <&dmac0 21>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; + status = "disabled"; + }; + + spi1: spi@fe620000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe620000 0x0 0x1000>; + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 22>, <&dmac0 23>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; + status = "disabled"; + }; + + spi2: spi@fe630000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe630000 0x0 0x1000>; + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 24>, <&dmac0 25>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; + status = "disabled"; + }; + + spi3: spi@fe640000 { + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; + reg = <0x0 0xfe640000 0x0 0x1000>; + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; + clock-names = "spiclk", "apb_pclk"; + dmas = <&dmac0 26>, <&dmac0 27>; + dma-names = "tx", "rx"; + pinctrl-names = "default", "high_speed"; + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; + status = "disabled"; + }; + uart1: serial@fe650000 { compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; reg = <0x0 0xfe650000 0x0 0x100>; -- 2.34.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2021-11-26 17:51 ` Johan Jonker -1 siblings, 0 replies; 24+ messages in thread From: Johan Jonker @ 2021-11-26 17:51 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Nicolas, Could you test with: ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > rk356x dtsi. These are from the downstream device tree, though > I have double-checked that their interrupts and DMA numbers are > correct. I have also tested spi1 with an SPI device. > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > --- > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > index 46d9552f6028..57c0197cc65a 100644 > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > @@ -39,6 +39,10 @@ aliases { > serial7 = &uart7; > serial8 = &uart8; > serial9 = &uart9; > + spi0 = &spi0; > + spi1 = &spi1; > + spi2 = &spi2; > + spi3 = &spi3; > }; > > cpus { > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > clock-names = "tclk", "pclk"; > }; > > + spi0: spi@fe610000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe610000 0x0 0x1000>; > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; Move above status. > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 20>, <&dmac0 21>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; Keep position of pinctrl-names in line with other nodes in this file below pinctrl-0. arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: pinctrl-names:1: 'sleep' was expected From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml Missing "high_speed" support? spi: rockchip: set higher io driver when sclk higher than 24MHz https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; Missing Linux mainline driver and document support? Maybe skip for now? spi0m0_cs0_hs spi0m0_cs1_hs ??? > + status = "disabled"; > + }; > + spi0 { /omit-if-no-ref/ spi0m0_pins: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_none>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_none>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs0: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs1: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_pins: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_none>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_none>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_cs0: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_none>; }; }; spi0-hs { /omit-if-no-ref/ spi0m0_pins_hs: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs0_hs: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs1_hs: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_pins_hs: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_cs0_hs: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; }; }; > + spi1: spi@fe620000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe620000 0x0 0x1000>; > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 22>, <&dmac0 23>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi2: spi@fe630000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe630000 0x0 0x1000>; > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 24>, <&dmac0 25>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi3: spi@fe640000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe640000 0x0 0x1000>; > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 26>, <&dmac0 27>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > + status = "disabled"; > + }; > + > uart1: serial@fe650000 { > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > reg = <0x0 0xfe650000 0x0 0x100>; > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 17:51 ` Johan Jonker 0 siblings, 0 replies; 24+ messages in thread From: Johan Jonker @ 2021-11-26 17:51 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Nicolas, Could you test with: ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > rk356x dtsi. These are from the downstream device tree, though > I have double-checked that their interrupts and DMA numbers are > correct. I have also tested spi1 with an SPI device. > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > --- > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > index 46d9552f6028..57c0197cc65a 100644 > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > @@ -39,6 +39,10 @@ aliases { > serial7 = &uart7; > serial8 = &uart8; > serial9 = &uart9; > + spi0 = &spi0; > + spi1 = &spi1; > + spi2 = &spi2; > + spi3 = &spi3; > }; > > cpus { > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > clock-names = "tclk", "pclk"; > }; > > + spi0: spi@fe610000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe610000 0x0 0x1000>; > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; Move above status. > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 20>, <&dmac0 21>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; Keep position of pinctrl-names in line with other nodes in this file below pinctrl-0. arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: pinctrl-names:1: 'sleep' was expected From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml Missing "high_speed" support? spi: rockchip: set higher io driver when sclk higher than 24MHz https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; Missing Linux mainline driver and document support? Maybe skip for now? spi0m0_cs0_hs spi0m0_cs1_hs ??? > + status = "disabled"; > + }; > + spi0 { /omit-if-no-ref/ spi0m0_pins: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_none>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_none>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs0: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs1: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_pins: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_none>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_none>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_cs0: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_none>; }; }; spi0-hs { /omit-if-no-ref/ spi0m0_pins_hs: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs0_hs: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs1_hs: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_pins_hs: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_cs0_hs: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; }; }; > + spi1: spi@fe620000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe620000 0x0 0x1000>; > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 22>, <&dmac0 23>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi2: spi@fe630000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe630000 0x0 0x1000>; > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 24>, <&dmac0 25>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi3: spi@fe640000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe640000 0x0 0x1000>; > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 26>, <&dmac0 27>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > + status = "disabled"; > + }; > + > uart1: serial@fe650000 { > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > reg = <0x0 0xfe650000 0x0 0x100>; > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 17:51 ` Johan Jonker 0 siblings, 0 replies; 24+ messages in thread From: Johan Jonker @ 2021-11-26 17:51 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Nicolas, Could you test with: ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > rk356x dtsi. These are from the downstream device tree, though > I have double-checked that their interrupts and DMA numbers are > correct. I have also tested spi1 with an SPI device. > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > --- > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > index 46d9552f6028..57c0197cc65a 100644 > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > @@ -39,6 +39,10 @@ aliases { > serial7 = &uart7; > serial8 = &uart8; > serial9 = &uart9; > + spi0 = &spi0; > + spi1 = &spi1; > + spi2 = &spi2; > + spi3 = &spi3; > }; > > cpus { > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > clock-names = "tclk", "pclk"; > }; > > + spi0: spi@fe610000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe610000 0x0 0x1000>; > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; Move above status. > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 20>, <&dmac0 21>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; Keep position of pinctrl-names in line with other nodes in this file below pinctrl-0. arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: pinctrl-names:1: 'sleep' was expected From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml Missing "high_speed" support? spi: rockchip: set higher io driver when sclk higher than 24MHz https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; Missing Linux mainline driver and document support? Maybe skip for now? spi0m0_cs0_hs spi0m0_cs1_hs ??? > + status = "disabled"; > + }; > + spi0 { /omit-if-no-ref/ spi0m0_pins: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_none>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_none>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs0: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m0_cs1: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_pins: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_none>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_none>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_none>; }; /omit-if-no-ref/ spi0m1_cs0: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_none>; }; }; spi0-hs { /omit-if-no-ref/ spi0m0_pins_hs: spi0m0-pins { rockchip,pins = /* spi0_clkm0 */ <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, /* spi0_misom0 */ <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, /* spi0_mosim0 */ <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs0_hs: spi0m0-cs0 { rockchip,pins = /* spi0_cs0m0 */ <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m0_cs1_hs: spi0m0-cs1 { rockchip,pins = /* spi0_cs1m0 */ <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_pins_hs: spi0m1-pins { rockchip,pins = /* spi0_clkm1 */ <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, /* spi0_misom1 */ <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, /* spi0_mosim1 */ <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; }; /omit-if-no-ref/ spi0m1_cs0_hs: spi0m1-cs0 { rockchip,pins = /* spi0_cs0m1 */ <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; }; }; > + spi1: spi@fe620000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe620000 0x0 0x1000>; > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 22>, <&dmac0 23>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi2: spi@fe630000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe630000 0x0 0x1000>; > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 24>, <&dmac0 25>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > + status = "disabled"; > + }; > + > + spi3: spi@fe640000 { > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > + reg = <0x0 0xfe640000 0x0 0x1000>; > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > + clock-names = "spiclk", "apb_pclk"; > + dmas = <&dmac0 26>, <&dmac0 27>; > + dma-names = "tx", "rx"; > + pinctrl-names = "default", "high_speed"; > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > + status = "disabled"; > + }; > + > uart1: serial@fe650000 { > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > reg = <0x0 0xfe650000 0x0 0x100>; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x 2021-11-26 17:51 ` Johan Jonker (?) @ 2021-11-26 18:59 ` Heiko Stübner -1 siblings, 0 replies; 24+ messages in thread From: Heiko Stübner @ 2021-11-26 18:59 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Johan Jonker Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Am Freitag, 26. November 2021, 18:51:29 CET schrieb Johan Jonker: > Hi Nicolas, > > Could you test with: > ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check > DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > > rk356x dtsi. These are from the downstream device tree, though > > I have double-checked that their interrupts and DMA numbers are > > correct. I have also tested spi1 with an SPI device. > > > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > > 1 file changed, 68 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > index 46d9552f6028..57c0197cc65a 100644 > > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > @@ -39,6 +39,10 @@ aliases { > > serial7 = &uart7; > > serial8 = &uart8; > > serial9 = &uart9; > > + spi0 = &spi0; > > + spi1 = &spi1; > > + spi2 = &spi2; > > + spi3 = &spi3; > > }; > > > > cpus { > > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > > clock-names = "tclk", "pclk"; > > }; > > > > + spi0: spi@fe610000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe610000 0x0 0x1000>; > > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > > > + #address-cells = <1>; > > + #size-cells = <0>; > > Move above status. > > > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 20>, <&dmac0 21>; > > + dma-names = "tx", "rx"; > > > + pinctrl-names = "default", "high_speed"; > > Keep position of pinctrl-names in line with other nodes in this file > below pinctrl-0. I do believe we always have pinctrl-names on top in most dts files, as it describes what the -0, -1 etc are, i.e. pinctrl-names = "statename-for-0", "statename-for-1", "statename-for-2"... pinctrl-0 = pinctrl-1 = Heiko > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: > pinctrl-names:1: 'sleep' was expected > From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > Missing "high_speed" support? > > spi: rockchip: set higher io driver when sclk higher than 24MHz > https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > > > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > > > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; > > Missing Linux mainline driver and document support? Maybe skip for now? > > spi0m0_cs0_hs spi0m0_cs1_hs ??? > > > + status = "disabled"; > > + }; > > + > > > > spi0 { > /omit-if-no-ref/ > spi0m0_pins: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_none>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_none>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs0: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs1: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_pins: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_none>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_none>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_cs0: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_none>; > }; > }; > > spi0-hs { > /omit-if-no-ref/ > spi0m0_pins_hs: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs0_hs: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs1_hs: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_pins_hs: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_cs0_hs: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; > }; > }; > > > > > + spi1: spi@fe620000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe620000 0x0 0x1000>; > > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 22>, <&dmac0 23>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi2: spi@fe630000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe630000 0x0 0x1000>; > > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 24>, <&dmac0 25>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi3: spi@fe640000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe640000 0x0 0x1000>; > > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 26>, <&dmac0 27>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > uart1: serial@fe650000 { > > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > > reg = <0x0 0xfe650000 0x0 0x100>; > > > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 18:59 ` Heiko Stübner 0 siblings, 0 replies; 24+ messages in thread From: Heiko Stübner @ 2021-11-26 18:59 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Johan Jonker Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Am Freitag, 26. November 2021, 18:51:29 CET schrieb Johan Jonker: > Hi Nicolas, > > Could you test with: > ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check > DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > > rk356x dtsi. These are from the downstream device tree, though > > I have double-checked that their interrupts and DMA numbers are > > correct. I have also tested spi1 with an SPI device. > > > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > > 1 file changed, 68 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > index 46d9552f6028..57c0197cc65a 100644 > > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > @@ -39,6 +39,10 @@ aliases { > > serial7 = &uart7; > > serial8 = &uart8; > > serial9 = &uart9; > > + spi0 = &spi0; > > + spi1 = &spi1; > > + spi2 = &spi2; > > + spi3 = &spi3; > > }; > > > > cpus { > > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > > clock-names = "tclk", "pclk"; > > }; > > > > + spi0: spi@fe610000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe610000 0x0 0x1000>; > > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > > > + #address-cells = <1>; > > + #size-cells = <0>; > > Move above status. > > > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 20>, <&dmac0 21>; > > + dma-names = "tx", "rx"; > > > + pinctrl-names = "default", "high_speed"; > > Keep position of pinctrl-names in line with other nodes in this file > below pinctrl-0. I do believe we always have pinctrl-names on top in most dts files, as it describes what the -0, -1 etc are, i.e. pinctrl-names = "statename-for-0", "statename-for-1", "statename-for-2"... pinctrl-0 = pinctrl-1 = Heiko > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: > pinctrl-names:1: 'sleep' was expected > From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > Missing "high_speed" support? > > spi: rockchip: set higher io driver when sclk higher than 24MHz > https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > > > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > > > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; > > Missing Linux mainline driver and document support? Maybe skip for now? > > spi0m0_cs0_hs spi0m0_cs1_hs ??? > > > + status = "disabled"; > > + }; > > + > > > > spi0 { > /omit-if-no-ref/ > spi0m0_pins: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_none>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_none>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs0: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs1: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_pins: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_none>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_none>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_cs0: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_none>; > }; > }; > > spi0-hs { > /omit-if-no-ref/ > spi0m0_pins_hs: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs0_hs: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs1_hs: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_pins_hs: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_cs0_hs: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; > }; > }; > > > > > + spi1: spi@fe620000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe620000 0x0 0x1000>; > > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 22>, <&dmac0 23>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi2: spi@fe630000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe630000 0x0 0x1000>; > > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 24>, <&dmac0 25>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi3: spi@fe640000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe640000 0x0 0x1000>; > > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 26>, <&dmac0 27>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > uart1: serial@fe650000 { > > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > > reg = <0x0 0xfe650000 0x0 0x100>; > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x @ 2021-11-26 18:59 ` Heiko Stübner 0 siblings, 0 replies; 24+ messages in thread From: Heiko Stübner @ 2021-11-26 18:59 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Johan Jonker Cc: devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Am Freitag, 26. November 2021, 18:51:29 CET schrieb Johan Jonker: > Hi Nicolas, > > Could you test with: > ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make dtbs_check > DT_SCHEMA_FILES=Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > On 11/26/21 4:43 PM, Nicolas Frattaroli wrote: > > This adds the four spi nodes (spi0, spi1, spi2, spi3) to the > > rk356x dtsi. These are from the downstream device tree, though > > I have double-checked that their interrupts and DMA numbers are > > correct. I have also tested spi1 with an SPI device. > > > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> > > --- > > arch/arm64/boot/dts/rockchip/rk356x.dtsi | 68 ++++++++++++++++++++++++ > > 1 file changed, 68 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > index 46d9552f6028..57c0197cc65a 100644 > > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > @@ -39,6 +39,10 @@ aliases { > > serial7 = &uart7; > > serial8 = &uart8; > > serial9 = &uart9; > > + spi0 = &spi0; > > + spi1 = &spi1; > > + spi2 = &spi2; > > + spi3 = &spi3; > > }; > > > > cpus { > > @@ -742,6 +746,70 @@ wdt: watchdog@fe600000 { > > clock-names = "tclk", "pclk"; > > }; > > > > + spi0: spi@fe610000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe610000 0x0 0x1000>; > > + interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; > > > + #address-cells = <1>; > > + #size-cells = <0>; > > Move above status. > > > + clocks = <&cru CLK_SPI0>, <&cru PCLK_SPI0>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 20>, <&dmac0 21>; > > + dma-names = "tx", "rx"; > > > + pinctrl-names = "default", "high_speed"; > > Keep position of pinctrl-names in line with other nodes in this file > below pinctrl-0. I do believe we always have pinctrl-names on top in most dts files, as it describes what the -0, -1 etc are, i.e. pinctrl-names = "statename-for-0", "statename-for-1", "statename-for-2"... pinctrl-0 = pinctrl-1 = Heiko > arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dt.yaml: spi@fe610000: > pinctrl-names:1: 'sleep' was expected > From schema: /Documentation/devicetree/bindings/spi/spi-rockchip.yaml > > Missing "high_speed" support? > > spi: rockchip: set higher io driver when sclk higher than 24MHz > https://github.com/rockchip-linux/kernel/commit/7e4349ec9d70dbcfe7f78125700d2685539ea1bd > > > + pinctrl-0 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins>; > > > + pinctrl-1 = <&spi0m0_cs0 &spi0m0_cs1 &spi0m0_pins_hs>; > > Missing Linux mainline driver and document support? Maybe skip for now? > > spi0m0_cs0_hs spi0m0_cs1_hs ??? > > > + status = "disabled"; > > + }; > > + > > > > spi0 { > /omit-if-no-ref/ > spi0m0_pins: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_none>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_none>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs0: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m0_cs1: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_pins: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_none>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_none>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_none>; > }; > > /omit-if-no-ref/ > spi0m1_cs0: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_none>; > }; > }; > > spi0-hs { > /omit-if-no-ref/ > spi0m0_pins_hs: spi0m0-pins { > rockchip,pins = > /* spi0_clkm0 */ > <0 RK_PB5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_misom0 */ > <0 RK_PC5 2 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim0 */ > <0 RK_PB6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs0_hs: spi0m0-cs0 { > rockchip,pins = > /* spi0_cs0m0 */ > <0 RK_PC6 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m0_cs1_hs: spi0m0-cs1 { > rockchip,pins = > /* spi0_cs1m0 */ > <0 RK_PC4 2 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_pins_hs: spi0m1-pins { > rockchip,pins = > /* spi0_clkm1 */ > <2 RK_PD3 3 &pcfg_pull_up_drv_level_1>, > /* spi0_misom1 */ > <2 RK_PD0 3 &pcfg_pull_up_drv_level_1>, > /* spi0_mosim1 */ > <2 RK_PD1 3 &pcfg_pull_up_drv_level_1>; > }; > > /omit-if-no-ref/ > spi0m1_cs0_hs: spi0m1-cs0 { > rockchip,pins = > /* spi0_cs0m1 */ > <2 RK_PD2 3 &pcfg_pull_up_drv_level_1>; > }; > }; > > > > > + spi1: spi@fe620000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe620000 0x0 0x1000>; > > + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI1>, <&cru PCLK_SPI1>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 22>, <&dmac0 23>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins>; > > + pinctrl-1 = <&spi1m0_cs0 &spi1m0_cs1 &spi1m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi2: spi@fe630000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe630000 0x0 0x1000>; > > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI2>, <&cru PCLK_SPI2>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 24>, <&dmac0 25>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins>; > > + pinctrl-1 = <&spi2m0_cs0 &spi2m0_cs1 &spi2m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > + spi3: spi@fe640000 { > > + compatible = "rockchip,rk3568-spi", "rockchip,rk3066-spi"; > > + reg = <0x0 0xfe640000 0x0 0x1000>; > > + interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&cru CLK_SPI3>, <&cru PCLK_SPI3>; > > + clock-names = "spiclk", "apb_pclk"; > > + dmas = <&dmac0 26>, <&dmac0 27>; > > + dma-names = "tx", "rx"; > > + pinctrl-names = "default", "high_speed"; > > + pinctrl-0 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins>; > > + pinctrl-1 = <&spi3m0_cs0 &spi3m0_cs1 &spi3m0_pins_hs>; > > + status = "disabled"; > > + }; > > + > > uart1: serial@fe650000 { > > compatible = "rockchip,rk3568-uart", "snps,dw-apb-uart"; > > reg = <0x0 0xfe650000 0x0 0x100>; > > > ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2021-11-26 15:43 ` Nicolas Frattaroli -1 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 Model A has the SPI pins broken out on its pin header. The actual pins being used though are not the m0 variant, but the m1 variant, which also lacks the cs1 pin. This commit overrides pinctrl-0 accordingly for this board. spi1 is intentionally left disabled, as anyone wishing to add SPI devices needs to edit the dts anyway, and the pins are more useful as GPIOs for the rest of the users. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts index 4d4b2a301b1a..166399b7f13f 100644 --- a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts @@ -509,6 +509,11 @@ &spdif { status = "okay"; }; +&spi1 { + pinctrl-names = "default"; + pinctrl-0 = <&spi1m1_cs0 &spi1m1_pins>; +}; + &tsadc { /* tshut mode 0:CRU 1:GPIO */ rockchip,hw-tshut-mode = <1>; -- 2.34.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 Model A has the SPI pins broken out on its pin header. The actual pins being used though are not the m0 variant, but the m1 variant, which also lacks the cs1 pin. This commit overrides pinctrl-0 accordingly for this board. spi1 is intentionally left disabled, as anyone wishing to add SPI devices needs to edit the dts anyway, and the pins are more useful as GPIOs for the rest of the users. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts index 4d4b2a301b1a..166399b7f13f 100644 --- a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts @@ -509,6 +509,11 @@ &spdif { status = "okay"; }; +&spi1 { + pinctrl-names = "default"; + pinctrl-0 = <&spi1m1_cs0 &spi1m1_pins>; +}; + &tsadc { /* tshut mode 0:CRU 1:GPIO */ rockchip,hw-tshut-mode = <1>; -- 2.34.0 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A @ 2021-11-26 15:43 ` Nicolas Frattaroli 0 siblings, 0 replies; 24+ messages in thread From: Nicolas Frattaroli @ 2021-11-26 15:43 UTC (permalink / raw) To: Rob Herring, Heiko Stuebner Cc: Nicolas Frattaroli, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel The Quartz64 Model A has the SPI pins broken out on its pin header. The actual pins being used though are not the m0 variant, but the m1 variant, which also lacks the cs1 pin. This commit overrides pinctrl-0 accordingly for this board. spi1 is intentionally left disabled, as anyone wishing to add SPI devices needs to edit the dts anyway, and the pins are more useful as GPIOs for the rest of the users. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts index 4d4b2a301b1a..166399b7f13f 100644 --- a/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3566-quartz64-a.dts @@ -509,6 +509,11 @@ &spdif { status = "okay"; }; +&spi1 { + pinctrl-names = "default"; + pinctrl-0 = <&spi1m1_cs0 &spi1m1_pins>; +}; + &tsadc { /* tshut mode 0:CRU 1:GPIO */ rockchip,hw-tshut-mode = <1>; -- 2.34.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: (subset) [PATCH 0/3] RK356x/Quartz64 Model A SPI 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2021-11-29 16:45 ` Mark Brown -1 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2021-11-29 16:45 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Nicolas Frattaroli Cc: linux-arm-kernel, linux-rockchip, devicetree, linux-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: 07fb78a78de4e67b5d6d5407aeee1250a327a698 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: (subset) [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2021-11-29 16:45 ` Mark Brown 0 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2021-11-29 16:45 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Nicolas Frattaroli Cc: linux-arm-kernel, linux-rockchip, devicetree, linux-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: 07fb78a78de4e67b5d6d5407aeee1250a327a698 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: (subset) [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2021-11-29 16:45 ` Mark Brown 0 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2021-11-29 16:45 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Nicolas Frattaroli Cc: linux-arm-kernel, linux-rockchip, devicetree, linux-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: 07fb78a78de4e67b5d6d5407aeee1250a327a698 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/3] RK356x/Quartz64 Model A SPI 2021-11-26 15:43 ` Nicolas Frattaroli (?) @ 2022-01-25 10:20 ` Mark Brown -1 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2022-01-25 10:20 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: linux-kernel, devicetree, linux-rockchip, linux-arm-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: bdac3bbd0dc63873a9c606b8e4f814e6d61d288d [2/3] arm64: dts: rockchip: Add spi nodes on rk356x (no commit info) [3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A (no commit info) All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2022-01-25 10:20 ` Mark Brown 0 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2022-01-25 10:20 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: linux-kernel, devicetree, linux-rockchip, linux-arm-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: bdac3bbd0dc63873a9c606b8e4f814e6d61d288d [2/3] arm64: dts: rockchip: Add spi nodes on rk356x (no commit info) [3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A (no commit info) All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/3] RK356x/Quartz64 Model A SPI @ 2022-01-25 10:20 ` Mark Brown 0 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2022-01-25 10:20 UTC (permalink / raw) To: Nicolas Frattaroli, Rob Herring, Heiko Stuebner Cc: linux-kernel, devicetree, linux-rockchip, linux-arm-kernel On Fri, 26 Nov 2021 16:43:41 +0100, Nicolas Frattaroli wrote: > The first patch of this series adds a compatible for rk3568-spi > to the DT bindings. > > The second adds the SPI nodes for RK3566 and RK3568 SoCs. The nodes > were lifted from the downstream vendor kernel's devicetree, and were > double-checked for correctness. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-linus Thanks! [1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible commit: bdac3bbd0dc63873a9c606b8e4f814e6d61d288d [2/3] arm64: dts: rockchip: Add spi nodes on rk356x (no commit info) [3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A (no commit info) All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2022-01-25 10:31 UTC | newest] Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-11-26 15:43 [PATCH 0/3] RK356x/Quartz64 Model A SPI Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` [PATCH 1/3] dt-bindings: spi: spi-rockchip: Add rk3568-spi compatible Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` [PATCH 2/3] arm64: dts: rockchip: Add spi nodes on rk356x Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 17:51 ` Johan Jonker 2021-11-26 17:51 ` Johan Jonker 2021-11-26 17:51 ` Johan Jonker 2021-11-26 18:59 ` Heiko Stübner 2021-11-26 18:59 ` Heiko Stübner 2021-11-26 18:59 ` Heiko Stübner 2021-11-26 15:43 ` [PATCH 3/3] arm64: dts: rockchip: Add spi1 pins on Quartz64 A Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-26 15:43 ` Nicolas Frattaroli 2021-11-29 16:45 ` (subset) [PATCH 0/3] RK356x/Quartz64 Model A SPI Mark Brown 2021-11-29 16:45 ` Mark Brown 2021-11-29 16:45 ` Mark Brown 2022-01-25 10:20 ` Mark Brown 2022-01-25 10:20 ` Mark Brown 2022-01-25 10:20 ` Mark Brown
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.