Re: [PATCH 1/4] ARM: dts: imx6: RDU2: Use new CODEC reset pin name

[Shawn Guo]

On Fri, Mar 29, 2019 at 01:13:10PM -0500, Andrew F. Davis wrote:
> The correct DT property for specifying a GPIO used for reset
> is "reset-gpios", the driver now accepts this name, use it here.
> 
> Note the GPIO polarity in the driver was ignored before and always
> assumed to be active low, when all the DTs are fixed we will start
> respecting the specified polarity. Switch polarity in DT to the
> currently assumed one, this way when the driver changes the
> behavior will not change.
> 
> Signed-off-by: Andrew F. Davis 

I fixed up the prefix to use board name, and applied patch #1 ~ #3.

Shawn

[PATCH 1/4] ARM: dts: imx6: RDU2: Use new CODEC reset pin name

[Andrew F. Davis]

The correct DT property for specifying a GPIO used for reset
is "reset-gpios", the driver now accepts this name, use it here.

Note the GPIO polarity in the driver was ignored before and always
assumed to be active low, when all the DTs are fixed we will start
respecting the specified polarity. Switch polarity in DT to the
currently assumed one, this way when the driver changes the
behavior will not change.

Signed-off-by: Andrew F. Davis 
---
 arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi 
b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
index 69942c7ff89d..cbf7c9cae565 100644
--- a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
@@ -384,7 +384,7 @@
AVDD-supply = <_3p3v>;
IOVDD-supply = <_3p3v>;
DVDD-supply = <_reg>;
-   gpio-reset = < 2 GPIO_ACTIVE_HIGH>;
+   reset-gpios = < 2 GPIO_ACTIVE_LOW>;
};
 
accel@1c {
@@ -572,7 +572,7 @@
AVDD-supply = <_3p3v>;
IOVDD-supply = <_3p3v>;
DVDD-supply = <_reg>;
-   gpio-reset = < 0 GPIO_ACTIVE_HIGH>;
+   reset-gpios = < 0 GPIO_ACTIVE_LOW>;
};
 
touchscreen@20 {
-- 
2.21.0