On Thursday 19 January 2012 19:53:19 Joe Hershberger wrote: > Changes to NetState now go through an accessor function called > NetSetState()
i'm not sure i see the value here. all i see is overhead being added. if you really want this to be an improvement, there's two minor tweaks to make. > --- a/include/net.h > +++ b/include/net.h > > +/* Network loop state */ > +extern void NetSetState(int state); make this into a static inline > +#define NETLOOP_CONTINUE 1 > +#define NETLOOP_RESTART 2 > +#define NETLOOP_SUCCESS 3 > +#define NETLOOP_FAIL 4 turn these into an enum -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