Re: [PATCH] dt-bindings: dma: qcom,gpi: Document the Nord GPI DMA engine

From: Frank Li

Date: Fri Oct 02 2026 - 12:39:34 EST


On Fri, Oct 02, 2026 at 12:28:40PM -0400, Bartosz Golaszewski wrote:
> On Fri, 2 Oct 2026 17:31:41 +0200, Frank Li <Frank.li@xxxxxxxxxxx> said:
> > On Fri, Oct 02, 2026 at 11:46:57AM +0300, Bartosz Golaszewski wrote:
> >> On Wed, 16 Sep 2026 17:21:29 +0200, Bartosz Golaszewski
> >> <bartosz.golaszewski@xxxxxxxxxxxxxxxx> said:
> >> > Add the compatible string for the GPI DMA engine found on the Nord
> >> > family of SoCs.
> >> >
> >> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxxxxxxxx>
> >> > ---
> >> > Documentation/devicetree/bindings/dma/qcom,gpi.yaml | 1 +
> >> > 1 file changed, 1 insertion(+)
> >> >
> >> > diff --git a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
> >> > index 54dca623223d51373a4d54591f3ebbf5e31fca17..ad70717b5b86e600e74d6cfe42015dc03db25ba9 100644
> >> > --- a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
> >> > +++ b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
> >> > @@ -29,6 +29,7 @@ properties:
> >> > - qcom,hawi-gpi-dma
> >> > - qcom,kaanapali-gpi-dma
> >> > - qcom,milos-gpi-dma
> >> > + - qcom,nord-gpi-dma
> >> > - qcom,qcm2290-gpi-dma
> >> > - qcom,qcs8300-gpi-dma
> >> > - qcom,qdu1000-gpi-dma
> >> >
> >> > ---
> >> > base-commit: e6e35979777d646fe3c7c94dca7dd32fb25d45f4
> >> > change-id: 20260916-nord-gpi-dt-bindings-f4b10102d22b
> >> >
> >> > Best regards,
> >> > --
> >> > Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxxxxxxxx>
> >> >
> >> >
> >>
> >> Vinod: gentle ping, can this make v7.4, please? Otherwise, can you Ack it
> >> and let Bjorn pick it up?
> >
> > I have not seen dt team's review tags yet. You should ping DT Team firstly.
> >
> > And subject:
> > dt-bindings: dmaengine: qcom,gpi, although there some use "dma",
> > but avoid confuse in future, all will consistently use "dmeangine".
> >
>
> Do we require DT maintainers' review tags even for a single liner changes
> like this one that don't carry much potential for errors? For GPIO I typically
> take small changes like this even without them.

That's my understand, Krzysztof or Rob may provide more guide line.

Frank

>
> Bart