1 files changed, 90 insertions, 20 deletions

diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index d9d531cce27f..3fac011f9cf9 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -59,7 +59,7 @@ static int create_modalias(struct acpi_device *acpi_dev, char *modalias,
                         count = snprintf(&modalias[len], size, "%s:",
                                          cid_list->id[i].value);
                         if (count < 0 || count >= size) {
-                                printk(KERN_ERR "acpi: %s cid[%i] exceeds event buffer size",
+                                printk(KERN_ERR PREFIX "%s cid[%i] exceeds event buffer size",
                                        acpi_dev->pnp.device_name, i);
                                 break;
                         }
@@ -453,7 +453,7 @@ static int acpi_device_register(struct acpi_device *device,
         device->dev.release = &acpi_device_release;
         result = device_add(&device->dev);
         if(result) {
-                printk("Error adding device %s", device->dev.bus_id);
+                printk(KERN_ERR PREFIX "Error adding device %s", device->dev.bus_id);
                 goto end;
         }
 
@@ -941,6 +941,15 @@ static int acpi_bay_match(struct acpi_device *device){
         return -ENODEV;
 }
 
+/*
+ * acpi_dock_match - see if a device has a _DCK method
+ */
+static int acpi_dock_match(struct acpi_device *device)
+{
+        acpi_handle tmp;
+        return acpi_get_handle(device->handle, "_DCK", &tmp);
+}
+
 static void acpi_device_set_id(struct acpi_device *device,
                                struct acpi_device *parent, acpi_handle handle,
                                int type)
@@ -950,13 +959,14 @@ static void acpi_device_set_id(struct acpi_device *device,
         char *hid = NULL;
         char *uid = NULL;
         struct acpi_compatible_id_list *cid_list = NULL;
+        const char *cid_add = NULL;
         acpi_status status;
 
         switch (type) {
         case ACPI_BUS_TYPE_DEVICE:
                 status = acpi_get_object_info(handle, &buffer);
                 if (ACPI_FAILURE(status)) {
-                        printk("%s: Error reading device info\n", __FUNCTION__);
+                        printk(KERN_ERR PREFIX "%s: Error reading device info\n", __FUNCTION__);
                         return;
                 }
 
@@ -972,15 +982,18 @@ static void acpi_device_set_id(struct acpi_device *device,
                         device->flags.bus_address = 1;
                 }
 
-                if(!(info->valid & (ACPI_VALID_HID | ACPI_VALID_CID))){
-                        status = acpi_video_bus_match(device);
-                        if(ACPI_SUCCESS(status))
-                                hid = ACPI_VIDEO_HID;
+                /* If we have a video/bay/dock device, add our selfdefined
+                   HID to the CID list. Like that the video/bay/dock drivers
+                   will get autoloaded and the device might still match
+                   against another driver.
+                */
+                if (ACPI_SUCCESS(acpi_video_bus_match(device)))
+                        cid_add = ACPI_VIDEO_HID;
+                else if (ACPI_SUCCESS(acpi_bay_match(device)))
+                        cid_add = ACPI_BAY_HID;
+                else if (ACPI_SUCCESS(acpi_dock_match(device)))
+                        cid_add = ACPI_DOCK_HID;
 
-                        status = acpi_bay_match(device);
-                        if (ACPI_SUCCESS(status))
-                                hid = ACPI_BAY_HID;
-                }
                 break;
         case ACPI_BUS_TYPE_POWER:
                 hid = ACPI_POWER_HID;
@@ -1021,12 +1034,45 @@ static void acpi_device_set_id(struct acpi_device *device,
                 strcpy(device->pnp.unique_id, uid);
                 device->flags.unique_id = 1;
         }
-        if (cid_list) {
-                device->pnp.cid_list = kmalloc(cid_list->size, GFP_KERNEL);
-                if (device->pnp.cid_list)
-                        memcpy(device->pnp.cid_list, cid_list, cid_list->size);
-                else
-                        printk(KERN_ERR "Memory allocation error\n");
+        if (cid_list || cid_add) {
+                struct  acpi_compatible_id_list *list;
+                int size = 0;
+                int count = 0;
+
+                if (cid_list) {
+                        size = cid_list->size;
+                } else if (cid_add) {
+                        size = sizeof(struct acpi_compatible_id_list);
+                        cid_list = ACPI_ALLOCATE_ZEROED((acpi_size) size);
+                        if (!cid_list) {
+                                printk(KERN_ERR "Memory allocation error\n");
+                                kfree(buffer.pointer);
+                                return;
+                        } else {
+                                cid_list->count = 0;
+                                cid_list->size = size;
+                        }
+                }
+                if (cid_add)
+                        size += sizeof(struct acpi_compatible_id);
+                list = kmalloc(size, GFP_KERNEL);
+
+                if (list) {
+                        if (cid_list) {
+                                memcpy(list, cid_list, cid_list->size);
+                                count = cid_list->count;
+                        }
+                        if (cid_add) {
+                                strncpy(list->id[count].value, cid_add,
+                                        ACPI_MAX_CID_LENGTH);
+                                count++;
+                                device->flags.compatible_ids = 1;
+                        }
+                        list->size = size;
+                        list->count = count;
+                        device->pnp.cid_list = list;
+                } else
+                        printk(KERN_ERR PREFIX "Memory allocation error\n");
         }
 
         kfree(buffer.pointer);
@@ -1050,7 +1096,7 @@ static int acpi_device_set_context(struct acpi_device *device, int type)
                                           acpi_bus_data_handler, device);
 
                 if (ACPI_FAILURE(status)) {
-                        printk("Error attaching device data\n");
+                        printk(KERN_ERR PREFIX "Error attaching device data\n");
                         result = -ENODEV;
                 }
         }
@@ -1081,6 +1127,20 @@ static int acpi_bus_remove(struct acpi_device *dev, int rmdevice)
 }
 
 static int
+acpi_is_child_device(struct acpi_device *device,
+                        int (*matcher)(struct acpi_device *))
+{
+        int result = -ENODEV;
+
+        do {
+                if (ACPI_SUCCESS(matcher(device)))
+                        return AE_OK;
+        } while ((device = device->parent));
+
+        return result;
+}
+
+static int
 acpi_add_single_object(struct acpi_device **child,
                        struct acpi_device *parent, acpi_handle handle, int type,
                         struct acpi_bus_ops *ops)
@@ -1131,10 +1191,20 @@ acpi_add_single_object(struct acpi_device **child,
         case ACPI_BUS_TYPE_PROCESSOR:
         case ACPI_BUS_TYPE_DEVICE:
                 result = acpi_bus_get_status(device);
-                if (ACPI_FAILURE(result) || !device->status.present) {
-                        result = -ENOENT;
+                if (ACPI_FAILURE(result)) {
+                        result = -ENODEV;
                         goto end;
                 }
+                if (!device->status.present) {
+                        /* Bay and dock should be handled even if absent */
+                        if (!ACPI_SUCCESS(
+                             acpi_is_child_device(device, acpi_bay_match)) &&
+                            !ACPI_SUCCESS(
+                             acpi_is_child_device(device, acpi_dock_match))) {
+                                        result = -ENODEV;
+                                        goto end;
+                        }
+                }
                 break;
         default:
                 STRUCT_TO_INT(device->status) =