arm: mxs: dts: small fixes for I2SE's duckbills

This series fixes two issues with the DT file: first, it frees up some GPIOs which are mis-used by MMC interface, and second, it fixes the GPIO used to reset the ethernet phy.

While at it, use symbolic names for LED definitions.

Changes in v2:
- add missing patch for imx28.dtsi to include gpio.h

Michael Heimpold (4):
arm: mxs: dt: imx28-duckbill: fix mmc settings
arm: mxs: dt: imx28-duckbill: fix phy reset gpio
arm: mxs: dt: imx28: include gpio.h to allow use of symbolic names
arm: mxs: dt: imx28-duckbill: use symbolic names from gpio.h

 arch/arm/boot/dts/imx28-duckbill.dts |   12 ++++++------
 arch/arm/boot/dts/imx28.dtsi         |    1 +
 2 files changed, 7 insertions(+), 6 deletions(-)

1.7.10.4

I2SE's duckbills are only equipped with a micro SD card slot and thus only provide a 4-bit interface.

---
Changes since v1:
- none

 arch/arm/boot/dts/imx28-duckbill.dts |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

index 5f326c1..a697870 100644
@@ -25,9 +25,9 @@
 			ssp0: ssp at 80010000 {
 				compatible = "fsl,imx28-mmc";
 				pinctrl-names = "default";
-				pinctrl-0 = <&mmc0_8bit_pins_a
+				pinctrl-0 = <&mmc0_4bit_pins_a
 					&mmc0_cd_cfg &mmc0_sck_cfg>;
-				bus-width = <8>;
+				bus-width = <4>;
 				vmmc-supply = <&reg_3p3v>;
 				status = "okay";
 			};

1.7.10.4

Fix a copy & paste error: on duckbills the GPIO used for resetting the ethernet phy differs from FSL's MX28EVK board.

---
Changes since v1:
- none

 arch/arm/boot/dts/imx28-duckbill.dts |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

index a697870..91c1a9a 100644
@@ -39,7 +39,7 @@
 				hog_pins_a: hog at 0 {
 					reg = <0>;
 					fsl,pinmux-ids = <
-						MX28_PAD_ENET0_RX_CLK__GPIO_4_13 /* PHY Reset */
+						MX28_PAD_SSP0_DATA7__GPIO_2_7 /* PHY Reset */
 					>;
 					fsl,drive-strength = <MXS_DRIVE_4mA>;
 					fsl,voltage = <MXS_VOLTAGE_HIGH>;

@@ -82,7 +82,7 @@
 			pinctrl-names = "default";
 			pinctrl-0 = <&mac0_pins_a>;
 			phy-supply = <&reg_3p3v>;
-			phy-reset-gpios = <&gpio4 13 0>;
+			phy-reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>;
 			phy-reset-duration = <100>;
 			status = "okay";
 		};

1.7.10.4

---
Changes since v1:
- new patch

 arch/arm/boot/dts/imx28.dtsi |    1 +
 1 file changed, 1 insertion(+)

index 90a5795..a95cc53 100644
@@ -9,6 +9,7 @@
  * http://www.gnu.org/copyleft/gpl.html
  */
 
+#include <dt-bindings/gpio/gpio.h>
 #include "skeleton.dtsi"
 #include "imx28-pinfunc.h"
 

1.7.10.4

Use GPIO_ACTIVE_HIGH and GPIO_ACTIVE_LOW instead of 0 and 1.

---
Changes since v1:
- none

 arch/arm/boot/dts/imx28-duckbill.dts |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

index 91c1a9a..ce1a7ef 100644
@@ -110,12 +110,12 @@
 
 		status {
 			label = "duckbill:green:status";
-			gpios = <&gpio3 5 0>;
+			gpios = <&gpio3 5 GPIO_ACTIVE_HIGH>;
 		};
 
 		failure {
 			label = "duckbill:red:status";
-			gpios = <&gpio3 4 0>;
+			gpios = <&gpio3 4 GPIO_ACTIVE_HIGH>;
 		};
 	};
 };

1.7.10.4

Am 27.03.2014 23:51, schrieb Michael Heimpold:

They all look good to me.

BR Stefan Wahren

Changed patch prefix as below and applied all patches.

ARM: dts: imx28-duckbill: fix mmc settings
ARM: dts: imx28-duckbill: fix phy reset gpio
ARM: dts: imx28: include gpio.h to allow use of symbolic names
ARM: dts: imx28-duckbill: use symbolic names from gpio.h

Shawn

Forgot mentioning that the patches should be reordered as below, since the patch fixing phy reset gpio already uses GPIO symbolic name.

ARM: dts: imx28-duckbill: fix mmc settings
ARM: dts: imx28: include gpio.h to allow use of symbolic names
ARM: dts: imx28-duckbill: fix phy reset gpio
ARM: dts: imx28-duckbill: use symbolic names from gpio.h

Shawn