On 8/24/07, <b class="gmail_sendername">Darmawan Salihun</b> <<a href="mailto:darmawan.salihun@gmail.com">darmawan.salihun@gmail.com</a>> wrote:<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<span class="q">On 8/23/07, <b class="gmail_sendername">Peter Stuge</b> <<a href="mailto:peter@stuge.se" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">peter@stuge.se</a>> wrote:<div><span class="gmail_quote">

</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Thu, Aug 23, 2007 at 01:28:04PM +0700, Darmawan Salihun wrote:<br>> However, it seems to be the patch will stay as different file<br>> because among all of the flashchip support files, there's no<br>> one of them that can provide the register unprotect routine(s).
<br><br>The idea is to keep code duplication to a minimum where possible, so<br>if W39V are quite similar to another already-supported chip except<br>for these registers it may be a good idea to just add the neccessary<br>


register stuff to that chip code instead. (But only called for W39V,<br>of course.)<br></blockquote></div><br></span>Done.<br><br>During testing I found that W39V040FA doesn't seem supporting per-sector erase. <br>As other SST Firmware Hub chips. Therefore, I add the code to give erase support 
<br>for the chip. See patch.<div><span class="e" id="q_11498562295ed3ac_2"><br></span></div></blockquote></div>I had the chance to test per-sector erase once more last night and I can confirm that it's _not_ working in my testbed (DFI 865PE Infinity motherboard -- ICH5-W39V040FA). Therefore, a whole chip erase is the only option as in the latest patch I sent. 
<br> <br>Regards,<br><br>Darmawan Salihun<br>--------------------------------------------------------------------<br>-= Human knowledge belongs to the world =-