[U-Boot] [PATCH] tsec.c bug: Every key typing in netconsole prints the Eth link mode

[Michael Zaidman]

Sorry for submitting of this patch. I proposed here similar to the
http://lists.denx.de/pipermail/u-boot/2010-January/067259.html patch
solution about which I was not aware. The only difference  is that I
also added  CONFIG_NETCONSOLE conditional compilation for the cases
when no netconsole is used at all.

On Mon, Feb 15, 2010 at 11:40 AM, Michael Zaidman
<michael.zaidman at gmail.com> wrote:
> The problem description:
> The Ethernet link operation mode printing (such as "Speed: 1000,
> full duplex") accomplishes every key typing when communicating via
> netconsole to the mpc834x based board. The string is printed by
> adjust_link() routine of tsec.c, which in turn is part of the
> eth_init routine called at every netloop entry.
>
> The solution:
> Do not print the string when compiled with CONFIG_NETCONSOLE and
> stdin or stdout is set to "nc"
>
> Signed-off-by: Michael Zaidman <michael.zaidman at gmail.com>
> ---
>  drivers/net/tsec.c |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/tsec.c b/drivers/net/tsec.c
> index 3f74118..87093e6 100644
> --- a/drivers/net/tsec.c
> +++ b/drivers/net/tsec.c
> @@ -847,10 +847,12 @@ static void adjust_link(struct eth_device *dev)
>                        printf("%s: Speed was bad\n", dev->name);
>                        break;
>                }
> -
> -               printf("Speed: %d, %s duplex\n", priv->speed,
> -                      (priv->duplexity) ? "full" : "half");
> -
> +#ifdef CONFIG_NETCONSOLE
> +               if ((strcmp(getenv("stdin"),"nc") != 0) &&
> +                       (strcmp(getenv("stdout"),"nc") != 0))
> +#endif
> +                       printf("Speed: %d, %s duplex\n", priv->speed,
> +                                  (priv->duplexity) ? "full" : "half");
>        } else {
>                printf("%s: No link.\n", dev->name);
>        }
> --
> 1.6.3.3
>
>