The mcp23s08 device and friends are interrupt /client/ nodes, and should not reference the interrupt controller device tree properties "interrupt-controller" and "interrupt-cells" [0]. Remove the confusing "interrupt-controller" and "interrupt-cells" properties from the pinctrl-mcp23s08 devicetree bindings documentation. [0] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt Signed-off-by: Thomas Preston <thomas.preston@xxxxxxxxxxxxxxx> --- .../devicetree/bindings/pinctrl/pinctrl-mcp23s08.txt | 8 -------- 1 file changed, 8 deletions(-) diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.txt b/Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.txt index 8b94aa8f5971..bb1b53030552 100644 --- a/Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.txt +++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.txt @@ -43,10 +43,6 @@ Required device specific properties (only for SPI chips): - spi-max-frequency = The maximum frequency this chip is able to handle Optional properties: -- #interrupt-cells : Should be two. - - first cell is the pin number - - second cell is used to specify flags. -- interrupt-controller: Marks the device node as a interrupt controller. - drive-open-drain: Sets the ODR flag in the IOCON register. This configures the IRQ output as open drain active low. @@ -72,8 +68,6 @@ gpiom1: gpio@20 { interrupt-parent = <&gpio1>; interrupts = <17 IRQ_TYPE_LEVEL_LOW>; - interrupt-controller; - #interrupt-cells=<2>; microchip,irq-mirror; }; @@ -130,8 +124,6 @@ gpio21: gpio@21 { interrupt-parent = <&socgpio>; interrupts = <0x17 0x8>; interrupt-names = "mcp23017@21 irq"; - interrupt-controller; - #interrupt-cells = <0x2>; microchip,irq-mirror; pinctrl-names = "default"; pinctrl-0 = <&i2cgpio0irq &gpio21pullups>; -- 2.26.2