On Mon, Jul 26, 2010 at 2:49 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><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;">

Hi Misha,<br>
<div class="im"><br>
On 15.07.2010 01:44, Michael Manulis wrote:<br>
> On Wed, Jul 14, 2010 at 4:03 PM, Carl-Daniel Hailfinger<br>
> <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
><br>
</div><div class="im">>> I have added <a href="mailto:flashrom@flashrom.org">flashrom@flashrom.org</a> to CC again to make sure others can<br>
>> follow us.<br>
>><br>
>> On 12.07.2010 20:06, Michael Manulis wrote:<br>
>><br>
>>> I finally got around to checking your patch.  Short story, flashrom could<br>
>>> not find any flash chips.  The log as follows.  I had to make one change<br>
>>> to get the code to compile on PPC, here's the diff:<br>
>>><br>
>>> Index: chipset_enable.c<br>
>>> ===================================================================<br>
>>> --- chipset_enable.c    (revision 1075)<br>
>>> +++ chipset_enable.c    (working copy)<br>
>>> @@ -33,10 +33,10 @@<br>
>>>  #include <unistd.h><br>
>>>  #include "flash.h"<br>
>>><br>
>>> +#define NOT_DONE_YET 1<br>
>>> +<br>
>>>  #if defined(__i386__) || defined(__x86_64__)<br>
>>><br>
>>> -#define NOT_DONE_YET 1<br>
>>> -<br>
>>>  static int enable_flash_ali_m1533(struct pci_dev *dev, const char *name)<br>
>>>  {<br>
>>>         uint8_t tmp;<br>
>>><br>
>>><br>
>> Thanks. I already had that change in my local tree, but forgot to push it.<br>
>><br>
>><br>
>><br>
</div><div class="im">>> Good. That's expected when using the default internal programmer. IIRC<br>
>> you wanted to flash the SATA controller on your CurtisWright Raptor.<br>
>> Could you try<br>
>> flashrom -p satasii -V<br>
>><br>
</div><div class="im">> I ran the following commands to test the patch:<br>
><br>
> ./flashrom -p satasii -V<br>
> ./flashrom -p satasii -Vr sata.dump<br>
> ls -lh sata.dump<br>
> ./flashrom -p satasii -VE<br>
> ./flashrom -p satasii -Vr zero.dump<br>
> hexdump zero.dump<br>
> ./flashrom -p satasii -Vw sata.dump<br>
> ./flashrom -p satasii -Vv sata.dump<br>
><br>
> I attached the log as the file is 100K and I was reluctant to just paste it<br>
> in the body.<br>
><br>
> The log is also available on <a href="http://pastebin.com" target="_blank">pastebin.com</a> at <a href="http://pastebin.com/kHkynj7s" target="_blank">http://pastebin.com/kHkynj7s</a><br>
><br>
> Please let me know if you would like me to run other tests.<br>
><br>
<br>
</div>I tried to find people with deep PCI and architecture knowledge for<br>
reviews, but they are all extremely busy. However, since you tested this<br>
successfully, I think we can say this is safe.<br>
<br>
Could you reply-to-all with the following line:<br>
> Acked-by: Your Name <your@email><br>
so I can go ahead and commit this change?<br>
Thanks!<br>
<div><div></div><div class="h5"><br>
Regards,<br>
Carl-Daniel<br>
<br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</div></div></blockquote></div><br>Acked-by: Misha Manulis <<a href="mailto:misha@manulis.com">misha@manulis.com</a>><br>