* [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0
2015-04-14 20:37 [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0 Stefan Wahren
@ 2015-04-14 20:43 ` Fabio Estevam
2015-04-15 2:59 ` Marek Vasut
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Fabio Estevam @ 2015-04-14 20:43 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 14, 2015 at 5:37 PM, Stefan Wahren <stefan.wahren@i2se.com> wrote:
> The dr_mode of usb0 on imx233-olinuxino is left to default "otg".
> Since the green LED (GPIO2_1) on imx233-olinuxino is connected to the
> same pin as USB_OTG_ID it's possible to disable USB host by LED toggling:
>
> echo 0 > /sys/class/leds/green/brightness
> [ 1068.890000] ci_hdrc ci_hdrc.0: remove, state 1
> [ 1068.890000] usb usb1: USB disconnect, device number 1
> [ 1068.920000] usb 1-1: USB disconnect, device number 2
> [ 1068.920000] usb 1-1.1: USB disconnect, device number 3
> [ 1069.070000] usb 1-1.2: USB disconnect, device number 4
> [ 1069.450000] ci_hdrc ci_hdrc.0: USB bus 1 deregistered
> [ 1074.460000] ci_hdrc ci_hdrc.0: timeout waiting for 00000800 in 11
>
> This patch fixes the issue by setting dr_mode to "host" in the dts file.
>
> Reported-by: Harald Geyer <harald@ccbib.org>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
Thanks
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0
2015-04-14 20:37 [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0 Stefan Wahren
2015-04-14 20:43 ` Fabio Estevam
@ 2015-04-15 2:59 ` Marek Vasut
2015-04-15 3:32 ` Peter Chen
2015-04-27 12:26 ` Shawn Guo
3 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2015-04-15 2:59 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday, April 14, 2015 at 10:37:26 PM, Stefan Wahren wrote:
> The dr_mode of usb0 on imx233-olinuxino is left to default "otg".
> Since the green LED (GPIO2_1) on imx233-olinuxino is connected to the
> same pin as USB_OTG_ID it's possible to disable USB host by LED toggling:
>
> echo 0 > /sys/class/leds/green/brightness
> [ 1068.890000] ci_hdrc ci_hdrc.0: remove, state 1
> [ 1068.890000] usb usb1: USB disconnect, device number 1
> [ 1068.920000] usb 1-1: USB disconnect, device number 2
> [ 1068.920000] usb 1-1.1: USB disconnect, device number 3
> [ 1069.070000] usb 1-1.2: USB disconnect, device number 4
> [ 1069.450000] ci_hdrc ci_hdrc.0: USB bus 1 deregistered
> [ 1074.460000] ci_hdrc ci_hdrc.0: timeout waiting for 00000800 in 11
>
> This patch fixes the issue by setting dr_mode to "host" in the dts file.
>
> Reported-by: Harald Geyer <harald@ccbib.org>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Marek Vasut <marex@denx.de>
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0
2015-04-14 20:37 [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0 Stefan Wahren
2015-04-14 20:43 ` Fabio Estevam
2015-04-15 2:59 ` Marek Vasut
@ 2015-04-15 3:32 ` Peter Chen
2015-04-15 19:11 ` Stefan Wahren
2015-04-27 12:26 ` Shawn Guo
3 siblings, 1 reply; 6+ messages in thread
From: Peter Chen @ 2015-04-15 3:32 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 14, 2015 at 08:37:26PM +0000, Stefan Wahren wrote:
> The dr_mode of usb0 on imx233-olinuxino is left to default "otg".
> Since the green LED (GPIO2_1) on imx233-olinuxino is connected to the
> same pin as USB_OTG_ID it's possible to disable USB host by LED toggling:
>
> echo 0 > /sys/class/leds/green/brightness
> [ 1068.890000] ci_hdrc ci_hdrc.0: remove, state 1
> [ 1068.890000] usb usb1: USB disconnect, device number 1
> [ 1068.920000] usb 1-1: USB disconnect, device number 2
> [ 1068.920000] usb 1-1.1: USB disconnect, device number 3
> [ 1069.070000] usb 1-1.2: USB disconnect, device number 4
> [ 1069.450000] ci_hdrc ci_hdrc.0: USB bus 1 deregistered
> [ 1074.460000] ci_hdrc ci_hdrc.0: timeout waiting for 00000800 in 11
>
> This patch fixes the issue by setting dr_mode to "host" in the dts file.
>
> Reported-by: Harald Geyer <harald@ccbib.org>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> ---
> arch/arm/boot/dts/imx23-olinuxino.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/imx23-olinuxino.dts b/arch/arm/boot/dts/imx23-olinuxino.dts
> index 7e6eef2..3b2d74d 100644
> --- a/arch/arm/boot/dts/imx23-olinuxino.dts
> +++ b/arch/arm/boot/dts/imx23-olinuxino.dts
> @@ -93,6 +93,7 @@
>
> ahb at 80080000 {
> usb0: usb at 80080000 {
> + dr_mode = "host";
> vbus-supply = <®_usb0_vbus>;
> status = "okay";
> };
> --
> 1.7.9.5
>
If this board has only host function, I am ok for it.
Acked-by: Peter Chen <peter.chen@freescale.com>
--
Best Regards,
Peter Chen
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0
2015-04-14 20:37 [PATCH] ARM: dts: imx23-olinuxino: Fix dr_mode of usb0 Stefan Wahren
` (2 preceding siblings ...)
2015-04-15 3:32 ` Peter Chen
@ 2015-04-27 12:26 ` Shawn Guo
3 siblings, 0 replies; 6+ messages in thread
From: Shawn Guo @ 2015-04-27 12:26 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 14, 2015 at 08:37:26PM +0000, Stefan Wahren wrote:
> The dr_mode of usb0 on imx233-olinuxino is left to default "otg".
> Since the green LED (GPIO2_1) on imx233-olinuxino is connected to the
> same pin as USB_OTG_ID it's possible to disable USB host by LED toggling:
>
> echo 0 > /sys/class/leds/green/brightness
> [ 1068.890000] ci_hdrc ci_hdrc.0: remove, state 1
> [ 1068.890000] usb usb1: USB disconnect, device number 1
> [ 1068.920000] usb 1-1: USB disconnect, device number 2
> [ 1068.920000] usb 1-1.1: USB disconnect, device number 3
> [ 1069.070000] usb 1-1.2: USB disconnect, device number 4
> [ 1069.450000] ci_hdrc ci_hdrc.0: USB bus 1 deregistered
> [ 1074.460000] ci_hdrc ci_hdrc.0: timeout waiting for 00000800 in 11
>
> This patch fixes the issue by setting dr_mode to "host" in the dts file.
>
> Reported-by: Harald Geyer <harald@ccbib.org>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread