[U-Boot] [RFC PATCH v2 0/5] SPL: DFU Support in SPL

Tom Rini trini at konsulko.com
Mon Jul 25 17:34:08 CEST 2016


On Mon, Jul 25, 2016 at 03:25:52PM +0000, B, Ravi wrote:
> Tom
> 
> >> >How well did you test DFU on other platforms after these changes?
> >> >Thanks
> >> 
> >> Actually earlier platforms did not had this support,  support for 
> >> either RNDIS boot mode (CONFIG_SPL_USBETH_SUPPORT) or CONFIG_SPL_USB_SUPPORT (boot from mass-storage device) were only supported.
> 
> >Yes, sorry, I mean you're making generic DFU code changes too, did you test them on other TI platforms which support DFU in other cases to make sure nothing slipped in and broke them with these changes?
> 
> I got your point, You are referring to generic dfu code changes, that is moving common dfu code to  common/dfu.c , 
> so that both u-boot and SPL can leverage common code. But this change is generic cleanup code.
> Yes, I did test DFU functionality on other platforms (say AM335X).  I can check other platform as well. 
> 
> But SPL-DFU is tied with ROM-USB peripheral boot mode, I cannot test other than DRA7X/AM57X.
> But in order to verify the SPL-DFU boot on other platform,  I have tested SPL-DFU on AM335x,  hacking common/spl/spl.c to
> execute SPL-DFU code by default and could able to load u-boot.img into RAM and boot to u-boot. (using dfu-utils from PC host).
> 
> Let me know I have clarified.

OK.  Please note in the next round (and in general) when you also test
that generic changes have been confirmed on other platforms, thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160725/5b8b7b13/attachment.sig>


More information about the U-Boot mailing list