Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
Hi all -Ursprüngliche Nachricht- Von: Heiko Schocher [mailto:h...@denx.de] Gesendet: Dienstag, 9. September 2014 16:22 An: Lukasz Majewski Cc: u-boot@lists.denx.de; Tom Rini; Marek Vasut; Liu Bin; Stockmann, Lukas Betreff: Re: [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only Hello Lukasz, Am 09.09.2014 15:43, schrieb Lukasz Majewski: 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? They have a USB HW protection circuit and therefore they only allowed to use fullspeed ... @Lukas Stockman: Maybe you could answer this better than me? Yes, it's because of a protection circuit. Our device is 24V AC powered, if someone uses a non galvanically isolated transformer and misswires it, it could burn down the USB host. That's why we have a protection on the USB side which limits the speed to 12Mbit/sec. Regards, Lukas ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
This patchserie adds the new config option CONFIG_DFU_FULLSPEED. If this is enabled DFU uses fullspeed only. This is used on the siemens boards. 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(-) -- 1.8.3.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
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 RD Institute Poland (SRPOL) | Linux Platform Group ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
Lukasz, On 09/09/2014 08:43 AM, Lukasz Majewski wrote: 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? The drivers must support full-speed anyway based on the USB2.0 Specs. I have seen many cases in which only full-speed is suitable due to either board design or specific USB use case. One example is that to better fit to the production firmware flashing automation, a board does not use standard USB receptacle or plug, but a four touch pads on the PCB, so that the robot can grab the board and touch the pads to download the firmware. In this case signal integrity is a concern, but not download speed. Regards, -Bin. 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(-) ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
Hello Lukasz, Am 09.09.2014 15:43, schrieb Lukasz Majewski: 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? They have a USB HW protection circuit and therefore they only allowed to use fullspeed ... @Lukas Stockman: Maybe you could answer this better than me? bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only
Hi Bin, Lukasz, On 09/09/2014 08:43 AM, Lukasz Majewski wrote: 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? The drivers must support full-speed anyway based on the USB2.0 Specs. I have seen many cases in which only full-speed is suitable due to either board design or specific USB use case. One example is that to better fit to the production firmware flashing automation, a board does not use standard USB receptacle or plug, but a four touch pads on the PCB, so that the robot can grab the board and touch the pads to download the firmware. In this case signal integrity is a concern, but not download speed. Thanks for example. Best regards, Lukasz Regards, -Bin. 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 RD Institute Poland (SRPOL) | Linux Platform Group ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
