2022-01-21 13:57:42

by Rob Herring (Arm)

Subject: [PATCH] dt-bindings: mfd: cirrus,madera: Fix 'interrupts' in example

The 'interrupts' properties takes an irq number, not a phandle, and
'interrupt-parent' isn't needed in examples.
---
Documentation/devicetree/bindings/mfd/cirrus,madera.yaml | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/mfd/cirrus,madera.yaml b/Documentation/devicetree/bindings/mfd/cirrus,madera.yaml
index 5dce62a7eff2..68c75a517c92 100644
--- a/Documentation/devicetree/bindings/mfd/cirrus,madera.yaml
+++ b/Documentation/devicetree/bindings/mfd/cirrus,madera.yaml
@@ -245,8 +245,7 @@ examples:

interrupt-controller;
#interrupt-cells = <2>;
- interrupts = <&host_irq1>;
- interrupt-parent = <&gic>;
+ interrupts = <4 1 0>;

gpio-controller;
#gpio-cells = <2>;
--
2.32.0

2022-01-21 19:11:00

by Charles Keepax

Subject: Re: [PATCH] dt-bindings: mfd: cirrus,madera: Fix 'interrupts' in example

On Tue, Jan 18, 2022 at 07:56:11PM -0600, Rob Herring wrote:
> The 'interrupts' properties takes an irq number, not a phandle, and
> 'interrupt-parent' isn't needed in examples.
> ---

Acked-by: Charles Keepax <[email protected]>

Thanks,
Charles

2022-01-21 22:35:32

by Rob Herring (Arm)

Subject: Re: [PATCH] dt-bindings: mfd: cirrus,madera: Fix 'interrupts' in example

On Tue, 18 Jan 2022 19:56:11 -0600, Rob Herring wrote:
> The 'interrupts' properties takes an irq number, not a phandle, and
> 'interrupt-parent' isn't needed in examples.
> ---
> Documentation/devicetree/bindings/mfd/cirrus,madera.yaml | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>

Applied (with my Sob added), thanks!

2022-01-22 00:41:29

by Lee Jones

Subject: Re: [PATCH] dt-bindings: mfd: cirrus,madera: Fix 'interrupts' in example

On Thu, 20 Jan 2022, Rob Herring wrote:

> On Tue, 18 Jan 2022 19:56:11 -0600, Rob Herring wrote:
> > The 'interrupts' properties takes an irq number, not a phandle, and
> > 'interrupt-parent' isn't needed in examples.
> > ---
> > Documentation/devicetree/bindings/mfd/cirrus,madera.yaml | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
>
> Applied (with my Sob added), thanks!

Was going to say ... :)

Please add my Ack too:

Acked-by: Lee Jones <[email protected]>

--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog