<div class="gmail_quote">fyi, the following small errors are made in multiple times (copy + paste?) in ichspi.c</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Tue, Jun 14, 2011 at 3:20 PM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</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">+       msg_pdbg("Erasing %d bytes starting at 0x%06x\n", len, addr);</div><div><div class="h5">


+<br>
+       hsfc = REGREAD16(ICH9_REG_HSFC);<br>
+       /* make sure FDONE, FCERR, AEL are cleared by writing 1 to them */<br>
+       REGWRITE16(ICH9_REG_HSFC, hsfc);<br></div></div></blockquote><div><br></div><div>Should be done using HSFS.</div><div><br></div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div>On Tue, Jun 14, 2011 at 3:20 PM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</a>></span> wrote: </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="h5">+       hsfs = REGREAD16(ICH9_REG_HSFS);<br>
+       if (!timeout) {<br>
+               msg_perr("timeout!\n");<br>
+               prettyprint_ich9_reg_hsfs(hsfs);<br>
+               prettyprint_ich9_reg_hsfc(hsfs);<br></div></div></blockquote><div><br></div><div>The second prettyprint should take hsfc. Don't forget to read hsfc before going into the if() statement (like you did with hsfs).</div>

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