[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4479c4cc-404b-48b9-9e0f-b6b371c8c159@cjdns.fr>
Date: Fri, 21 Mar 2025 18:19:10 +0100
From: Caleb James DeLisle <cjd@...ns.fr>
To: "Rob Herring (Arm)" <robh@...nel.org>
Cc: Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, devicetree@...r.kernel.org,
Thomas Gleixner <tglx@...utronix.de>, linux-mips@...r.kernel.org,
linux-kernel@...r.kernel.org, benjamin.larsson@...exis.eu,
Daniel Lezcano <daniel.lezcano@...aro.org>
Subject: Re: [PATCH v1 2/8] dt-bindings: interrupt-controller: Add EcoNet
EN751221 INTC
On 21/03/2025 16:52, Rob Herring (Arm) wrote:
> On Fri, 21 Mar 2025 13:46:27 +0000, Caleb James DeLisle wrote:
>> Document the device tree binding for the interrupt controller in the
>> EcoNet EN751221 MIPS SoC.
>>
>> Signed-off-by: Caleb James DeLisle <cjd@...ns.fr>
>> ---
>> If anyone is aware of a standard name for this "shadow interrupt" pattern,
>> please let me know and I will re-send with updated naming.
>> ---
>> .../econet,en751221-intc.yaml | 77 +++++++++++++++++++
>> 1 file changed, 77 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/econet,en751221-intc.yaml
>>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/interrupt-controller/econet,en751221-intc.example.dtb: interrupt-controller@...40000: 'interrupt-parent' is a required property
> from schema $id: http://devicetree.org/schemas/interrupt-controller/econet,en751221-intc.yaml#
>
Reproduced and fixed, thank you.
In the interest of limiting mailing list noise, I plan to send a v2 sometime
around Monday, ideally to aggregate a few more reviews in the process.
Thanks,
Caleb
Powered by blists - more mailing lists