[U-Boot] [PATCH v3 5/6] da850/omap-l138: davinci_emac: Suppress auto negotiation if needed

[Tom Rini]

On Fri, Sep 7, 2012 at 1:08 AM, Prabhakar Lad
<prabhakar.csengg at gmail.com> wrote:
> Hi Bastian,
>
> Thanks for the patch.
>
> On Thu, Sep 6, 2012 at 11:37 AM, Bastian Ruppert
> <Bastian.Ruppert at sewerin.de> wrote:
>> From this commit id: b78375a806ed04eb22b963255cfdef8df702de47 auto
>> negotiation is enabled in RMII mode. Some boards based on da850 need
>> to suppress this procedure.
>>
>> CC: Rajashekhara, Sudhakar <sudhakar.raj at ti.com>
>> CC: Lad, Prabhakar <prabhakar.lad at ti.com>
>> CC: Hadli, Manjunath <manjunath.hadli at ti.com>
>> CC: sbabic at denx.de
>> Acked-by: Stefano Babic <sbabic at denx.de>
>> CC: Tom Rini <trini at ti.com>
>> Signed-off-by: Bastian Ruppert <Bastian.Ruppert at Sewerin.de>
>> ---
>>  drivers/net/davinci_emac.c |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
>> index b2516d1..fe988d7 100644
>> --- a/drivers/net/davinci_emac.c
>> +++ b/drivers/net/davinci_emac.c
>> @@ -897,7 +897,8 @@ int davinci_emac_initialize(void)
>>         }
>>
>>  #if defined(CONFIG_DRIVER_TI_EMAC_USE_RMII) && \
>> -               defined(CONFIG_MACH_DAVINCI_DA850_EVM)
>> +               defined(CONFIG_MACH_DAVINCI_DA850_EVM) && \
>> +                       !defined(CONFIG_DRIVER_TI_EMAC_RMII_NONEG)
> instead of CONFIG_DRIVER_TI_EMAC_RMII_NONEG why not have
> CONFIG_DRIVER_TI_EMAC_RMII_AUTO_NEGOTIATE ?

Good idea, opt-in is better than opt-out, please make it so.

-- 
Tom