2009/10/26 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><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Luc,<br>
<br>
can you bounce/forward/... your mail to the users you had copied in your<br>
first mail so they can test? Thanks!<br>
<div class="im"><br>
On 25.10.2009 18:58, Luc Verhaegen wrote:<br>
> After last night, i was disgusted to such an extent that i went and did<br>
> this code that should meet all the (at least) logical demands made<br>
> earlier. And to think that this started with adding 1 more simple board<br>
> enable<br>
> (<a href="http://www.flashrom.org/pipermail/flashrom/2009-October/000750.html" target="_blank">http://www.flashrom.org/pipermail/flashrom/2009-October/000750.html</a>).<br>
><br>
> Some remarks here:<br>
> * we do not care about the name of the intel ICH. ICH name was detected<br>
> and mentioned before anyway.<br>
> * we do not try to match a specific intel ICH in the board enable.<br>
> That's what the big table was for to begin with.<br>
> * This tiny 190 line function is entirely redone if we have a second<br>
> io line that needs to be toggled.<br>
> * dell poweredge: comment is useless, use the commit message. There<br>
> might be a time when 50 boards might need exactly this gpio pin raised.<br>
> * i might have messed up a bit or so in the gpio line checking masks,<br>
> but cannot be bothered to go and trawl through everything yet again to<br>
> verify this.<br>
><br>
<br>
</div>I checked the old vs. the new code and it looks OK. There are some minor<br>
nitpicks, but I can send them as followup patch after this is merged.<br>
<br>
Acked-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>><br></blockquote><div><br><a href="http://patchwork.coreboot.org/patch/487/">http://patchwork.coreboot.org/patch/487/</a><br>
Tested on real hardware.<br><br>Acked-by: <span>Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>></span><br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<br>
Regards,<br>
Carl-Daniel<br>
<div class="im"><br>
--<br>
Developer quote of the week:<br>
"We are juggling too many chainsaws and flaming arrows and tigers."<br>
<br>
<br>
</div>_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org">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>
</blockquote></div><br>