diff mbox series

[3/4] ARM: dts: imx6qdl-phytec-pfla02: Replace magic numbers

Message ID 20181223163141.6792-3-shc_work@mail.ru (mailing list archive)
State New, archived
Headers show
Series [1/4] ARM: dts: imx6q-phytec-pbab01: Remove duplicate "chosen" node | expand

Commit Message

Alexander Shiyan Dec. 23, 2018, 4:31 p.m. UTC
Use GPIO_ACTIVE_LOW/HIGH for GPIO description.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Shawn Guo Jan. 12, 2019, 3:13 a.m. UTC | #1
On Sun, Dec 23, 2018 at 07:31:40PM +0300, Alexander Shiyan wrote:
> Use GPIO_ACTIVE_LOW/HIGH for GPIO description.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>  arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
> index 211640e..81007d6 100644
> --- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
> @@ -23,7 +23,7 @@
>  			regulator-name = "usb_otg_vbus";
>  			regulator-min-microvolt = <5000000>;
>  			regulator-max-microvolt = <5000000>;
> -			gpio = <&gpio4 15 0>;
> +			gpio = <&gpio4 15 GPIO_ACTIVE_HIGH>;
>  			enable-active-high;
>  		};
>  
> @@ -33,7 +33,7 @@
>  			regulator-name = "usb_h1_vbus";
>  			regulator-min-microvolt = <5000000>;
>  			regulator-max-microvolt = <5000000>;
> -			gpio = <&gpio1 0 0>;
> +			gpio = <&gpio1 0 GPIO_ACTIVE_HIGH>;
>  			enable-active-high;
>  		};
>  	};
> @@ -43,12 +43,12 @@
>  
>  		green {
>  			label = "phyflex:green";
> -			gpios = <&gpio1 30 0>;
> +			gpios = <&gpio1 30 GPIO_ACTIVE_HIGH>;
>  		};
>  
>  		red {
>  			label = "phyflex:red";
> -			gpios = <&gpio2 31 0>;
> +			gpios = <&gpio2 31 GPIO_ACTIVE_HIGH>;
>  		};
>  	};
>  };
> @@ -69,7 +69,7 @@
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_ecspi3>;
>  	status = "okay";
> -	cs-gpios = <&gpio4 24 0>;
> +	cs-gpios = <&gpio4 24 GPIO_ACTIVE_LOW>;

GPIO_ACTIVE_LOW is defined as 1 rather than 0.

Shawn

>  
>  	flash@0 {
>  		compatible = "m25p80", "jedec,spi-nor";
> -- 
> 2.10.2
>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
index 211640e..81007d6 100644
--- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
@@ -23,7 +23,7 @@ 
 			regulator-name = "usb_otg_vbus";
 			regulator-min-microvolt = <5000000>;
 			regulator-max-microvolt = <5000000>;
-			gpio = <&gpio4 15 0>;
+			gpio = <&gpio4 15 GPIO_ACTIVE_HIGH>;
 			enable-active-high;
 		};
 
@@ -33,7 +33,7 @@ 
 			regulator-name = "usb_h1_vbus";
 			regulator-min-microvolt = <5000000>;
 			regulator-max-microvolt = <5000000>;
-			gpio = <&gpio1 0 0>;
+			gpio = <&gpio1 0 GPIO_ACTIVE_HIGH>;
 			enable-active-high;
 		};
 	};
@@ -43,12 +43,12 @@ 
 
 		green {
 			label = "phyflex:green";
-			gpios = <&gpio1 30 0>;
+			gpios = <&gpio1 30 GPIO_ACTIVE_HIGH>;
 		};
 
 		red {
 			label = "phyflex:red";
-			gpios = <&gpio2 31 0>;
+			gpios = <&gpio2 31 GPIO_ACTIVE_HIGH>;
 		};
 	};
 };
@@ -69,7 +69,7 @@ 
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_ecspi3>;
 	status = "okay";
-	cs-gpios = <&gpio4 24 0>;
+	cs-gpios = <&gpio4 24 GPIO_ACTIVE_LOW>;
 
 	flash@0 {
 		compatible = "m25p80", "jedec,spi-nor";