PCI/ACPI: Remove unused global list acpi_pci_roots
authorJiang Liu <jiang.liu@huawei.com>
Tue, 28 May 2013 23:59:25 +0000 (17:59 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Wed, 29 May 2013 20:35:49 +0000 (14:35 -0600)
Now the global list acpi_pci_roots pci_root.c is useless, remove it.

Signed-off-by: Jiang Liu <jiang.liu@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Len Brown <lenb@kernel.org>
drivers/acpi/pci_root.c
include/acpi/acpi_bus.h

index 852fb96f94e5c91dbe6fc325f7370d0a978f6a9f..18b7ab2dee4fe52b934f670dae255e9e74a7af98 100644 (file)
@@ -65,10 +65,6 @@ static struct acpi_scan_handler pci_root_handler = {
        .detach = acpi_pci_root_remove,
 };
 
-/* Lock to protect both acpi_pci_roots lists */
-static DEFINE_MUTEX(acpi_pci_root_lock);
-static LIST_HEAD(acpi_pci_roots);
-
 static DEFINE_MUTEX(osc_lock);
 
 /**
@@ -423,7 +419,6 @@ static int acpi_pci_root_add(struct acpi_device *device,
                }
        }
 
-       INIT_LIST_HEAD(&root->node);
        root->device = device;
        root->segment = segment & 0xFFFF;
        strcpy(acpi_device_name(device), ACPI_PCI_ROOT_DEVICE_NAME);
@@ -447,10 +442,6 @@ static int acpi_pci_root_add(struct acpi_device *device,
         * TBD: Need PCI interface for enumeration/configuration of roots.
         */
 
-       mutex_lock(&acpi_pci_root_lock);
-       list_add_tail(&root->node, &acpi_pci_roots);
-       mutex_unlock(&acpi_pci_root_lock);
-
        /*
         * Scan the Root Bridge
         * --------------------
@@ -464,7 +455,7 @@ static int acpi_pci_root_add(struct acpi_device *device,
                            "Bus %04x:%02x not present in PCI namespace\n",
                            root->segment, (unsigned int)root->secondary.start);
                result = -ENODEV;
-               goto out_del_root;
+               goto end;
        }
 
        /* Indicate support for various _OSC capabilities. */
@@ -545,11 +536,6 @@ static int acpi_pci_root_add(struct acpi_device *device,
        pci_bus_add_devices(root->bus);
        return 1;
 
-out_del_root:
-       mutex_lock(&acpi_pci_root_lock);
-       list_del(&root->node);
-       mutex_unlock(&acpi_pci_root_lock);
-
 end:
        kfree(root);
        return result;
@@ -566,9 +552,6 @@ static void acpi_pci_root_remove(struct acpi_device *device)
 
        pci_remove_root_bus(root->bus);
 
-       mutex_lock(&acpi_pci_root_lock);
-       list_del(&root->node);
-       mutex_unlock(&acpi_pci_root_lock);
        kfree(root);
 }
 
index 98db31d9f9b410908b2056b192bf9813bc8ce4aa..af0f840f77c8c55680afa4a0a1518a19890c7006 100644 (file)
@@ -472,7 +472,6 @@ int register_acpi_bus_type(struct acpi_bus_type *);
 int unregister_acpi_bus_type(struct acpi_bus_type *);
 
 struct acpi_pci_root {
-       struct list_head node;
        struct acpi_device * device;
        struct pci_bus *bus;
        u16 segment;