[U-Boot] [PATCH RFC] dfu: ram support

Afzal Mohammed afzal.mohd.ma at gmail.com
Mon Sep 9 17:22:03 CEST 2013


Hi Lukasz Majewski,

On Mon, Sep 09, 2013 at 08:50:58AM +0200, Lukasz Majewski wrote:

> > usage: <image name> ram <start address> <size>
> > eg. kernel ram 0x81000000 0x1000000
> > 
> > Downloading images to RAM using DFU is not something new, this is
> > acheived in openmoko also.
> > 
> > DFU on RAM can be used for extracting RAM contents to host using dfu
> > upload. Perhaps this can be extended to io for squeezing out register
> > dump through usb, if it is worth.

> Above idea sounds very interesting. 

Thanks

> One minor thing: 
> It also would be good to have dfu_alt_info environment properly defined
> to have "ram" alt setting for beagle bone. Then we would have at least
> one board which supports this new feature.

Sure

> > +enum dfu_ram_op {
> > +	DFU_OP_READ = 1,
> > +	DFU_OP_WRITE,
> > +};

> Minor:
> Now I've realised that the dfu_nand_op and dfu_mmc_op have the same
> defines. Maybe it is a good time to combine this and store it at
> dfu.h?

Yes it is better

> Despite one minor comment, I like the code and looking forward for a
> patch :-).

Thanks, will wait for a working day for those on other side of the
globe and if no negative, would repost addressing your comments.

Regards
Afzal


More information about the U-Boot mailing list