On Monday 30 July 2012 17:08:41 Joe Hershberger wrote: > On Wed, Jul 25, 2012 at 1:49 PM, Mike Frysinger wrote: > > On Tuesday 24 July 2012 16:11:15 Joe Hershberger wrote: > >> --- a/net/eth.c > >> +++ b/net/eth.c > >> > >> +#ifdef CONFIG_NETCONSOLE_PERSIST_ETH > >> +int eth_init_state_only(bd_t *bis) > >> +{ > >> + eth_current->state = ETH_STATE_ACTIVE; > >> + > >> + return 0; > >> +} > >> + > >> +void eth_halt_state_only(void) > >> +{ > >> + eth_current->state = ETH_STATE_PASSIVE; > >> +} > >> +#endif > > > > these *really* should be static inlines in the global header. they're so > > dirt simple, the overhead of the function call is probably much higher > > than the single memory store. > > I can do that, but I don't think it will save anything. Since > eth_current is static, I would have to change it to eth_get_dev(), and > we're back to a function call. Thoughts?
i wonder why eth_get_dev is an external func then :) -mike
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