On Thu, 24 Jul 2008, Haavard Skinnemoen wrote:
> Julien May <[EMAIL PROTECTED]> wrote:
> > I am currently having problems in initializing the eth.
> >
> > in net/eth.c the following is defined
> >
> > static int __def_eth_init(bd_t *bis)
> > {
> > return -1;
> > }
> > int cpu_eth_init(bd_t *bis) __attribute((weak, alias("__def_eth_init")));
> > int board_eth_init(bd_t *bis) __attribute((weak, alias("__def_eth_init")));
> >
> > this is not calling my implementation of board_eth_init.
> > I see and like the idea behind this but do not understand why my
> > implementation is not getting called...
>
> Hmm, that's probably the "weak functions cannot be overriden by
> functions defined in their own file" crap that was discussed earlier.
> Can you try moving board_eth_init() into hammerhead.c?
Did so and it works now. I could make for this an incremental patch and
send this to you.
>
> Which reminds me...Ben posted a patch which did that for all the
> existing avr32 boards. I should probably apply it.
Otherwise I wait until you applied bens patch and test again hammerheads
functionality in u-boot.
>
> > btw. I did update the implementation function header from void to int.
>
> Yeah, that's probably a good idea too :-)
>
> Haavard
>
- Julien
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
