Hi Mike, On Fri, Feb 3, 2012 at 5:51 AM, Mike Frysinger <vap...@gentoo.org> wrote: > On Thursday 19 January 2012 19:53:07 Joe Hershberger wrote: >> --- a/net/bootp.c >> +++ b/net/bootp.c >> >> +#ifdef CONFIG_BOOTP_RANDOM_DELAY >> +#include "net_rand.h" >> +#endif > > let's push this CONFIG_xxx check into net_rand.h so we don't have to check it > every time we include the file
In this case, it doesn't make sense. The rand is enabled based on other features that need it... it does not have a config of its own. As a result, putting this in the header would make it worse. This guard is still needed in bootp.c, since rand is only needed when that subfeature is enabled. For link-local, it will always be included. -Joe _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
