On Thu, 24 Jul 2008, Haavard Skinnemoen wrote:
Julien May mailinglist@miromico.ch 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