[U-Boot] [PATCH] usb:composite:fix Provide function data when addressing device with only one interface

Marek Vasut marex at denx.de
Wed Mar 6 00:24:51 CET 2013


Dear Lukasz Majewski,

> Hi Marek,
> 
> > Dear Lukasz Majewski,
> > 
> > > This commit fixes problems with some non-standard requests send with
> > > device address instead of interface address (bmRequestType.Receipent
> > > field).
> > > 
> > > This happens with dfu-util (debian version: 0.5), which address non
> > > standard requests (like w_value=0x21 and bRequest=GET_DESCRIPTOR) to
> > > device. Without this fix, the above request is STALLED, and hence
> > > causes dfu-util to assume some standard configuration (packet size
> > > = 1024B instead of 4096B) In turn it displays following errors:
> > > Error obtaining DFU functional descriptor
> > > Warning: Assuming DFU version 1.0
> > > Warning: Transfer size can not be detected
> > > ...
> > > Warning: Trying default transfer size 1024
> > > 
> > > This fix allows passing non-standard request to function setup
> > > code, where it shall be handled.
> > > 
> > > Tested at: 	Trats (exynos4210)
> > > Tested with:DFU and UMS gadgets
> > 
> > Can you please repost all the patches you want applied on
> > u-boot-usb ? I'm completely lost in all this DFU stuff.
> 
> This is only one patch to replace patch done by Pantelis:
> 
> Patch to replace (revert):
> dfu: Send correct DFU response from composite_setup
> SHA1: fef31049fa06e516945114195eb14ac3549d0ad2
> 
> And this shall be replaced with posted patch.
> 
> Another patches (with UMS gadget support - done by Piotr Wilczek), also
> shall be added to u-boot-usb tree:
> 
> http://patchwork.ozlabs.org/patch/219744/
> http://patchwork.ozlabs.org/patch/219746/
> http://patchwork.ozlabs.org/patch/219745/

Ok, please check the u-boot-usb ... if it's OK, I will send Tom a pullrq 
(finally). Tom, do you still accept those for .04 ?

Best regards,
Marek Vasut


More information about the U-Boot mailing list