Hi,

On 10-07-15 16:30, Hans de Goede wrote:
Hi,

On 10-07-15 15:16, Samuel Egli wrote:
From: Bin Liu <b-...@ti.com>

Do not config MUSB to highspeed mode if CONFIG_USB_GADGET_DUALSPEED
is not set, in which case Ether gadget only operates in fullspeed.

Note:
This patch is necessary for devices like some siemens boards
that allow only FULL SPEED USB 1.1, e.g. DFU download.

Signed-off-by: Bin Liu <b-...@ti.com>
Reviewed-by: Tom Rini <tr...@konsulko.com>
Tested-by: Samuel Egli <samuel.e...@siemens.com>
CC: Marek Vasut <ma...@denx.de>
CC: Heiko Schocher <h...@denx.de>
CC: Daniel Mack <zon...@gmail.com>
CC: Roger Meier <r.me...@siemens.com>

Nack this breaks highspeed mode on boards which use the musb
in host mode, and thus do not set CONFIG_USB_GADGET_DUALSPEED.

p.s.

Given that you want to use this as a hack to work around the broken
pcb design of your board I suggest adding a new option for this
titled: CONFIG_USB_MUSB_NO_HIGHSPEED and then do:

+#ifndef CONFIG_USB_MUSB_NO_HIGHSPEED
                           | MUSB_POWER_HSENAB
+#endif

Using CONFIG_USB_GADGET_DUALSPEED for this seems wrong, since this
has nothing to do with enabling dualspeed mode for the gadget code
really.

Regards,

Hans



Regards,

Hans


---
  drivers/usb/musb-new/musb_core.c |    2 ++
  1 file changed, 2 insertions(+)

diff --git a/drivers/usb/musb-new/musb_core.c b/drivers/usb/musb-new/musb_core.c
index 242cc30..4edd6d7 100644
--- a/drivers/usb/musb-new/musb_core.c
+++ b/drivers/usb/musb-new/musb_core.c
@@ -942,7 +942,9 @@ void musb_start(struct musb *musb)

      /* put into basic highspeed mode and start session */
      musb_writeb(regs, MUSB_POWER, MUSB_POWER_ISOUPDATE
+#ifdef CONFIG_USB_GADGET_DUALSPEED
                          | MUSB_POWER_HSENAB
+#endif
                          /* ENSUSPEND wedges tusb */
                          /* | MUSB_POWER_ENSUSPEND */
                          );
--
1.7.10.4

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to