[U-Boot] [PATCH 6/8] AM35xx: add EMAC support

[Igor Grinberg]

On 10/14/11 09:51, Ilya Yanok wrote:
> Hi Mike,
> 
> On 14.10.2011 02:55, Mike Frysinger wrote:
>>> --- /dev/null
>>> +++ b/arch/arm/cpu/armv7/omap3/emac.c
>>>
>>> +/*
>>> + * Initializes on-chip ethernet controllers.
>>> + * to override, implement board_eth_init()
>>> + */
>>> +int cpu_eth_init(bd_t *bis)
>>> +{
>>> +	davinci_emac_initialize();
>>> +
>>> +	return 0;
>>> +}
>>
>> shouldn't emac support always be up to the board porter and not the SoC code ?  
> 
> It's up to the board porter to define CONFIG_DRIVER_TI_EMAC (or not),
> isn't it enough?

Yes, it should be enough, especially in case of EMAC on Davinci and AM35x,
when this IP block is always present inside the SoC.

> 
>> also, this should be:
>> int cpu_eth_init(bd_t *bis)
>> {
>> 	return davinci_emac_initialize();
>> }
> 
> Agreed.
> 
> Regards, Ilya.
> 
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
> 

-- 
Regards,
Igor.