diff mbox

[U-Boot,1/2] exynos5250: usb: Fix VBus gpio numbers for ehci and xhci controllers

Message ID 1388745653-7069-1-git-send-email-gautam.vivek@samsung.com
State Accepted
Delegated to: Minkyu Kang
Headers show

Commit Message

Vivek Gautam Jan. 3, 2014, 10:40 a.m. UTC
The gpio_*() apis require the exact gpio line number to deduce
the gpio bank and the gpio pin addresses.
So fix the gpio number for VBUS used for EHCI ports as well as
XHCI ports on exynos5250 boards.

Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
Cc: Julius Werner <jwerner@chromium.org>
Cc: Minkyu Kang <mk7.kang@samsung.com>
Cc: Marek Vasut <marex@denx.de>
---
 board/samsung/dts/exynos5250-smdk5250.dts |    2 +-
 board/samsung/dts/exynos5250-snow.dts     |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Comments

Vivek Gautam Jan. 3, 2014, 10:39 a.m. UTC | #1
On Fri, Jan 3, 2014 at 4:10 PM, Vivek Gautam <gautam.vivek@samsung.com> wrote:
> The gpio_*() apis require the exact gpio line number to deduce
> the gpio bank and the gpio pin addresses.
> So fix the gpio number for VBUS used for EHCI ports as well as
> XHCI ports on exynos5250 boards.
>
> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
> Cc: Julius Werner <jwerner@chromium.org>
> Cc: Minkyu Kang <mk7.kang@samsung.com>
> Cc: Marek Vasut <marex@denx.de>
> ---

Based on u-boot-samsung 'master' branch.
Tested on smkd5250 and snow board for EHCI and XHCI ports.

>  board/samsung/dts/exynos5250-smdk5250.dts |    2 +-
>  board/samsung/dts/exynos5250-snow.dts     |    4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/board/samsung/dts/exynos5250-smdk5250.dts b/board/samsung/dts/exynos5250-smdk5250.dts
> index c4ed346..9020382 100644
> --- a/board/samsung/dts/exynos5250-smdk5250.dts
> +++ b/board/samsung/dts/exynos5250-smdk5250.dts
> @@ -146,6 +146,6 @@
>         };
>
>         ehci@12110000 {
> -               samsung,vbus-gpio = <&gpio 0xbe 0>; /* X26 */
> +               samsung,vbus-gpio = <&gpio 0x316 0>; /* X26 */
>         };
>  };
> diff --git a/board/samsung/dts/exynos5250-snow.dts b/board/samsung/dts/exynos5250-snow.dts
> index 091cdb9..9b48a0c 100644
> --- a/board/samsung/dts/exynos5250-snow.dts
> +++ b/board/samsung/dts/exynos5250-snow.dts
> @@ -110,11 +110,11 @@
>         };
>
>         ehci@12110000 {
> -               samsung,vbus-gpio = <&gpio 0xb1 0>; /* X11 */
> +               samsung,vbus-gpio = <&gpio 0x309 0>; /* X11 */
>         };
>
>         xhci@12000000 {
> -               samsung,vbus-gpio = <&gpio 0xbf 0>; /* X27 */
> +               samsung,vbus-gpio = <&gpio 0x317 0>; /* X27 */
>         };
>
>         tmu@10060000 {
> --
> 1.7.10.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Marek Vasut Jan. 4, 2014, 7:14 a.m. UTC | #2
On Friday, January 03, 2014 at 11:40:52 AM, Vivek Gautam wrote:
> The gpio_*() apis require the exact gpio line number to deduce
> the gpio bank and the gpio pin addresses.
> So fix the gpio number for VBUS used for EHCI ports as well as
> XHCI ports on exynos5250 boards.
> 
> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
> Cc: Julius Werner <jwerner@chromium.org>
> Cc: Minkyu Kang <mk7.kang@samsung.com>
> Cc: Marek Vasut <marex@denx.de>

Reiewed-by: Marek Vasut <marex@denx.de>

Minkyu, can you please pick this?

Best regards,
Marek Vasut
Minkyu Kang Jan. 6, 2014, 1:43 a.m. UTC | #3
On 04/01/14 16:14, Marek Vasut wrote:
> On Friday, January 03, 2014 at 11:40:52 AM, Vivek Gautam wrote:
>> The gpio_*() apis require the exact gpio line number to deduce
>> the gpio bank and the gpio pin addresses.
>> So fix the gpio number for VBUS used for EHCI ports as well as
>> XHCI ports on exynos5250 boards.
>>
>> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
>> Cc: Julius Werner <jwerner@chromium.org>
>> Cc: Minkyu Kang <mk7.kang@samsung.com>
>> Cc: Marek Vasut <marex@denx.de>
> 
> Reiewed-by: Marek Vasut <marex@denx.de>
> 
> Minkyu, can you please pick this?

OK.

> 
> Best regards,
> Marek Vasut
> 

Thanks,
Minkyu Kang.
Minkyu Kang Feb. 3, 2014, 4:45 a.m. UTC | #4
On 03/01/14 19:40, Vivek Gautam wrote:
> The gpio_*() apis require the exact gpio line number to deduce
> the gpio bank and the gpio pin addresses.
> So fix the gpio number for VBUS used for EHCI ports as well as
> XHCI ports on exynos5250 boards.
> 
> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
> Cc: Julius Werner <jwerner@chromium.org>
> Cc: Minkyu Kang <mk7.kang@samsung.com>
> Cc: Marek Vasut <marex@denx.de>
> ---
>  board/samsung/dts/exynos5250-smdk5250.dts |    2 +-
>  board/samsung/dts/exynos5250-snow.dts     |    4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 

applied to u-boot-samsung.

Thanks,
Minkyu Kang.
diff mbox

Patch

diff --git a/board/samsung/dts/exynos5250-smdk5250.dts b/board/samsung/dts/exynos5250-smdk5250.dts
index c4ed346..9020382 100644
--- a/board/samsung/dts/exynos5250-smdk5250.dts
+++ b/board/samsung/dts/exynos5250-smdk5250.dts
@@ -146,6 +146,6 @@ 
 	};
 
 	ehci@12110000 {
-		samsung,vbus-gpio = <&gpio 0xbe 0>; /* X26 */
+		samsung,vbus-gpio = <&gpio 0x316 0>; /* X26 */
 	};
 };
diff --git a/board/samsung/dts/exynos5250-snow.dts b/board/samsung/dts/exynos5250-snow.dts
index 091cdb9..9b48a0c 100644
--- a/board/samsung/dts/exynos5250-snow.dts
+++ b/board/samsung/dts/exynos5250-snow.dts
@@ -110,11 +110,11 @@ 
 	};
 
 	ehci@12110000 {
-		samsung,vbus-gpio = <&gpio 0xb1 0>; /* X11 */
+		samsung,vbus-gpio = <&gpio 0x309 0>; /* X11 */
 	};
 
 	xhci@12000000 {
-		samsung,vbus-gpio = <&gpio 0xbf 0>; /* X27 */
+		samsung,vbus-gpio = <&gpio 0x317 0>; /* X27 */
 	};
 
 	tmu@10060000 {