On Thu, Feb 21, 2019 at 9:45 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > > This adds device tree bindings for the IXP4xx interrupt > controller. It's a standard 2-cell controller. > > Cc: Marc Zyngier <marc.zyngier@xxxxxxx> > Cc: Jason Cooper <jason@xxxxxxxxxxxxxx> > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > ChangeLog v1->v2: > - Converted to use JSON yaml schema, why not. > - Not keeping Rob's ACK because I think he wants to take > a second look. > --- > .../intel,ixp4xx-interrupt.yaml | 57 +++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 58 insertions(+) > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml b/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml > new file mode 100644 > index 000000000000..f32c08f270d6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml > @@ -0,0 +1,57 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright 2018 Linaro Ltd. > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/interrupt/intel-ixp4xx-interrupt.yaml#"; > +$schema: "http://devicetree.org/meta-schemas/core.yaml#"; > + > +title: Intel IXP4xx XScale Networking Processors Interrupt Controller > + > +maintainers: > + - Linus Walleij <linus.walleij@xxxxxxxxxx> > + > +description: | > + This interrupt controller is found in the Intel IXP4xx processors. > + Some processors have 32 interrupts, some have up to 64 interrupts. > + The exact number of interrupts is determined from the compatible > + string. > + > + The distinct IXP4xx families with different interrupt controller > + variations are IXP42x, IXP43x, IXP45x and IXP46x. Those four > + families were the only ones to reach the developer and consumer > + market. > + > +properties: > + compatible: > + oneOf: You can drop oneOf since there is only 1. > + - items: > + - enum: > + - intel,ixp42x-interrupt > + - intel,ixp43x-interrupt > + - intel,ixp45x-interrupt > + - intel,ixp46x-interrupt > + > + reg: > + description: The register bank for the interrupt controller. No need to define what reg is here. Just 'maxItems: 1' is sufficient. > + > + interrupt-controller: true > + > + '#interrupt-cells': > + const: 2 > + description: The number of cells to define the interrupts. > + with two cells specified in interrupt-controller/interrupts.txt I'd drop description here, but I prefer things terse. I guess this does define the format of the cells. With the other changes, Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
