Merge branch 'acpi-scan'

* acpi-scan: (30 commits)
  ACPI / scan: Fix acpi_bus_get_device() check in acpi_match_device()
  ACPI / scan: Make namespace scanning and trimming mutually exclusive
  ACPI / scan: Make it clear that acpi_bus_trim() cannot fail
  ACPI / scan: Drop acpi_bus_add() and use acpi_bus_scan() instead
  ACPI: update ej_event interface to take acpi_device
  ACPI / scan: Add second pass to acpi_bus_trim()
  ACPI / scan: Change the implementation of acpi_bus_trim()
  ACPI / scan: Drop the second argument of acpi_bus_trim()
  ACPI / scan: Drop the second argument of acpi_device_unregister()
  ACPI: Remove the ops field from struct acpi_device
  ACPI: remove unused acpi_op_bind and acpi_op_unbind
  ACPI / scan: Fix check of device_attach() return value.
  ACPI / scan: Treat power resources in a special way
  ACPI: Remove unused struct acpi_pci_root.id member
  ACPI: Drop ACPI device .bind() and .unbind() callbacks
  ACPI / PCI: Move the _PRT setup and cleanup code to pci-acpi.c
  ACPI / PCI: Rework the setup and cleanup of device wakeup
  ACPI: Add .setup() and .cleanup() callbacks to struct acpi_bus_type
  ACPI: Make acpi_bus_scan() and acpi_bus_add() take only one argument
  ACPI: Replace ACPI device add_type field with a match_driver flag
  ...

1 files changed, 35 insertions, 15 deletions

diff --git a/drivers/acpi/glue.c b/drivers/acpi/glue.c
index 35da18113216..9aee4fc2b218 100644
--- a/drivers/acpi/glue.c
+++ b/drivers/acpi/glue.c
@@ -68,6 +68,9 @@ static struct acpi_bus_type *acpi_get_bus_type(struct bus_type *type)
 {
         struct acpi_bus_type *tmp, *ret = NULL;
 
+        if (!type)
+                return NULL;
+
         down_read(&bus_type_sem);
         list_for_each_entry(tmp, &bus_type_list, list) {
                 if (tmp->bus == type) {
@@ -269,28 +272,39 @@ static int acpi_platform_notify(struct device *dev)
 {
         struct acpi_bus_type *type;
         acpi_handle handle;
-        int ret = -EINVAL;
+        int ret;
 
         ret = acpi_bind_one(dev, NULL);
-        if (!ret)
-                goto out;
-
-        if (!dev->bus || !dev->parent) {
+        if (ret && (!dev->bus || !dev->parent)) {
                 /* bridge devices genernally haven't bus or parent */
                 ret = acpi_find_bridge_device(dev, &handle);
-                goto end;
+                if (!ret) {
+                        ret = acpi_bind_one(dev, handle);
+                        if (ret)
+                                goto out;
+                }
         }
+
         type = acpi_get_bus_type(dev->bus);
-        if (!type) {
-                DBG("No ACPI bus support for %s\n", dev_name(dev));
-                ret = -EINVAL;
-                goto end;
+        if (ret) {
+                if (!type || !type->find_device) {
+                        DBG("No ACPI bus support for %s\n", dev_name(dev));
+                        ret = -EINVAL;
+                        goto out;
+                }
+
+                ret = type->find_device(dev, &handle);
+                if (ret) {
+                        DBG("Unable to get handle for %s\n", dev_name(dev));
+                        goto out;
+                }
+                ret = acpi_bind_one(dev, handle);
+                if (ret)
+                        goto out;
         }
-        if ((ret = type->find_device(dev, &handle)) != 0)
-                DBG("Can't get handler for %s\n", dev_name(dev));
- end:
-        if (!ret)
-                acpi_bind_one(dev, handle);
+
+        if (type && type->setup)
+                type->setup(dev);
 
  out:
 #if ACPI_GLUE_DEBUG
@@ -309,6 +323,12 @@ static int acpi_platform_notify(struct device *dev)
 
 static int acpi_platform_notify_remove(struct device *dev)
 {
+        struct acpi_bus_type *type;
+
+        type = acpi_get_bus_type(dev->bus);
+        if (type && type->cleanup)
+                type->cleanup(dev);
+
         acpi_unbind_one(dev);
         return 0;
 }