All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4
@ 2017-07-06  6:54 Kunihiko Hayashi
  2017-07-06  7:09 ` Masahiro Yamada
  2017-07-31 13:50 ` Linus Walleij
  0 siblings, 2 replies; 3+ messages in thread
From: Kunihiko Hayashi @ 2017-07-06  6:54 UTC (permalink / raw)
  To: linux-gpio, yamada.masahiro; +Cc: Kunihiko Hayashi

According to pinctrl assignment for Pro4, each definition of USB#2 and
USB#3 are as follows.

  184: USB2VBUS
  185: USB2OD
  186: USB2ID
  187: USB3VBUS
  188: USB3OD

USB#2 has an additional pin "USB2ID", but the chip doesn't use this pin
while in host-mode. Considering this pin, the pin definitions for USB#3
should be {187, 188}.

Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
---
 drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c b/drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c
index a433a30..c75e094 100644
--- a/drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c
+++ b/drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c
@@ -1084,7 +1084,7 @@ static const unsigned usb1_pins[] = {182, 183};
 static const int usb1_muxvals[] = {0, 0};
 static const unsigned usb2_pins[] = {184, 185};
 static const int usb2_muxvals[] = {0, 0};
-static const unsigned usb3_pins[] = {186, 187};
+static const unsigned usb3_pins[] = {187, 188};
 static const int usb3_muxvals[] = {0, 0};
 static const unsigned port_range0_pins[] = {
 	300, 301, 302, 303, 304, 305, 306, 307,		/* PORT0x */
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4
  2017-07-06  6:54 [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4 Kunihiko Hayashi
@ 2017-07-06  7:09 ` Masahiro Yamada
  2017-07-31 13:50 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Masahiro Yamada @ 2017-07-06  7:09 UTC (permalink / raw)
  To: Kunihiko Hayashi; +Cc: linux-gpio, Linus Walleij

2017-07-06 15:54 GMT+09:00 Kunihiko Hayashi <hayashi.kunihiko@socionext.com>:
> According to pinctrl assignment for Pro4, each definition of USB#2 and
> USB#3 are as follows.
>
>   184: USB2VBUS
>   185: USB2OD
>   186: USB2ID
>   187: USB3VBUS
>   188: USB3OD
>
> USB#2 has an additional pin "USB2ID", but the chip doesn't use this pin
> while in host-mode. Considering this pin, the pin definitions for USB#3
> should be {187, 188}.
>
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>


Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com>



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4
  2017-07-06  6:54 [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4 Kunihiko Hayashi
  2017-07-06  7:09 ` Masahiro Yamada
@ 2017-07-31 13:50 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2017-07-31 13:50 UTC (permalink / raw)
  To: Kunihiko Hayashi; +Cc: linux-gpio, Masahiro Yamada

On Thu, Jul 6, 2017 at 8:54 AM, Kunihiko Hayashi
<hayashi.kunihiko@socionext.com> wrote:

> According to pinctrl assignment for Pro4, each definition of USB#2 and
> USB#3 are as follows.
>
>   184: USB2VBUS
>   185: USB2OD
>   186: USB2ID
>   187: USB3VBUS
>   188: USB3OD
>
> USB#2 has an additional pin "USB2ID", but the chip doesn't use this pin
> while in host-mode. Considering this pin, the pin definitions for USB#3
> should be {187, 188}.
>
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>

Patch applied for fixes with Masahiro's ACK.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-07-31 13:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-06  6:54 [PATCH] pinctrl: uniphier: fix USB3 pin assignment for Pro4 Kunihiko Hayashi
2017-07-06  7:09 ` Masahiro Yamada
2017-07-31 13:50 ` Linus Walleij

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.