On Wed, Jun 16, 2010 at 1:29 AM, Stefan Hajnoczi <[email protected]> wrote:
> pcnet32.c:pcnet32_setup_mac_addr:
>        for ( i = 0; i < ETH_ALEN; i++ )
>                promaddr[i] = inb ( ioaddr + i );
> This can be replaced with:
>        insb ( ioaddr, promaddr, ETH_ALEN );

I'm pretty sure this is incorrect; insb() loads N bytes in sequence
from the same I/O port.

Otherwise good suggestions :-)

-- Josh
_______________________________________________
gPXE-devel mailing list
[email protected]
http://etherboot.org/mailman/listinfo/gpxe-devel

Reply via email to