[U-Boot] [PATCH 6/9] cmd_sf: add handler for +len arg for erase command

Mike Frysinger vapier at gentoo.org
Fri Apr 15 09:49:22 CEST 2011


On Thursday, April 14, 2011 16:52:36 Richard Retanubun wrote:
> On 04/12/11 02:35, Mike Frysinger wrote:
> > This patch adds [+]len handler for the erase command that will
> > automatically round up the requested erase length to the flash's
> > sector_size.
> > ---
> > Richard: I noticed some issues in your original patch, so please try
> > 	this one instead and see if it still works for you.
> 
> Code looks okay. I won't have time to test for a while. Feel free to
> mainline if it works for you if it makes problems for me, I'll figure it
> out and poke you then.

ive already pushed it ;)

mainly the thing i noticed was that the boundary cases werent working as 
expected.  e.g. if the sector size was 0x10000, doing +0x10000 would round up 
to 0x20000.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110415/18c4d755/attachment.pgp 


More information about the U-Boot mailing list