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>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.<br><br>Regards,<br><br>Darmawan Salihun
<br>--------------------------------------------------------------------<br>-= Human knowledge belongs to the world =-