2010/10/11 Maciej Pijanka <span dir="ltr"><<a href="mailto:maciej.pijanka@gmail.com" target="_blank">maciej.pijanka@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div>On Mon, 11 Oct 2010, Carl-Daniel Hailfinger wrote:<br>
<br>
> On 10.10.2010 15:19, Uwe Hermann wrote:<br>
> > On Sat, Oct 09, 2010 at 11:59:24PM +0200, Carl-Daniel Hailfinger wrote:<br>
> ><br>
> >> Refactor remaining write wrappers.<br>
> >> Kill duplicated code.<br>
> >><br>
> >> Signed-off-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net" target="_blank">c-d.hailfinger.devel.2006@gmx.net</a>><br>
> >><br>
> ><br>
> > Acked-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.de</a>><br></div></blockquote><div><br>Acked-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>><br>

</div><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>> ><br>
><br>
> Thanks for the review. Here is an extended patch which has some more<br>
> refactoring to make the switchover to partial write less painful. Parts<br>
> have already been reviewed by Uwe.<br>
><br>
> If possible, please test this code on all chip classes, one per write<br>
> function listed below.<br>
> - write_82802ab()<br>
> - write_jedec()<br>
> - write_jedec_1()<br>
> - write_m29f400bt()<br>
> - write_28sf040()<br>
> Testing of SPI chips is not needed. They are not touched in this patch.<br>
><br>
> Refactor remaining write wrappers.<br>
> Kill duplicated code.<br>
> Annotate write functions with their chunk size.<br>
> Mark Fujitsu MBM29F400BC and ST M29F400BB as untested because their<br>
> write code no longer uses a broken layout.<br>
><br>
> Signed-off-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net" target="_blank">c-d.hailfinger.devel.2006@gmx.net</a>><br>
<br>
</div>I did only testing with this patch applied on nic3com programmer, and at29c512 chip<br>
works without any problem here.<br>
No problems with compilation or applying over r1209<br>
<br>
Maciej<br>
<font color="#888888"><br>
--<br>
Maciej Pijanka<br>
I don't fear computers, I fear lack of them -- Isaac Asimov<br>
</font><div><div></div><div><br>
_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org" target="_blank">flashrom@flashrom.org</a><br>
<a href="http://www.flashrom.org/mailman/listinfo/flashrom" target="_blank">http://www.flashrom.org/mailman/listinfo/flashrom</a><br>
</div></div></blockquote></div><br>