[U-Boot] MPC83xx and uec

[Joakim Tjernlund]

Ben Warren <biggerbadderben at gmail.com> wrote on 22/09/2009 18:55:22:
>
> 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?

I did this quite some time ago so memory isn't fresh, but at time
I looked high and low(including ethprime) and nothing worked all the way so I
ended up swapping by patching generic code instead. Now I can do the
same in board code, but current u-boot is a bit fuzzy in this area.

  Jocke