Enabling dtc interrupt_provider check reveals the example is missing the
"#interrupt-cells" property as it is a dependency of
"interrupt-controller".

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 Documentation/devicetree/bindings/gpio/st,nomadik-gpio.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/gpio/st,nomadik-gpio.yaml b/Documentation/devicetree/bindings/gpio/st,nomadik-gpio.yaml
index 38d37d8f7201..b3e8951959b5 100644
--- a/Documentation/devicetree/bindings/gpio/st,nomadik-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/st,nomadik-gpio.yaml
@@ -89,6 +89,7 @@ examples:
         interrupts = <0 120 0x4>;
         #gpio-cells = <2>;
         gpio-controller;
+        #interrupt-cells = <2>;
         interrupt-controller;
         st,supports-sleepmode;
         gpio-bank = <1>;
-- 
2.45.2

From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Wed, 25 Sep 2024 12:35:03 -0500, Rob Herring (Arm) wrote:
> Enabling dtc interrupt_provider check reveals the example is missing the
> "#interrupt-cells" property as it is a dependency of
> "interrupt-controller".
> 
> 

Applied, thanks!

[1/1] dt-bindings: gpio: st,nomadik-gpio: Add missing "#interrupt-cells" to example
      commit: 35c013ac7362a5470d5f30f3da0af50215095121

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

On Wed, Sep 25, 2024 at 7:35 PM Rob Herring (Arm) <robh@kernel.org> wrote:

> Enabling dtc interrupt_provider check reveals the example is missing the
> "#interrupt-cells" property as it is a dependency of
> "interrupt-controller".
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Can you please merge this into the DT git tree?

Yours,
Linus Walleij

On Wed, Oct 2, 2024 at 3:04 PM Linus Walleij <linus.walleij@linaro.org> wrote:
>
> On Wed, Sep 25, 2024 at 7:35 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> > Enabling dtc interrupt_provider check reveals the example is missing the
> > "#interrupt-cells" property as it is a dependency of
> > "interrupt-controller".
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>
> Can you please merge this into the DT git tree?
>
> Yours,
> Linus Walleij

It's already upstream. I learned it when I tried picking it up. Rob:
any chance you could send b4 notifications when applying patches to
your tree?

Bartosz

On Wed, Oct 2, 2024 at 8:32 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> On Wed, Oct 2, 2024 at 3:04 PM Linus Walleij <linus.walleij@linaro.org> wrote:
> >
> > On Wed, Sep 25, 2024 at 7:35 PM Rob Herring (Arm) <robh@kernel.org> wrote:
> >
> > > Enabling dtc interrupt_provider check reveals the example is missing the
> > > "#interrupt-cells" property as it is a dependency of
> > > "interrupt-controller".
> > >
> > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> >
> > Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> >
> > Can you please merge this into the DT git tree?
> >
> > Yours,
> > Linus Walleij
>
> It's already upstream. I learned it when I tried picking it up. Rob:
> any chance you could send b4 notifications when applying patches to
> your tree?

It's not upstream. The ep9301 one is because Arnd applied it.

I do send notifications when applying things. Not using b4 though
because I haven't converted my scripts to it for that yet.

Rob

On Wed, Oct 2, 2024 at 4:22 PM Rob Herring <robh@kernel.org> wrote:
>
> On Wed, Oct 2, 2024 at 8:32 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > On Wed, Oct 2, 2024 at 3:04 PM Linus Walleij <linus.walleij@linaro.org> wrote:
> > >
> > > On Wed, Sep 25, 2024 at 7:35 PM Rob Herring (Arm) <robh@kernel.org> wrote:
> > >
> > > > Enabling dtc interrupt_provider check reveals the example is missing the
> > > > "#interrupt-cells" property as it is a dependency of
> > > > "interrupt-controller".
> > > >
> > > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> > >
> > > Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> > >
> > > Can you please merge this into the DT git tree?
> > >
> > > Yours,
> > > Linus Walleij
> >
> > It's already upstream. I learned it when I tried picking it up. Rob:
> > any chance you could send b4 notifications when applying patches to
> > your tree?
>
> It's not upstream. The ep9301 one is because Arnd applied it.
>
> I do send notifications when applying things. Not using b4 though
> because I haven't converted my scripts to it for that yet.
>
> Rob

Indeed, I confused the patches. I picked it up into the GPIO tree.

Bartosz

On Wed, Sep 25, 2024 at 12:35:03PM -0500, Rob Herring (Arm) wrote:
> Enabling dtc interrupt_provider check reveals the example is missing the
> "#interrupt-cells" property as it is a dependency of
> "interrupt-controller".
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Acked-by: Conor Dooley <conor.dooley@microchip.com>