diff mbox

[U-Boot] tegra: colibri_t30: asix usb ethernet reset regression

Message ID 610941893d227d351b0f6156e6c29ca9c57becc2.1412953433.git.marcel@ziswiler.com
State Accepted
Delegated to: Tom Warren
Headers show

Commit Message

Marcel Ziswiler Oct. 10, 2014, 3:04 p.m. UTC
Fix ASIX USB to Ethernet reset which due to the new driver model Tegra
GPIO driver changes now requires a label string to be provided
otherwise the reservation and subsequent direction/value calls will
fail.

This fixes a regression introduced by commit:

2fccd2d96badcdf6165658a99771a4c475586279
tegra: Convert tegra GPIO driver to use driver model

Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>
---
 board/toradex/colibri_t30/colibri_t30.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass Oct. 10, 2014, 3:19 p.m. UTC | #1
On 10 October 2014 09:04, Marcel Ziswiler <marcel@ziswiler.com> wrote:
> Fix ASIX USB to Ethernet reset which due to the new driver model Tegra
> GPIO driver changes now requires a label string to be provided
> otherwise the reservation and subsequent direction/value calls will
> fail.
>
> This fixes a regression introduced by commit:
>
> 2fccd2d96badcdf6165658a99771a4c475586279
> tegra: Convert tegra GPIO driver to use driver model
>
> Signed-off-by: Marcel Ziswiler <marcel@ziswiler.com>

This limitation is removed by some pending patches (moving the
request/free logic to the uclass). Even then this patch improves
things.

Acked-by: Simon Glass <sjg@chromium.org>

> ---
>  board/toradex/colibri_t30/colibri_t30.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/board/toradex/colibri_t30/colibri_t30.c b/board/toradex/colibri_t30/colibri_t30.c
> index ed043f4..f4bc7d8 100644
> --- a/board/toradex/colibri_t30/colibri_t30.c
> +++ b/board/toradex/colibri_t30/colibri_t30.c
> @@ -35,7 +35,7 @@ void pinmux_init(void)
>  void pin_mux_usb(void)
>  {
>         /* Reset ASIX using LAN_RESET */
> -       gpio_request(GPIO_PDD0, NULL);
> +       gpio_request(GPIO_PDD0, "LAN_RESET");
>         gpio_direction_output(GPIO_PDD0, 0);
>         udelay(5);
>         gpio_set_value(GPIO_PDD0, 1);
> --
> 1.9.3
>
diff mbox

Patch

diff --git a/board/toradex/colibri_t30/colibri_t30.c b/board/toradex/colibri_t30/colibri_t30.c
index ed043f4..f4bc7d8 100644
--- a/board/toradex/colibri_t30/colibri_t30.c
+++ b/board/toradex/colibri_t30/colibri_t30.c
@@ -35,7 +35,7 @@  void pinmux_init(void)
 void pin_mux_usb(void)
 {
 	/* Reset ASIX using LAN_RESET */
-	gpio_request(GPIO_PDD0, NULL);
+	gpio_request(GPIO_PDD0, "LAN_RESET");
 	gpio_direction_output(GPIO_PDD0, 0);
 	udelay(5);
 	gpio_set_value(GPIO_PDD0, 1);