[U-Boot] [PATCH] at91: Support for the at91sam9g20 : Atmel 400Mhz ARM 926ej-s SOC.

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Fri Mar 13 12:55:32 CET 2009


On 13:40 Wed 11 Mar     , Nicolas Ferre wrote:
> Ben Warren :
>> Hi Nicolas,
>>
>> Nicolas Ferre wrote:
>> <snip>
>>> diff --git a/drivers/net/macb.c b/drivers/net/macb.c
>>> index af0409b..6de0a04 100644
>>> --- a/drivers/net/macb.c
>>> +++ b/drivers/net/macb.c
>>> @@ -447,14 +447,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)
>>>      macb_writel(macb, USRIO, MACB_BIT(RMII) | MACB_BIT(CLKEN));
>>>  #else
>>>      macb_writel(macb, USRIO, 0);
>>>  #endif
>>>  #else
>>>  #if defined(CONFIG_AT91CAP9) || defined(CONFIG_AT91SAM9260) || \
>>> -    defined(CONFIG_AT91SAM9263)
>>> +    defined(CONFIG_AT91SAM9263) || defined(CONFIG_AT91SAM9G20)
>>>      macb_writel(macb, USRIO, MACB_BIT(CLKEN));
>>>  #else
>>>      macb_writel(macb, USRIO, MACB_BIT(MII));
>>>   
>> Please find a more generic way of doing this.  There should be no  
>> board/cpu specific #ifdefs in common drivers.  While you're at it, fix  
>> up the existing ones.
>
> Ben,
>
> Maybe we can separate the two topics in two separate actions.
> I propose that we go on with this patch for at91sam9g20 adding and,  
> after this, we can concentrate on chasing #ifdefs in drivers.
Agreed

Best Regards,
J.


More information about the U-Boot mailing list