[gPXE-devel] [PATCH] [pcnet32] Replace pcnet32 driver

Joshua Oreman oremanj at rwcr.net
Wed Jun 16 11:14:15 EDT 2010


On Wed, Jun 16, 2010 at 1:29 AM, Stefan Hajnoczi <stefanha at gmail.com> 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


More information about the gPXE-devel mailing list