Re: [PATCH v2 1/2] arm64: dts: qcom: nord: add GPI DMA controller nodes

From: Bartosz Golaszewski

Date: Thu Sep 24 2026 - 04:57:48 EST


On Thu, Sep 24, 2026 at 10:50 AM Mukesh Savaliya
<mukesh.savaliya@xxxxxxxxxxxxxxxx> wrote:
> >>>
> >>> we should be keeping GPI node default enabled similar to common QUPv3
> >>> wrapper node.
> >>> status = "ok" for all the nodes.
> >>>
> >>
> >> The pattern across Qualcomm platforms seems to disable the GPI DMA nodes at
> >> SoC-level and only enable them at board-level.
> >
> > For no reason - the DMA engines may be reserved through TZ config,
> > but for the past (high) N years that hasn't been the case
> >
>
> There is a reason to reserve this from TZ. Some use cases require GPI to
> be available during early boot, before Linux is up and running, which
> necessitates keeping the GPI instance enabled.
>
> Also, GPII is allocated per SE, while the GPI instance itself is shared
> across all SEs within a QUP. Given that a QUP typically contains eight
> SEs, it is likely that at least one SE will require GPI operation.
>
> For that reason, my preference is to keep the GPI node enabled by
> default. However, if there is a strong preference to make this
> board-specific and enable it only where required, I can work with that
> approach as well.
>
> I think it would be better to align on the expected default behavior
> now, rather than revisiting it later once additional users start
> depending on it.

If I understood Konrad correctly, he too is in favor of enabling the
GPI nodes. I was about to resend v3 with that changed.

Bart