Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210422145330.73452-1-tsbogend@alpha.franken.de>
 <20210422145330.73452-2-tsbogend@alpha.franken.de> <1619115798.231872.3355876.nullmailer@robh.at.kernel.org>
In-Reply-To: <1619115798.231872.3355876.nullmailer@robh.at.kernel.org>
From:   Rob Herring <robh+dt@kernel.org>
Date:   Tue, 27 Apr 2021 16:57:43 -0500
Message-ID: <CAL_JsqKnAvc_rewNRDipEpJbp=Aj1yrTvgkJQMSH4E-pK4XBYg@mail.gmail.com>
Subject: Re: [PATCH v4 2/2] dt-bindings: interrupt-controller: Add IDT
 79RC3243x Interrupt Controller
To:     Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Cc:     Marc Zyngier <maz@kernel.org>, devicetree@vger.kernel.org,
        Thomas Gleixner <tglx@linutronix.de>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

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