diff mbox

[U-Boot] mx6qsabrelite: No need to set the direction for GPIO3_23 again

Message ID 1334175744-26707-1-git-send-email-festevam@gmail.com
State Accepted
Commit bdb9f7604ddaffe623267df5c381353ac23a90a6
Headers show

Commit Message

Fabio Estevam April 11, 2012, 8:22 p.m. UTC
There is a 'gpio_direction_output(87, 0);' call previously, so the GPIO direction is 
already established.

Use gpio_set_value() for changing the GPIO output then.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 board/freescale/mx6qsabrelite/mx6qsabrelite.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Dirk Behme April 12, 2012, 5:53 p.m. UTC | #1
On 11.04.2012 22:22, Fabio Estevam wrote:
> There is a 'gpio_direction_output(87, 0);' call previously, so the GPIO direction is
> already established.
>
> Use gpio_set_value() for changing the GPIO output then.
>
> Signed-off-by: Fabio Estevam<fabio.estevam@freescale.com>
> ---
>   board/freescale/mx6qsabrelite/mx6qsabrelite.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/board/freescale/mx6qsabrelite/mx6qsabrelite.c b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
> index 1d09a72..2155d87 100644
> --- a/board/freescale/mx6qsabrelite/mx6qsabrelite.c
> +++ b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
> @@ -135,7 +135,7 @@ static void setup_iomux_enet(void)
>
>   	/* Need delay 10ms according to KSZ9021 spec */
>   	udelay(1000 * 10);
> -	gpio_direction_output(87, 1);  /* GPIO 3-23 */
> +	gpio_set_value(87, 1);  /* GPIO 3-23 */
>
>   	imx_iomux_v3_setup_multiple_pads(enet_pads2, ARRAY_SIZE(enet_pads2));
>   }

Acked-by: Dirk Behme <dirk.behme@googlemail.com>

Thanks

Dirk
Stefano Babic April 23, 2012, 9:48 a.m. UTC | #2
On 11/04/2012 22:22, Fabio Estevam wrote:
> There is a 'gpio_direction_output(87, 0);' call previously, so the GPIO direction is 
> already established.
> 
> Use gpio_set_value() for changing the GPIO output then.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---


Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic
diff mbox

Patch

diff --git a/board/freescale/mx6qsabrelite/mx6qsabrelite.c b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
index 1d09a72..2155d87 100644
--- a/board/freescale/mx6qsabrelite/mx6qsabrelite.c
+++ b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
@@ -135,7 +135,7 @@  static void setup_iomux_enet(void)
 
 	/* Need delay 10ms according to KSZ9021 spec */
 	udelay(1000 * 10);
-	gpio_direction_output(87, 1);  /* GPIO 3-23 */
+	gpio_set_value(87, 1);  /* GPIO 3-23 */
 
 	imx_iomux_v3_setup_multiple_pads(enet_pads2, ARRAY_SIZE(enet_pads2));
 }