Hi Heiko,

> This patchserie adds the new config option CONFIG_DFU_FULLSPEED.

Is there any special reason to support Full Speed (12 Mbit/sec - USB
1.1) and not rely solely on the High Speed (USB 2.0) as we do now?

> If this is enabled DFU uses fullspeed only. This is used on the
> siemens boards.

Is there any DFU problem with the mentioned Siemens board, that we must
use USB 1.1?

I'd also appreciate more verbose rationale for this patch series - why
this change is needed? What is the expected improvement?

Thanks in advance.

> 
> Cc: Tom Rini <tr...@ti.com>
> Cc: Lukasz Majewski <l.majew...@samsung.com>
> Cc: Marek Vasut <ma...@denx.de>
> Cc: Liu Bin <b-...@ti.com>
> Cc: Lukas Stockmann <lukas.stockm...@siemens.com>
> 
> Heiko Schocher (2):
>   usb: dfu: add config option to use in dfu mode fullspeed only
>   arm: am335x: siemens board use in DFU mode fullspeed only
> 
>  README                                 | 3 +++
>  drivers/usb/gadget/f_dfu.c             | 3 +++
>  drivers/usb/gadget/g_dnl.c             | 3 +++
>  include/configs/siemens-am33x-common.h | 3 ++-
>  4 files changed, 11 insertions(+), 1 deletion(-)
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to