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.zaid...@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.zaid...@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 > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot