[U-Boot] [PULL] u-boot-usb/master

The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
---------------------------------------------------------------- Alberto Panizzo (7): usb: rockchip: fix command failed on host side due to missing data usb: rockchip: implement skeleton for K_FW_GET_CHIP_VER command usb: rockchip: implement K_FW_LBA_READ_10 command usb: rockchip: implement K_FW_LBA_ERASE_10 command usb: rockchip: be quiet on serial port while transferring data usb: rockchip: fix printing csw debug info usb: rockchip: on K_FW_LBA_WRITE_10 remove magic block size of 512 bytes
Sam Protsenko (3): dfu: Fix data abort in dfu_free_entities() dfu: Fix memory leak in dfu_init_env_entities() dfu: Provide more verbose error message
Seung-Woo Kim (1): gadget: f_thor: fix hang-up with ctrl-c
Vipul Kumar (1): usb: dwc3: convert to livetree
arch/arm/include/asm/arch-rockchip/f_rockusb.h | 4 +++ doc/README.rockusb | 11 ++++-- drivers/dfu/dfu.c | 10 +++--- drivers/usb/common/common.c | 11 +++--- drivers/usb/dwc3/dwc3-generic.c | 17 ++++------ drivers/usb/gadget/f_rockusb.c | 261 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++------------- drivers/usb/gadget/f_thor.c | 4 +-- drivers/usb/host/xhci-dwc3.c | 3 +- drivers/usb/host/xhci-zynqmp.c | 3 +- include/linux/usb/otg.h | 6 ++-- 10 files changed, 276 insertions(+), 54 deletions(-)

On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260

On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.

On Thu, 26 Jul 2018 21:25:02 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.
Could you share how and where it breaks?
I did standard Travis-CI build test on DFU PR.
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de

On 07/26/2018 11:08 PM, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 21:25:02 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.
Could you share how and where it breaks?
I did standard Travis-CI build test on DFU PR.
I agree it'd be helpful if Tom shared more information than just "NAK", yes. Then again, did you try building one of those targets ?

On Thu, 26 Jul 2018 23:14:01 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 11:08 PM, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 21:25:02 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.
Could you share how and where it breaks?
I did standard Travis-CI build test on DFU PR.
I agree it'd be helpful if Tom shared more information than just "NAK", yes. Then again, did you try building one of those targets ?
I did the Travis-CI run during the night of Marek's u-boot-usb branch.
https://travis-ci.org/lmajewski/u-boot-dfu/jobs/408692849
The problem seems now to be with x86 sandbox. However it seems not to be related to USB:
Building current source for 29 boards (2 threads, 1 job per thread) x86: + theadorable-x86-conga-qa3-e3845 +Could not delete: No such file or directory +mv: cannot stat ?board/congatec/conga-qeval20-qa3-e3845/dsdt.hex?: No such file or directory +make[2]: *** [board/congatec/conga-qeval20-qa3-e3845/dsdt.c] Error 1 +make[1]: *** [board/congatec/conga-qeval20-qa3-e3845] Error 2
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de

On Fri, Jul 27, 2018 at 11:59:32AM +0200, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 23:14:01 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 11:08 PM, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 21:25:02 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.
Could you share how and where it breaks?
I did standard Travis-CI build test on DFU PR.
I agree it'd be helpful if Tom shared more information than just "NAK", yes. Then again, did you try building one of those targets ?
I did the Travis-CI run during the night of Marek's u-boot-usb branch.
When I pull -usb right now I see: Testing /tmp/am335x_hs_evm_uart Fri Jul 27 07:38:02 EDT 2018 WARNING: no status info for 'pico-pi-imx7d' WARNING: no maintainers for 'pico-pi-imx7d' Building current source for 1 boards (1 thread, 16 jobs per thread) arm: + am335x_hs_evm_uart +(am335x_hs_evm_uart) drivers/usb/musb-new/ti-musb.c: In function 'ti_musb_wrapper_bind': +(am335x_hs_evm_uart) drivers/usb/musb-new/ti-musb.c:221:29: error: incompatible type for argument 1 of 'usb_get_dr_mode' +(am335x_hs_evm_uart) dr_mode = usb_get_dr_mode(node); +(am335x_hs_evm_uart) ^~~~ +(am335x_hs_evm_uart) In file included from drivers/usb/musb-new/ti-musb.c:12:0: +(am335x_hs_evm_uart) include/linux/usb/otg.h:28:18: note: expected 'ofnode {aka union ofnode_union}' but argument is of type 'int' +(am335x_hs_evm_uart) enum usb_dr_mode usb_get_dr_mode(ofnode node); +(am335x_hs_evm_uart) ^~~~~~~~~~~~~~~ +(am335x_hs_evm_uart) make[2]: *** [drivers/usb/musb-new/ti-musb.o] Error 1 +(am335x_hs_evm_uart) make[1]: *** [drivers/usb/musb-new] Error 2 +(am335x_hs_evm_uart) make: *** [sub-make] Error 2 0 0 1 /1 am335x_hs_evm_uart

