From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Fri, 15 Apr 2011 03:49:22 -0400 Subject: [U-Boot] [PATCH 6/9] cmd_sf: add handler for +len arg for erase command In-Reply-To: <4DA75E94.2020202@RuggedCom.com> References: <1302590128-8721-1-git-send-email-vapier@gentoo.org> <1302590128-8721-7-git-send-email-vapier@gentoo.org> <4DA75E94.2020202@RuggedCom.com> Message-ID: <201104150349.23292.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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