Hi, Peter,<br><br>    The LAN chip-set enabled by hardware. So it need not activable by software.<br>    Thanks.<br><br><br><div class="gmail_quote">2009/10/21 Peter Stuge <span dir="ltr"><<a href="mailto:peter@stuge.se">peter@stuge.se</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hello, Thanks for the patch!<br>
<div class="im"><br>
Libra Li wrote:<br>
> --- src/mainboard/technexion/tim5690/mainboard.c    (revision 4821)<br>
</div>..<br>
<div class="im">> -<br>
> -/***************************************************<br>
> -* This board, the TIM-5690 has two Marvel 88e5056 PCI-E<br>
> -* 10/100/1000 chips on board.<br>
> -* Both of their pin PERSTn pins are connected to GPIO 5 of the<br>
> -* SB600 southbridge.<br>
> -****************************************************/<br>
> -static void enable_onboard_nic()<br>
> -{<br>
<br>
</div>Can you explain briefly why this code can be removed without loss of<br>
functionality?<br>
<br>
Is it simply not needed for this board?<br>
<br>
<br>
Thanks!<br>
<br>
//Peter<br>
<font color="#888888"><br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
</font></blockquote></div><br>