On 07/27/2018 01:39 PM, Tom Rini wrote:
On Fri, Jul 27, 2018 at 11:59:32AM +0200, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 23:14:01 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 11:08 PM, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 21:25:02 +0200 Marek Vasut marek.vasut@gmail.com wrote:
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
> The following changes since commit > 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784: > > mtd: nand: add new enum for storing ECC algorithm (2018-07-23 > 14:33:21 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-usb.git master > > for you to fetch changes up to > 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7: > > usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200) >
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
CCing the authors, I expect a patch.
Could you share how and where it breaks?
I did standard Travis-CI build test on DFU PR.
I agree it'd be helpful if Tom shared more information than just "NAK", yes. Then again, did you try building one of those targets ?
I did the Travis-CI run during the night of Marek's u-boot-usb branch.
When I pull -usb right now I see: Testing /tmp/am335x_hs_evm_uart Fri Jul 27 07:38:02 EDT 2018 WARNING: no status info for 'pico-pi-imx7d' WARNING: no maintainers for 'pico-pi-imx7d' Building current source for 1 boards (1 thread, 16 jobs per thread) arm: + am335x_hs_evm_uart +(am335x_hs_evm_uart) drivers/usb/musb-new/ti-musb.c: In function 'ti_musb_wrapper_bind': +(am335x_hs_evm_uart) drivers/usb/musb-new/ti-musb.c:221:29: error: incompatible type for argument 1 of 'usb_get_dr_mode' +(am335x_hs_evm_uart) dr_mode = usb_get_dr_mode(node); +(am335x_hs_evm_uart) ^~~~ +(am335x_hs_evm_uart) In file included from drivers/usb/musb-new/ti-musb.c:12:0: +(am335x_hs_evm_uart) include/linux/usb/otg.h:28:18: note: expected 'ofnode {aka union ofnode_union}' but argument is of type 'int' +(am335x_hs_evm_uart) enum usb_dr_mode usb_get_dr_mode(ofnode node); +(am335x_hs_evm_uart) ^~~~~~~~~~~~~~~ +(am335x_hs_evm_uart) make[2]: *** [drivers/usb/musb-new/ti-musb.o] Error 1 +(am335x_hs_evm_uart) make[1]: *** [drivers/usb/musb-new] Error 2 +(am335x_hs_evm_uart) make: *** [sub-make] Error 2 0 0 1 /1 am335x_hs_evm_uart
Seems this is broken because of the patch from Vipul , and while we all scramble to get this fixed, there is zero activity from Xilinx. So I'll just drop that patch and resubmit the PR. It's a real disappointment...

Hi
On 07/26/2018 05:14 PM, Tom Rini wrote:
On Thu, Jul 26, 2018 at 09:35:54AM +0200, Marek Vasut wrote:
The following changes since commit 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
mtd: nand: add new enum for storing ECC algorithm (2018-07-23 14:33:21 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch changes up to 67f1c59f40fde73cb2cbb51ebf0a05156ec403b7:
usb: dwc3: convert to livetree (2018-07-25 00:13:44 +0200)
NAK. This breaks at least: am335x_hs_evm am335x_hs_evm_uart brppt1_spi brppt1_mmc am335x_evm brppt1_nand and stih410-b2260
I can provide the fix for stih410-b2260.
Patrice
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
participants (4)
-
Lukasz Majewski
-
Marek Vasut
-
Patrice CHOTARD
-
Tom Rini