[Top] [All Lists]

Re: au1000_tx_timeout and promiscuous mode

To: "elmar gerdes" <>
Subject: Re: au1000_tx_timeout and promiscuous mode
From: "Jordan Crouse" <>
Date: Fri, 17 Mar 2006 09:38:06 -0700
In-reply-to: <>
Original-recipient: rfc822;
References: <>
User-agent: Mutt/1.5.11
On 17/03/06 02:02 +0100, elmar gerdes wrote:
> hi folks,

Greetings Elmar.

> @@ -2070,6 +2070,7 @@
>       printk(KERN_ERR "%s: au1000_tx_timeout: dev=%p\n", dev->name, dev);
>       reset_mac(dev);
>       au1000_init(dev);
> +     set_rx_mode(dev);       // EG 2006-03-15: set promiscuous mode
>       dev->trans_start = jiffies;
>       netif_wake_queue(dev);

I would move the comment to the previous line, use standard /* */ notation,
and your name and the date isn't really needed, as that information will
be stored in the GIT log.

Also, don't forget your Signed-off-by line and a short description of the
patch for posterity.

Other then that, I have no problems with the bug - unless Pete wants to
object, I think you should send the fixed-up patch to 
and CC this list. 


Jordan Crouse
Senior Linux Engineer
AMD - Personal Connectivity Solutions Group

<Prev in Thread] Current Thread [Next in Thread>