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
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
