Anton Vorontsov wrote: > On Tue, Sep 22, 2009 at 04:03:16PM +0200, Joakim Tjernlund wrote: > [...] > >>>>> Also >>>>> drivers/qe/uec.h:int uec_initialize(bd_t *bis, uec_info_t *uec_info); >>>>> include/netdev.h:int uec_initialize(int index); >>>>> different prototypes for the same function. >>>>> >>>> BTW, I am looking for a way to swap the order of ethernet interfaces: >>>> static uec_info_t uec_info[] = { >>>> #ifdef CONFIG_UEC_ETH1 >>>> STD_UEC_INFO(1), /* UEC1 */ >>>> #endif >>>> #ifdef CONFIG_UEC_ETH2 >>>> STD_UEC_INFO(2), /* UEC2 */ >>>> #endif >>>> #ifdef CONFIG_UEC_ETH3 >>>> STD_UEC_INFO(3), /* UEC3 */ >>>> #endif >>>> }; >>>> >>> Works for me: >>> >>> http://lists.denx.de/pipermail/u-boot/2009-September/060821.html >>> >> Right, but I don't consider a include as this: >> +#include "../../../drivers/qe/uec.h" >> as the correct way of getting of required data types and macros. >> Consider that uec_initialize() is exported by netdev.h (although with the >> wrong prototype ATM). As far as I can tell, I should only have to include >> netdev.h to get the required types and macros. >> > > Not sure if having all-in-one netdev header is a good idea. > It might be a good idea to move uec.h to "include/" though. > > This needs to be cleaned up. THE prototype for the global initialize() function needs to be in netdev.h and nowhere else.
BTW - can't you effectively swap the order of the Ethernet interfaces at runtime using the 'ethprime' environment variable? regards, Ben _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot