On Tue, Sep 26, 2017 at 5:32 PM, Bin Meng <bmeng...@gmail.com> wrote:
> Hi Stephen,
>
> On Tue, Sep 26, 2017 at 1:11 AM, Stephen Warren <swar...@wwwdotorg.org> wrote:
>> On 09/25/2017 10:40 AM, Marek Vasut wrote:
>>>
>>> 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 branch content (0184c6fb34b4 "usb: dwc2: Align size of
>> invalidating dcache before starting DMA") passes the tests.
>>
>
> My tests shows that 'ut dm usb_flash' fails, and should have failed
> for quite a long time.
>
> $ ./u-boot -d arch/sandbox/dts/test.dtb
>
>
> U-Boot 2017.09-00337-g0184c6f (Sep 26 2017 - 17:29:31 +0800)
>
> Model: sandbox
> DRAM: 128 MiB
> MMC: ** First descriptor is NOT a primary desc on 0:1 **
> ** First descriptor is NOT a primary desc on 1:1 **
> ** First descriptor is NOT a primary desc on 2:1 **
> mmc2: 2 (SD), mmc1: 1 (SD), mmc0: 0 (SD)
> Using default environment
>
> In: serial
> Out: vidconsole
> Err: vidconsole
> Model: sandbox
> SCSI: Net: eth0: eth@10002000, eth5: eth@10003000, eth3: sbe5,
> eth1: eth@10004000
> IDE: Bus 0: not available
> => ut dm usb_flash
> Test: dm_test_usb_flash: usb.c
> test/dm/usb.c:53, dm_test_usb_flash(): 2 == blk_dread(dev_desc, 0, 2,
> cmp): Expected 2, got 0
> Test: dm_test_usb_flash: usb.c (flat tree)
> test/dm/usb.c:48, dm_test_usb_flash(): 0 ==
> blk_get_device_by_str("usb", "0", &dev_desc): Expected 0, got -2
> Failures: 2
>
>>
>>>> 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]
>
> I've fixed all test failures except 'ut dm usb_flash'.
>
Looks I was jumping to conclusions. Previously I was just marking
sandbox as an exception in the 'usb_stop', and all tests were passed.
But I feel that's inconsistent. Now with a proper fix for sandbox, the
'usb remove' test no longer passed because I was trying to delete the
"/usb@1/hub/hub-emul/flash-stick@1" node from device tree but a
follow-up 'usb_init()' will fail. I am new to the sandbox emulator and
still investigating this. Any hints?
Regards,
Bin
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
