On Tuesday 17 February 2009 00:10:24 Mike Frysinger wrote:
> +int eth_getenv_enetaddr(char *name, uchar *enetaddr)
> +{
> +     int ret = 0;
> +     char *addr;
> +
> +     addr = getenv(name);
> +     if (!addr)
> +             ret = -1;
> +     eth_parse_enetaddr(addr, enetaddr);
> +
> +     return ret;
> +}

the return value isnt shaping up the way i expected in most every other place.  
i'll change it to read:
return is_valid_ether_addr(enetaddr);

that way it'll behave as expected: 0->failure, 1->success
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to