<div class="gmail_quote">On Mon, Oct 18, 2010 at 4:15 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coreboot.org" target="_blank">stepan@coreboot.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


* David Hendricks <<a href="mailto:dhendrix@google.com" target="_blank">dhendrix@google.com</a>> [101014 06:24]:<br>
<div><div></div><div>> patch attached.<br>
><br>
> Tested on a W25Q80...<br>
><br>
> Signed-off-by: David Hendricks <<a href="mailto:dhendrix@google.com" target="_blank">dhendrix@google.com</a>><br>
><br>
> --<br>
> David Hendricks (dhendrix)<br>
> Systems Software Engineer, Google Inc.<br>
<br>
</div></div>It seems this is included in the patch you sent on 2010-10-15<br>
However,<br>
<br>
Acked-by: Stefan Reinauer <<a href="mailto:stepan@coreboot.org" target="_blank">stepan@coreboot.org</a>><br></blockquote><div><br></div><div>Yeah, I was testing a patch using one system, came up with this patch, and then continued to test on more systems with different winbond chips. I should've revised this patch rather than sending a new one... sorry for the dupe, and thanks for the ack!</div>


<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
> Index: flashchips.c<br>
> ===================================================================<br>
> --- flashchips.c      (revision 1209)<br>
> +++ flashchips.c      (working copy)<br>
> @@ -7008,7 +7008,7 @@<br>
>               .total_size     = 1024,<br>
>               .page_size      = 256,<br>
>               .feature_bits   = FEATURE_WRSR_WREN,<br>
> -             .tested         = TEST_OK_PRE,<br>
> +             .tested         = TEST_OK_PREW,<br>
>               .probe          = probe_spi_rdid,<br>
>               .probe_timing   = TIMING_ZERO,<br>
>               .block_erasers  =<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>