[U-Boot] MPC83xx and uec

[Joakim Tjernlund]

Anton Vorontsov <avorontsov at ru.mvista.com> wrote on 22/09/2009 15:51:17:

> From:
>
> Anton Vorontsov <avorontsov at ru.mvista.com>
>
> To:
>
> Joakim Tjernlund <joakim.tjernlund at transmode.se>
>
> Cc:
>
> u-boot at lists.denx.de, kim.phillips at freescale.com
>
> Date:
>
> 22/09/2009 15:51
>
> Subject:
>
> Re: [U-Boot] MPC83xx and uec
>
> On Tue, Sep 22, 2009 at 01:02:27PM +0200, Joakim Tjernlund wrote:
> > >
> > >
> > > I can't find a way to get at uec_info_t/STD_UEC_INFO from my board_eth_init()
> > > in current u-boot. Am I missing something?
> > > 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.

  Jocke