<div class="gmail_quote">On Sat, Jul 10, 2010 at 4:02 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</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">On 10.07.2010 21:20, Carl-Daniel Hailfinger wrote:<br>
> On 10.07.2010 20:15, Michael Karcher wrote:<br>
><br>
>> Am Samstag, den 10.07.2010, 19:41 +0200 schrieb Carl-Daniel Hailfinger:<br>
>><br>
</div><div class="im">>>> The spi_programmer array has a .write_256<br>
>>> member for every controller. In case the controller can't do anything<br>
>>> except 1-byte writes, we set<br>
>>> .write_256 = spi_chip_write_1_new<br>
>>><br>
>> Not every controller. SPI_CONTROLLER_DUMMY really has NULL in this<br>
>> field. Your patch does not change that. And please kill the test if we<br>
>> are sure that write_256 is always non-null.<br>
>><br>
><br>
> Right. I wanted to add a write_256 function to the dummy programmer<br>
> anyway. I'll send an updated patch.<br>
> Side note: If no SPI controller is selected, write_256 is empty as well,<br>
> but I'd say that's not a problem because it should never happen.<br>
><br>
><br>
><br>
</div><div class="im">>> If your plan is to base a new 1371 on top of this patch, I'm perfectly<br>
>> fine with these wrappers.<br>
>><br>
><br>
> Thanks. That's indeed the plan.<br>
><br>
<br>
</div>New version, should hopefully address all comments.<br>
<div class="im"><br>
Convert SPI chips to partial write, but wrap the write functions in a<br>
compat layer to allow converting the rest of flashrom later.<br>
I actually have patches for most of the remaining conversion, but I<br>
wanted to get this out and reviewed first.<br>
<br>
<br>
</div>Compile tested, but that's it. I would really appreciate it if someone<br>
could test this on a machine with SPI flash.<br></blockquote><div><br></div><div>FWIW, I downloaded the patch from Patchwork and tried it out on my NM10 chipset / W25Q32 chip and it seemed to work fine. Log is attached.</div>

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