<div class="gmail_quote">On Mon, Nov 1, 2010 at 8:09 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">On 02.11.2010 04:01, David Hendricks wrote:<br>
> I think this patch is important and should go in very soon. The code looks<br>
> good to me, and I've successfully tested with it several times.<br>
><br>
> Are there any outstanding issues or patches that should be submitted first?<br>
><br>
<br>
</div>Error recovery from a failed erase could be better, but it is pretty<br>
much on par with the current code.<br>
<br>
I have some cleanups and patches for better error recovery which<br>
conflict with this patch, but they shouldn't hold this patch back. I<br>
hope to resend them immediately after this patch is merged (and some of<br>
them will be tiny afterwards).<br>
<br>
To summarize, I think nothing else is left to do before committing this<br>
patch.<br></blockquote><div><br></div><div>Alrighty then! I've stared at the code as much as my eyes will allow and have tested it on a couple different platforms with SPI flash (AMD870 and Intel NM10), so:</div><div>

<br></div><div>Acked-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>></div><div><br></div><div>Idwer's been doing a lot with this patch too, so you might want to wait for his ack as well.</div>

<div><br></div><div>-- </div></div>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>