"Hong Xu" <[EMAIL PROTECTED]> wrote:
> diff --git a/drivers/net/macb.c b/drivers/net/macb.c
> index aa39284..c3dcce6 100644
> --- a/drivers/net/macb.c
> +++ b/drivers/net/macb.c
> @@ -415,14 +415,14 @@ static int macb_init(struct eth_device *netdev, bd_t 
> *bd)
>       /* choose RMII or MII mode. This depends on the board */
>  #ifdef CONFIG_RMII
>  #if defined(CONFIG_AT91CAP9) || defined(CONFIG_AT91SAM9260) || \
> -    defined(CONFIG_AT91SAM9263)
> +    defined(CONFIG_AT91SAM9263) || defined(CONFIG_AT91SAM9G20)

Hmm...I don't really like the direction this is going. Any chance you
could have all AT91 boards define CONFIG_AT91 and check for that
instead?

Alternatively, we could do it the other way around and test for
CONFIG_AVR32 instead, since all AVR32 boards already define that.

In either case, this should probably be a separate cleanup; I don't
have any objections to your patch.

Haavard

-------------------------------------------------------------------------
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

Reply via email to