MIPS: NILE4: Remove odd locking in PCI config space access code
authorSergey Ryazanov <ryazanov.s.a@gmail.com>
Sat, 30 Aug 2014 02:06:24 +0000 (06:06 +0400)
committerRalf Baechle <ralf@linux-mips.org>
Mon, 24 Nov 2014 06:44:49 +0000 (07:44 +0100)
Caller (generic PCI code) already do proper locking so no need to add
another one here.

Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
Cc: Linux MIPS <linux-mips@linux-mips.org>
Patchwork: https://patchwork.linux-mips.org/patch/7600/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/pci/ops-nile4.c

index a1a7c9f4096e3fddb45ddd73dc06d1eadbc38bfb..b9d1fd0ff7e28a217380ede1e1c93adc8ef20b57 100644 (file)
@@ -13,8 +13,6 @@
 
 volatile unsigned long *const vrc_pciregs = (void *) Vrc5074_BASE;
 
-static DEFINE_SPINLOCK(nile4_pci_lock);
-
 static int nile4_pcibios_config_access(unsigned char access_type,
        struct pci_bus *bus, unsigned int devfn, int where, u32 *val)
 {
@@ -76,7 +74,6 @@ static int nile4_pcibios_config_access(unsigned char access_type,
 static int nile4_pcibios_read(struct pci_bus *bus, unsigned int devfn,
        int where, int size, u32 *val)
 {
-       unsigned long flags;
        u32 data = 0;
        int err;
 
@@ -85,11 +82,8 @@ static int nile4_pcibios_read(struct pci_bus *bus, unsigned int devfn,
        else if ((size == 4) && (where & 3))
                return PCIBIOS_BAD_REGISTER_NUMBER;
 
-       spin_lock_irqsave(&nile4_pci_lock, flags);
        err = nile4_pcibios_config_access(PCI_ACCESS_READ, bus, devfn, where,
-                                       &data);
-       spin_unlock_irqrestore(&nile4_pci_lock, flags);
-
+                                         &data);
        if (err)
                return err;
 
@@ -106,7 +100,6 @@ static int nile4_pcibios_read(struct pci_bus *bus, unsigned int devfn,
 static int nile4_pcibios_write(struct pci_bus *bus, unsigned int devfn,
        int where, int size, u32 val)
 {
-       unsigned long flags;
        u32 data = 0;
        int err;
 
@@ -115,11 +108,8 @@ static int nile4_pcibios_write(struct pci_bus *bus, unsigned int devfn,
        else if ((size == 4) && (where & 3))
                return PCIBIOS_BAD_REGISTER_NUMBER;
 
-       spin_lock_irqsave(&nile4_pci_lock, flags);
        err = nile4_pcibios_config_access(PCI_ACCESS_READ, bus, devfn, where,
                                          &data);
-       spin_unlock_irqrestore(&nile4_pci_lock, flags);
-
        if (err)
                return err;