[netdrvr] au1000_eth: Spinlock initialisation fix
authorMartin Gebert <martin.gebert@alpha-bit.de>
Wed, 23 Jul 2008 07:40:09 +0000 (09:40 +0200)
committerJeff Garzik <jgarzik@redhat.com>
Wed, 24 Sep 2008 22:48:58 +0000 (18:48 -0400)
Seems like the spinlock for the AU1x00 ethernet device is initialised too
late, as it is already used in enable_mac(), which is called via
mii_probe() before the init takes place.

The attached patch is working here for a Linux Au1100 2.6.22.6 kernel,
and as far as I checked should also be applicable to the current head
(just line numbers differ).

Signed-off-by: Martin Gebert <Martin.Gebert@alpha-bit.de>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/net/au1000_eth.c

index 5ee1b05..92c16c3 100644 (file)
@@ -653,6 +653,8 @@ static struct net_device * au1000_probe(int port_num)
 
        aup = dev->priv;
 
+       spin_lock_init(&aup->lock);
+
        /* Allocate the data buffers */
        /* Snooping works fine with eth on all au1xxx */
        aup->vaddr = (u32)dma_alloc_noncoherent(NULL, MAX_BUF_SIZE *
@@ -753,7 +755,6 @@ static struct net_device * au1000_probe(int port_num)
                aup->tx_db_inuse[i] = pDB;
        }
 
-       spin_lock_init(&aup->lock);
        dev->base_addr = base;
        dev->irq = irq;
        dev->open = au1000_open;