* [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller
[not found] <20210422145330.73452-1-tsbogend@alpha.franken.de>
@ 2021-04-22 14:53 ` Thomas Bogendoerfer
2021-04-22 18:23 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Bogendoerfer @ 2021-04-22 14:53 UTC (permalink / raw)
To: Thomas Gleixner, Marc Zyngier, Rob Herring, linux-kernel, devicetree
Document DT bindings for IDT 79RC3243x Interrupt Controller.
Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
---
Changes in v4:
- renamed to idt,32434-pic
Changes in v3:
- fixed compatible string in example
Changes in v2:
- added dt binding doc
.../interrupt-controller/idt,32434-pic.yaml | 48 +++++++++++++++++++
1 file changed, 48 insertions(+)
create mode 100644 Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
diff --git a/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml b/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
new file mode 100644
index 000000000000..df5d8d1ead70
--- /dev/null
+++ b/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
@@ -0,0 +1,48 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/interrupt-controller/idt,32434-pic.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: IDT 79RC32434 Interrupt Controller Device Tree Bindings
+
+maintainers:
+ - Thomas Bogendoerfer <tsbogend@alpha.franken.de>
+
+allOf:
+ - $ref: /schemas/interrupt-controller.yaml#
+
+properties:
+ "#interrupt-cells":
+ const: 1
+
+ compatible:
+ const: idt,32434-pic
+
+ reg:
+ maxItems: 1
+
+ interrupt-controller: true
+
+required:
+ - "#interrupt-cells"
+ - compatible
+ - reg
+ - interrupt-controller
+
+additionalProperties: false
+
+examples:
+ - |
+ idtpic3: interrupt-controller@3800c {
+ compatible = "idt,32434-pic";
+ reg = <0x3800c 0x0c>;
+
+ interrupt-controller;
+ #interrupt-cells = <1>;
+
+ interrupt-parent = <&cpuintc>;
+ interrupts = <3>;
+ };
+
+...
--
2.29.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller
2021-04-22 14:53 ` [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller Thomas Bogendoerfer
@ 2021-04-22 18:23 ` Rob Herring
2021-04-27 21:57 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2021-04-22 18:23 UTC (permalink / raw)
To: Thomas Bogendoerfer
Cc: Marc Zyngier, devicetree, Rob Herring, Thomas Gleixner, linux-kernel
On Thu, 22 Apr 2021 16:53:29 +0200, Thomas Bogendoerfer wrote:
> Document DT bindings for IDT 79RC3243x Interrupt Controller.
>
> Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
> ---
> Changes in v4:
> - renamed to idt,32434-pic
>
> Changes in v3:
> - fixed compatible string in example
>
> Changes in v2:
> - added dt binding doc
>
> .../interrupt-controller/idt,32434-pic.yaml | 48 +++++++++++++++++++
> 1 file changed, 48 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.example.dt.yaml: interrupt-controller@3800c: 'interrupts' does not match any of the regexes: 'pinctrl-[0-9]+'
From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
See https://patchwork.ozlabs.org/patch/1469240
This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller
2021-04-22 18:23 ` Rob Herring
@ 2021-04-27 21:57 ` Rob Herring
0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring @ 2021-04-27 21:57 UTC (permalink / raw)
To: Thomas Bogendoerfer
Cc: Marc Zyngier, devicetree, Thomas Gleixner, linux-kernel
On Thu, Apr 22, 2021 at 1:23 PM Rob Herring <robh@kernel.org> wrote:
>
> On Thu, 22 Apr 2021 16:53:29 +0200, Thomas Bogendoerfer wrote:
> > Document DT bindings for IDT 79RC3243x Interrupt Controller.
> >
> > Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
> > ---
> > Changes in v4:
> > - renamed to idt,32434-pic
> >
> > Changes in v3:
> > - fixed compatible string in example
> >
> > Changes in v2:
> > - added dt binding doc
> >
> > .../interrupt-controller/idt,32434-pic.yaml | 48 +++++++++++++++++++
> > 1 file changed, 48 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
> >
>
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.example.dt.yaml: interrupt-controller@3800c: 'interrupts' does not match any of the regexes: 'pinctrl-[0-9]+'
> From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/interrupt-controller/idt,32434-pic.yaml
Now this is an error in Linus' tree. Please send a fix.
Rob
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-04-27 21:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20210422145330.73452-1-tsbogend@alpha.franken.de>
2021-04-22 14:53 ` [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller Thomas Bogendoerfer
2021-04-22 18:23 ` Rob Herring
2021-04-27 21:57 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).