Linux-ARM-MSM Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH v2 0/2] arm64: sm8150: Add minimal DMA support
@ 2021-04-17  6:19 Felipe Balbi
  2021-04-17  6:19 ` [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150 Felipe Balbi
  2021-04-17  6:19 ` [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes Felipe Balbi
  0 siblings, 2 replies; 6+ messages in thread
From: Felipe Balbi @ 2021-04-17  6:19 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Vinod Koul
  Cc: Rob Herring, linux-arm-msm, devicetree, dmaengine, Konrad Dybcio,
	Felipe Balbi

From: Felipe Balbi <felipe.balbi@microsoft.com>

Hi,

With these two patches, GPI DMA probes fine on sm8150 (well, after
setting gpi_dma[012] status to okay). Future commits should come to add
relevant DMA channel mapping for the various IPs.

DTS patch a dependency on I2C patches by Caleb Connolly and SPI patch by
yours truly.

Felipe Balbi (2):
  DMA: qcom: gpi: add compatible for sm8150
  arm64: boot: dts: qcom: sm8150: Add DMA nodes

 .../devicetree/bindings/dma/qcom,gpi.yaml     |  1 +
 arch/arm64/boot/dts/qcom/sm8150.dtsi          | 70 +++++++++++++++++++
 drivers/dma/qcom/gpi.c                        |  1 +
 3 files changed, 72 insertions(+)

-- 
2.31.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150
  2021-04-17  6:19 [PATCH v2 0/2] arm64: sm8150: Add minimal DMA support Felipe Balbi
@ 2021-04-17  6:19 ` Felipe Balbi
  2021-04-20 10:08   ` Vinod Koul
  2021-04-17  6:19 ` [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes Felipe Balbi
  1 sibling, 1 reply; 6+ messages in thread
From: Felipe Balbi @ 2021-04-17  6:19 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Vinod Koul
  Cc: Rob Herring, linux-arm-msm, devicetree, dmaengine, Konrad Dybcio,
	Felipe Balbi

From: Felipe Balbi <felipe.balbi@microsoft.com>

No functional changes, just adding a new compatible for a diferent
SoC.

Signed-off-by: Felipe Balbi <felipe.balbi@microsoft.com>
---

Changes since v1:
	- No changes

 Documentation/devicetree/bindings/dma/qcom,gpi.yaml | 1 +
 drivers/dma/qcom/gpi.c                              | 1 +
 2 files changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
index f8142adf9aea..902e2e523386 100644
--- a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
+++ b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
@@ -20,6 +20,7 @@ properties:
   compatible:
     enum:
       - qcom,sdm845-gpi-dma
+      - qcom,sm8150-gpi-dma
 
   reg:
     maxItems: 1
diff --git a/drivers/dma/qcom/gpi.c b/drivers/dma/qcom/gpi.c
index 57f5ee4235c7..43ac3ab23d4c 100644
--- a/drivers/dma/qcom/gpi.c
+++ b/drivers/dma/qcom/gpi.c
@@ -2281,6 +2281,7 @@ static int gpi_probe(struct platform_device *pdev)
 
 static const struct of_device_id gpi_of_match[] = {
 	{ .compatible = "qcom,sdm845-gpi-dma" },
+	{ .compatible = "qcom,sm8150-gpi-dma" },
 	{ },
 };
 MODULE_DEVICE_TABLE(of, gpi_of_match);
-- 
2.31.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes
  2021-04-17  6:19 [PATCH v2 0/2] arm64: sm8150: Add minimal DMA support Felipe Balbi
  2021-04-17  6:19 ` [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150 Felipe Balbi
@ 2021-04-17  6:19 ` Felipe Balbi
  2021-04-17 12:03   ` Konrad Dybcio
  2021-04-20 10:09   ` Vinod Koul
  1 sibling, 2 replies; 6+ messages in thread
From: Felipe Balbi @ 2021-04-17  6:19 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Vinod Koul
  Cc: Rob Herring, linux-arm-msm, devicetree, dmaengine, Konrad Dybcio,
	Felipe Balbi

From: Felipe Balbi <felipe.balbi@microsoft.com>

With this patch, DMA has a chance of probing and doing something
useful.

Signed-off-by: Felipe Balbi <felipe.balbi@microsoft.com>
---

Changes since v1:

	- Remove dmas property from spi0 (will be added later as a
          separate patch

	- Correct dma unit name to dma-controller

	- Correct binding order (compatible, reg, dma-cells at the end)

 arch/arm64/boot/dts/qcom/sm8150.dtsi | 70 ++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
index c563f381a138..d2324c38fd73 100644
--- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
@@ -4,6 +4,7 @@
  * Copyright (c) 2019, Linaro Limited
  */
 
+#include <dt-bindings/dma/qcom-gpi.h>
 #include <dt-bindings/interrupt-controller/arm-gic.h>
 #include <dt-bindings/power/qcom-aoss-qmp.h>
 #include <dt-bindings/power/qcom-rpmpd.h>
@@ -577,6 +578,29 @@ gcc: clock-controller@100000 {
 				 <&sleep_clk>;
 		};
 
+		gpi_dma0: dma-controller@800000 {
+			compatible = "qcom,sm8150-gpi-dma";
+			reg = <0 0x800000 0 0x60000>;
+			interrupts = <GIC_SPI 244 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 245 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 247 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 248 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 249 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 250 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 251 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 252 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 253 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 254 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 255 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 256 IRQ_TYPE_LEVEL_HIGH>;
+			dma-channels = <13>;
+			dma-channel-mask = <0xfa>;
+			iommus = <&apps_smmu 0x00d6 0x0>;
+			#dma-cells = <3>;
+			status = "disabled";
+		};
+
 		qupv3_id_0: geniqup@8c0000 {
 			compatible = "qcom,geni-se-qup";
 			reg = <0x0 0x008c0000 0x0 0x6000>;
@@ -814,6 +838,29 @@ spi7: spi@89c000 {
 			};
 		};
 
+		gpi_dma1: dma-controller@a00000 {
+			compatible = "qcom,sm8150-gpi-dma";
+			reg = <0 0xa00000 0 0x60000>;
+			interrupts = <GIC_SPI 279 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 280 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 281 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 282 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 283 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 284 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 293 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 294 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 295 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 296 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 297 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 298 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 299 IRQ_TYPE_LEVEL_HIGH>;
+			dma-channels = <13>;
+			dma-channel-mask = <0xfa>;
+			iommus = <&apps_smmu 0x0616 0x0>;
+			#dma-cells = <3>;
+			status = "disabled";
+		};
+
 		qupv3_id_1: geniqup@ac0000 {
 			compatible = "qcom,geni-se-qup";
 			reg = <0x0 0x00ac0000 0x0 0x6000>;
@@ -1004,6 +1051,29 @@ spi16: spi@a94000 {
 			};
 		};
 
+		gpi_dma2: dma-controller@c00000 {
+			compatible = "qcom,sm8150-gpi-dma";
+			reg = <0 0xc00000 0 0x60000>;
+			interrupts = <GIC_SPI 588 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 589 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 590 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 591 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 592 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 593 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 594 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 595 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 596 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 597 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 598 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 599 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 600 IRQ_TYPE_LEVEL_HIGH>;
+			dma-channels = <13>;
+			dma-channel-mask = <0xfa>;
+			iommus = <&apps_smmu 0x07b6 0x0>;
+			#dma-cells = <3>;
+			status = "disabled";
+		};
+
 		qupv3_id_2: geniqup@cc0000 {
 			compatible = "qcom,geni-se-qup";
 			reg = <0x0 0x00cc0000 0x0 0x6000>;
-- 
2.31.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes
  2021-04-17  6:19 ` [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes Felipe Balbi
@ 2021-04-17 12:03   ` Konrad Dybcio
  2021-04-20 10:09   ` Vinod Koul
  1 sibling, 0 replies; 6+ messages in thread
From: Konrad Dybcio @ 2021-04-17 12:03 UTC (permalink / raw)
  To: Felipe Balbi, Andy Gross, Bjorn Andersson, Vinod Koul
  Cc: Rob Herring, linux-arm-msm, devicetree, dmaengine, Felipe Balbi

Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>


Konrad


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150
  2021-04-17  6:19 ` [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150 Felipe Balbi
@ 2021-04-20 10:08   ` Vinod Koul
  0 siblings, 0 replies; 6+ messages in thread
From: Vinod Koul @ 2021-04-20 10:08 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
	devicetree, dmaengine, Konrad Dybcio, Felipe Balbi

On 17-04-21, 09:19, Felipe Balbi wrote:
> From: Felipe Balbi <felipe.balbi@microsoft.com>
> 
> No functional changes, just adding a new compatible for a diferent
> SoC.

Applied after fixing subsystem name, thanks

-- 
~Vinod

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes
  2021-04-17  6:19 ` [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes Felipe Balbi
  2021-04-17 12:03   ` Konrad Dybcio
@ 2021-04-20 10:09   ` Vinod Koul
  1 sibling, 0 replies; 6+ messages in thread
From: Vinod Koul @ 2021-04-20 10:09 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
	devicetree, dmaengine, Konrad Dybcio, Felipe Balbi

On 17-04-21, 09:19, Felipe Balbi wrote:
> From: Felipe Balbi <felipe.balbi@microsoft.com>
> 
> With this patch, DMA has a chance of probing and doing something
> useful.

Reviewed-by: Vinod Koul <vkoul@kernel.org>

-- 
~Vinod

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, back to index

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-17  6:19 [PATCH v2 0/2] arm64: sm8150: Add minimal DMA support Felipe Balbi
2021-04-17  6:19 ` [PATCH v2 1/2] DMA: qcom: gpi: add compatible for sm8150 Felipe Balbi
2021-04-20 10:08   ` Vinod Koul
2021-04-17  6:19 ` [PATCH v2 2/2] arm64: boot: dts: qcom: sm8150: Add DMA nodes Felipe Balbi
2021-04-17 12:03   ` Konrad Dybcio
2021-04-20 10:09   ` Vinod Koul

Linux-ARM-MSM Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-arm-msm/0 linux-arm-msm/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-arm-msm linux-arm-msm/ https://lore.kernel.org/linux-arm-msm \
		linux-arm-msm@vger.kernel.org
	public-inbox-index linux-arm-msm

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-arm-msm


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git