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

Prabhakar Lad prabhakar.csengg at gmail.com
Fri Sep 7 10:08:34 CEST 2012


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 ?

Thx,
--Prabhakar Lad

>         for (i = 0; i < num_phy; i++) {
>                 if (phy[i].is_phy_connected(i))
>                         phy[i].auto_negotiate(i);
> --
> 1.7.0.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list