[netdrvr] typhoon: typhoon_resume - remove call to start_queue
authorGrant Grundler <grundler@parisc-linux.org>
Mon, 21 Apr 2008 04:44:15 +0000 (22:44 -0600)
committerJeff Garzik <jgarzik@redhat.com>
Fri, 25 Apr 2008 06:08:08 +0000 (02:08 -0400)
While trying to fix http://bugzilla.kernel.org/show_bug.cgi?id=8952
I looked at a few other drivers to figure out what drivers _should_
be doing for suspend/resume. I noticed typhoon driver is likely doing
more than it needs to.  Patch below is untested since I don't have the HW.

Suspend/resume code across NIC drivers is fairly inconsistent.
And I couldn't find any documentation on what the canonical sequence
NICs need to do for suspend or resume.  Is there any?

Barring contrary advice, I'm going model the tulip suspend/resume
fixes after tg3.c since a number of "modern" (< 5 years old) laptops
have that and I'm silly enough to assume it works.

Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/net/typhoon.c

index 333961bb7873973df7c75e5870aaee1f458ec47f..c0dd25ba7a18dcd9cc7969e85f1e02ce954af797 100644 (file)
@@ -2183,7 +2183,6 @@ typhoon_resume(struct pci_dev *pdev)
        }
 
        netif_device_attach(dev);
-       netif_start_queue(dev);
        return 0;
 
 reset: