arm64: dts: ten64: remove redundant interrupt declaration for gpio-keys
authorMathew McBride <matt@traverse.com.au>
Mon, 22 Nov 2021 02:55:54 +0000 (02:55 +0000)
committerShawn Guo <shawnguo@kernel.org>
Tue, 23 Nov 2021 12:25:34 +0000 (20:25 +0800)
gpio-keys already 'inherits' the interrupts from the controller
of the specified GPIO, so having another declaration is redundant.
On >=v5.15 this started causing an oops under gpio_keys_probe as
the IRQ was already claimed.

Signed-off-by: Mathew McBride <matt@traverse.com.au>
Fixes: 418962eea358 ("arm64: dts: add device tree for Traverse Ten64 (LS1088A)")
Cc: stable@vger.kernel.org
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts

index 3063851..d3f03dc 100644 (file)
@@ -38,7 +38,6 @@
                powerdn {
                        label = "External Power Down";
                        gpios = <&gpio1 17 GPIO_ACTIVE_LOW>;
-                       interrupts = <&gpio1 17 IRQ_TYPE_EDGE_FALLING>;
                        linux,code = <KEY_POWER>;
                };
 
@@ -46,7 +45,6 @@
                admin {
                        label = "ADMIN button";
                        gpios = <&gpio3 8 GPIO_ACTIVE_HIGH>;
-                       interrupts = <&gpio3 8 IRQ_TYPE_EDGE_RISING>;
                        linux,code = <KEY_WPS_BUTTON>;
                };
        };