Commit 05b0852e authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Shawn Guo
Browse files

ARM: dts: imx6qdl-gw5xxx: correct interrupt flags 



GPIO_ACTIVE_x flags are not correct in the context of interrupt flags.
These are simple defines so they could be used in DTS but they will not
have the same meaning:
1. GPIO_ACTIVE_HIGH = 0 = IRQ_TYPE_NONE
2. GPIO_ACTIVE_LOW  = 1 = IRQ_TYPE_EDGE_RISING

Correct the interrupt flags, assuming the author of the code wanted same
logical behavior behind the name "ACTIVE_xxx", this is:
  ACTIVE_LOW  => IRQ_TYPE_LEVEL_LOW

Signed-off-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Acked-By: default avatarTim Harvey <tharvey@gateworks.com>
Signed-off-by: default avatarShawn Guo <shawnguo@kernel.org>
parent 55df2079

arch/arm/boot/dts/imx6qdl-gw51xx.dtsi

+2 −1
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ 


#include <dt-bindings/gpio/gpio.h>

#include <dt-bindings/input/linux-event-codes.h>

#include <dt-bindings/interrupt-controller/irq.h>



/ {

	/* these are used by bootloader for disabling nodes */
@@ -154,7 +155,7 @@ 
		compatible = "gw,gsc";

		reg = <0x20>;

		interrupt-parent = <&gpio1>;

		interrupts = <4 GPIO_ACTIVE_LOW>;

		interrupts = <4 IRQ_TYPE_LEVEL_LOW>;

		interrupt-controller;

		#interrupt-cells = <1>;

		#size-cells = <0>;

arch/arm/boot/dts/imx6qdl-gw52xx.dtsi

+2 −1
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ 


#include <dt-bindings/gpio/gpio.h>

#include <dt-bindings/input/linux-event-codes.h>

#include <dt-bindings/interrupt-controller/irq.h>



/ {

	/* these are used by bootloader for disabling nodes */
@@ -217,7 +218,7 @@ 
		compatible = "gw,gsc";

		reg = <0x20>;

		interrupt-parent = <&gpio1>;

		interrupts = <4 GPIO_ACTIVE_LOW>;

		interrupts = <4 IRQ_TYPE_LEVEL_LOW>;

		interrupt-controller;

		#interrupt-cells = <1>;

		#size-cells = <0>;

arch/arm/boot/dts/imx6qdl-gw53xx.dtsi

+2 −1
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ 


#include <dt-bindings/gpio/gpio.h>

#include <dt-bindings/input/linux-event-codes.h>

#include <dt-bindings/interrupt-controller/irq.h>



/ {

	/* these are used by bootloader for disabling nodes */
@@ -210,7 +211,7 @@ 
		compatible = "gw,gsc";

		reg = <0x20>;

		interrupt-parent = <&gpio1>;

		interrupts = <4 GPIO_ACTIVE_LOW>;

		interrupts = <4 IRQ_TYPE_LEVEL_LOW>;

		interrupt-controller;

		#interrupt-cells = <1>;

		#size-cells = <0>;

arch/arm/boot/dts/imx6qdl-gw54xx.dtsi

+2 −1
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ 


#include <dt-bindings/gpio/gpio.h>

#include <dt-bindings/input/linux-event-codes.h>

#include <dt-bindings/interrupt-controller/irq.h>

#include <dt-bindings/sound/fsl-imx-audmux.h>



/ {
@@ -247,7 +248,7 @@ 
		compatible = "gw,gsc";

		reg = <0x20>;

		interrupt-parent = <&gpio1>;

		interrupts = <4 GPIO_ACTIVE_LOW>;

		interrupts = <4 IRQ_TYPE_LEVEL_LOW>;

		interrupt-controller;

		#interrupt-cells = <1>;

		#address-cells = <1>;

arch/arm/boot/dts/imx6qdl-gw551x.dtsi

+2 −1
Original line number Diff line number Diff line
@@ -48,6 +48,7 @@ 
#include <dt-bindings/gpio/gpio.h>

#include <dt-bindings/media/tda1997x.h>

#include <dt-bindings/input/linux-event-codes.h>

#include <dt-bindings/interrupt-controller/irq.h>

#include <dt-bindings/sound/fsl-imx-audmux.h>



/ {
@@ -219,7 +220,7 @@ 
		compatible = "gw,gsc";

		reg = <0x20>;

		interrupt-parent = <&gpio1>;

		interrupts = <4 GPIO_ACTIVE_LOW>;

		interrupts = <4 IRQ_TYPE_LEVEL_LOW>;

		interrupt-controller;

		#interrupt-cells = <1>;

		#size-cells = <0>;

CRA Git | Maintained and supported by SUSTech CRA and CCSE