[U-Boot-Users] [PATCH 1/1] Re-Submit: Add support for ATMEL AT91SAM9G20EK board.
Hong Xu
hongxu.cn at gmail.com
Tue Jul 29 08:19:02 CEST 2008
Thanks Ben.
I'll wait for more suggestions and re-generate the whole patch again.
Hope I do not bother you guys too much. It's my first time saying
something in this list. :-)
On Tue, Jul 29, 2008 at 12:57, Ben Warren <biggerbadderben at gmail.com> wrote:
> Hi Hong,
>
> Thanks for all the hard work. One more thing...
> <snip>
>> +#if defined(CONFIG_CMD_NET)
>> +int board_eth_init(bd_t *bis)
>> +{
>> + return at91sam9_eth_initialize(bis);
>> +}
>> +#endif
>
> Here's at91sam9_eth_initialize:
>
> #if defined(CONFIG_MACB) && defined(CONFIG_CMD_NET)
> void at91sam9_eth_initialize(bd_t *bi)
> {
> macb_eth_initialize(0, (void *)AT91_BASE_EMAC, 0x00);
> }
> #endif
>
> Issues:
> 1. It returns void
> 2. It just calls the MACB initializer.
>
> Please change your code to:
>
> extern int macb_eth_initialize(int id, void *regs, unsigned int phy_addr);
>
> #if defined(CONFIG_MACB) && defined(CONFIG_CMD_NET)
> int board_eth_init(bd_t *bi)
> {
> return macb_eth_initialize(0, (void *)AT91_BASE_EMAC, 0x00);
> }
> #endif
>
> or something like this. I don't know if AT91_BASE_EMAC is visible
> from this code, so you may need to modify slightly.
>
> regards,
> Ben
>
More information about the U-Boot
mailing list