[v1] arm64: dts: qcom: nord: add GPI DMA nodes

[PATCH 0/2] arm64: dts: qcom: nord: add GPI DMA nodes

Posted by Bartosz Golaszewski 1 week, 1 day ago

Add GPI DMA nodes for Nord and reference them from the QUPv3 I2C/SPI
nodes.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
---
Bartosz Golaszewski (2):
      arm64: dts: qcom: nord: add GPI DMA controller nodes
      arm64: dts: qcom: nord: wire up GPI DMA for QUP I2C/SPI engines

 arch/arm64/boot/dts/qcom/nord.dtsi | 219 +++++++++++++++++++++++++++++++++++++
 1 file changed, 219 insertions(+)
---
base-commit: 3c60a5740403002069d54463b6b0f603b9ba08bb
change-id: 20260916-nord-gpi-dts-40456500b7b1

Best regards,
--  
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>

Re: [PATCH 0/2] arm64: dts: qcom: nord: add GPI DMA nodes

Posted by Shawn Guo 1 week, 1 day ago

On Wed, Sep 16, 2026 at 05:55:37PM +0200, Bartosz Golaszewski wrote:
> Add GPI DMA nodes for Nord and reference them from the QUPv3 I2C/SPI
> nodes.

As the nord base dts series hasn't been applied, we need to call out the
dependency in the cover letter.

Shawn

> 
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> ---
> Bartosz Golaszewski (2):
>       arm64: dts: qcom: nord: add GPI DMA controller nodes
>       arm64: dts: qcom: nord: wire up GPI DMA for QUP I2C/SPI engines
> 
>  arch/arm64/boot/dts/qcom/nord.dtsi | 219 +++++++++++++++++++++++++++++++++++++
>  1 file changed, 219 insertions(+)
> ---
> base-commit: 3c60a5740403002069d54463b6b0f603b9ba08bb
> change-id: 20260916-nord-gpi-dts-40456500b7b1
> 
> Best regards,
> --  
> Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> 
>

Re: [PATCH 0/2] arm64: dts: qcom: nord: add GPI DMA nodes

Posted by Shawn Guo 4 days, 1 hour ago

On Thu, Sep 17, 2026 at 11:00:36AM +0800, Shawn Guo wrote:
> On Wed, Sep 16, 2026 at 05:55:37PM +0200, Bartosz Golaszewski wrote:
> > Add GPI DMA nodes for Nord and reference them from the QUPv3 I2C/SPI
> > nodes.
> 
> As the nord base dts series hasn't been applied, we need to call out the
> dependency in the cover letter.

This comment is not valid any more, as Bjorn just applied the Nord base
DTS series.

But I just found that CHECK_DTBS of qcom/nord-rrd.dtb is not clean.

arch/arm64/boot/dts/qcom/nord-rrd.dtb: dma-controller@800000 (qcom,nord-gpi-dma): compatible: 'oneOf' conditional failed, one must be fixed:
        ['qcom,nord-gpi-dma', 'qcom,sm6350-gpi-dma'] is too long

Shawn

Re: [PATCH 0/2] arm64: dts: qcom: nord: add GPI DMA nodes

Posted by Bartosz Golaszewski 3 days, 13 hours ago

On Mon, 21 Sep 2026 04:11:29 +0200, Shawn Guo
<shengchao.guo@oss.qualcomm.com> said:
> On Thu, Sep 17, 2026 at 11:00:36AM +0800, Shawn Guo wrote:
>> On Wed, Sep 16, 2026 at 05:55:37PM +0200, Bartosz Golaszewski wrote:
>> > Add GPI DMA nodes for Nord and reference them from the QUPv3 I2C/SPI
>> > nodes.
>>
>> As the nord base dts series hasn't been applied, we need to call out the
>> dependency in the cover letter.
>
> This comment is not valid any more, as Bjorn just applied the Nord base
> DTS series.
>
> But I just found that CHECK_DTBS of qcom/nord-rrd.dtb is not clean.
>
> arch/arm64/boot/dts/qcom/nord-rrd.dtb: dma-controller@800000 (qcom,nord-gpi-dma): compatible: 'oneOf' conditional failed, one must be fixed:
>         ['qcom,nord-gpi-dma', 'qcom,sm6350-gpi-dma'] is too long
>
> Shawn
>

The binding adding the compatible was sent separately[1] but there's still
another issue that needs fixing. I'm not sure if it's a bad rebase because I'd
swear I did remove the dmas from uart nodes but they are still there.

I'll send a v2.

Bart

[1] https://lore.kernel.org/all/20260916-nord-gpi-dt-bindings-v1-1-857bf9e84999@oss.qualcomm.com/