From mboxrd@z Thu Jan  1 00:00:00 1970
From: Afzal Mohammed <afzal.mohd.ma@gmail.com>
Date: Mon, 9 Sep 2013 20:52:03 +0530
Subject: [U-Boot] [PATCH RFC] dfu: ram support
In-Reply-To: <20130909085058.53c1921c@amdc308.digital.local>
References: <1378643888-2992-1-git-send-email-afzal.mohd.ma@gmail.com>
	<20130909085058.53c1921c@amdc308.digital.local>
Message-ID: <20130909152203.GA3391@afzal-ThinkPad-R50e>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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