On 09/25/2017 06:13 PM, Stephen Warren wrote:
Marek,
+CC Bin
I will drop the xhci patchset and hope to get a fixed one from him.
The latest u-boot-usb master branch breaks the following unit tests for the sandbox target (as run by test/py):
8 failed ... test_ut[ut_dm_blk_usb] ... test_ut[ut_dm_usb_flash] ... test_ut[ut_dm_usb_keyb] ... test_ut[ut_dm_usb_multi] ... test_ut[ut_dm_usb_remove] ... test_ut[ut_dm_usb_tree] ... test_ut[ut_dm_usb_tree_remove] ... test_ut[ut_dm_usb_tree_reorder]
The first failure is:
=> ut dm blk_usb Test: dm_test_blk_usb: blk.c USB0: scanning bus 1 for devices... usb_setup_descriptor: invalid max packet size failed, error -5 /var/lib/jenkins/workspace/u-boot-denx_uboot_usb-master-build/src/u-boot/test/dm/blk.c:78, dm_test_blk_usb(): 0 == uclass_get_device(UCLASS_MASS_STORAGE, 0, &usb_dev): Expected 0, got -19 Test: dm_test_blk_usb: blk.c (flat tree) USB0: scanning bus 1 for devices... usb_setup_descriptor: invalid max packet size failed, error -5 /var/lib/jenkins/workspace/u-boot-denx_uboot_usb-master-build/src/u-boot/test/dm/blk.c:78, dm_test_blk_usb(): 0 == uclass_get_device(UCLASS_MASS_STORAGE, 0, &usb_dev): Expected 0, got -19 Failures: 2
The others all look essentially the same.
It looks like this was introduced sometime on Saturday (US time, ignoring my Jenkins polling delay), and the two subsequent pushes on Sunday and Monday didn't solve this.