[U-Boot] [PATCH v2 00/26] usb: Drop requirement for USB unbinding, add tests

Simon Glass sjg at chromium.org
Mon Nov 9 17:54:40 CET 2015


Hi Hans,

On 9 November 2015 at 00:14, Hans de Goede <hdegoede at redhat.com> wrote:
> Hi Simon,
>
> On 09-11-15 07:47, Simon Glass wrote:
>>
>> There was quite a bit of discussion about the change that required the
>> unbinding of USB devices for the subsystem to function correctly. E.g.
>>
>> https://patchwork.ozlabs.org/patch/485637/
>>
>> The key issue is the usb_get_dev_index() function which is not a good API
>> for driver model. We can drop use of this function once everything is
>> converted to driver model. Then I believe the problems raised by Hans go
>> away. For now we can add a deprecation warning on the function.
>>
>> It is easy to convert USB keyboards to driver model. This series includes
>> a patch for that.
>>
>> This series also includes reverts for the three commits which as discussed
>> I would like to drop. U-Boot should be able to run normally and exit
>> without
>> unbinding anything.
>>
>> Also included are some tests for the 'usb tree' command. To make this
>> work,
>> console recording is implemented.
>>
>> Finally, a USB keyboard driver is provided for sandbox, so that this part
>> of the stack can be tested automatically.
>>
>> Changes in v2:
>> - Add various patches to support USB keyboards and additional tests
>
>
> Do you have a git branch with this patch-set somewhere for me to test ?

Yes I always upload branches before sending them. This one is
u-boot-dm/usb-working. I should get better at including the branch
name in the cover letter.

I still need to write a test for having two device types which switch
order. Now that I have a USB keyboard emulation I should be able to do
that. I'll then see the impact on 'usb tree'. So you may want to
review the test code only for now.

Regards,
Simon


More information about the U-Boot mailing list