[U-Boot] [PATCH 26/28] net: Add link-local addressing support

Joe Hershberger joe.hershberger at gmail.com
Mon Feb 13 21:21:12 CET 2012


Hi Mike,

On Fri, Feb 3, 2012 at 6:42 AM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Thursday 19 January 2012 19:53:29 Joe Hershberger wrote:
>> +     NetOurGatewayIP = 0;
>> +     ip_to_string(NetOurGatewayIP, tmp);
>> +     setenv("gatewayip", tmp);
>> +
>> +     ip_to_string(NetOurSubnetMask, tmp);
>> +     setenv("netmask", tmp);
>> +
>> +     ip_to_string(NetOurIP, tmp);
>> +     setenv("ipaddr", tmp);
>
> i feel like we should add a helper to do this
>        setenv_ip("ipaddr", NetOurIP);

I think that's a good idea, but I don't think it should be part of
this patch series.  It would probably be clearer to do that
independently later.

>> --- a/net/Makefile
>> +++ b/net/Makefile
>>
>> -COBJS        := $(COBJS-y)
>> +COBJS        := $(sort $(COBJS-y))
>
> why ?

Sort is needed because there are 2 different config options that could
add the net_rand source file and sort will eliminate duplicates.

-Joe


More information about the U-Boot mailing list