linux 2.6.16.38 w/ vs2.0.3-rc1
[linux-2.6.git] / drivers / acpi / video.c
index 56666a9..bd48875 100644 (file)
@@ -117,7 +117,7 @@ struct acpi_video_enumerated_device {
 };
 
 struct acpi_video_bus {
-       struct acpi_device *device;
+       acpi_handle handle;
        u8 dos_setting;
        struct acpi_video_enumerated_device *attached_array;
        u8 attached_count;
@@ -155,6 +155,7 @@ struct acpi_video_device_brightness {
 };
 
 struct acpi_video_device {
+       acpi_handle handle;
        unsigned long device_id;
        struct acpi_video_device_flags flags;
        struct acpi_video_device_cap cap;
@@ -271,10 +272,10 @@ static int
 acpi_video_device_query(struct acpi_video_device *device, unsigned long *state)
 {
        int status;
+       ACPI_FUNCTION_TRACE("acpi_video_device_query");
+       status = acpi_evaluate_integer(device->handle, "_DGS", NULL, state);
 
-       status = acpi_evaluate_integer(device->dev->handle, "_DGS", NULL, state);
-
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -283,9 +284,11 @@ acpi_video_device_get_state(struct acpi_video_device *device,
 {
        int status;
 
-       status = acpi_evaluate_integer(device->dev->handle, "_DCS", NULL, state);
+       ACPI_FUNCTION_TRACE("acpi_video_device_get_state");
+
+       status = acpi_evaluate_integer(device->handle, "_DCS", NULL, state);
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -296,11 +299,12 @@ acpi_video_device_set_state(struct acpi_video_device *device, int state)
        struct acpi_object_list args = { 1, &arg0 };
        unsigned long ret;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_set_state");
 
        arg0.integer.value = state;
-       status = acpi_evaluate_integer(device->dev->handle, "_DSS", &args, &ret);
+       status = acpi_evaluate_integer(device->handle, "_DSS", &args, &ret);
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -311,27 +315,28 @@ acpi_video_device_lcd_query_levels(struct acpi_video_device *device,
        struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL };
        union acpi_object *obj;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_lcd_query_levels");
 
        *levels = NULL;
 
-       status = acpi_evaluate_object(device->dev->handle, "_BCL", NULL, &buffer);
+       status = acpi_evaluate_object(device->handle, "_BCL", NULL, &buffer);
        if (!ACPI_SUCCESS(status))
-               return status;
+               return_VALUE(status);
        obj = (union acpi_object *)buffer.pointer;
-       if (!obj || (obj->type != ACPI_TYPE_PACKAGE)) {
-               printk(KERN_ERR PREFIX "Invalid _BCL data\n");
+       if (!obj && (obj->type != ACPI_TYPE_PACKAGE)) {
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid _BCL data\n"));
                status = -EFAULT;
                goto err;
        }
 
        *levels = obj;
 
-       return 0;
+       return_VALUE(0);
 
       err:
        kfree(buffer.pointer);
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -341,12 +346,13 @@ acpi_video_device_lcd_set_level(struct acpi_video_device *device, int level)
        union acpi_object arg0 = { ACPI_TYPE_INTEGER };
        struct acpi_object_list args = { 1, &arg0 };
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_lcd_set_level");
 
        arg0.integer.value = level;
-       status = acpi_evaluate_object(device->dev->handle, "_BCM", &args, NULL);
+       status = acpi_evaluate_object(device->handle, "_BCM", &args, NULL);
 
        printk(KERN_DEBUG "set_level status: %x\n", status);
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -354,10 +360,11 @@ acpi_video_device_lcd_get_level_current(struct acpi_video_device *device,
                                        unsigned long *level)
 {
        int status;
+       ACPI_FUNCTION_TRACE("acpi_video_device_lcd_get_level_current");
 
-       status = acpi_evaluate_integer(device->dev->handle, "_BQC", NULL, level);
+       status = acpi_evaluate_integer(device->handle, "_BQC", NULL, level);
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -370,33 +377,34 @@ acpi_video_device_EDID(struct acpi_video_device *device,
        union acpi_object arg0 = { ACPI_TYPE_INTEGER };
        struct acpi_object_list args = { 1, &arg0 };
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_get_EDID");
 
        *edid = NULL;
 
        if (!device)
-               return -ENODEV;
+               return_VALUE(-ENODEV);
        if (length == 128)
                arg0.integer.value = 1;
        else if (length == 256)
                arg0.integer.value = 2;
        else
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
-       status = acpi_evaluate_object(device->dev->handle, "_DDC", &args, &buffer);
+       status = acpi_evaluate_object(device->handle, "_DDC", &args, &buffer);
        if (ACPI_FAILURE(status))
-               return -ENODEV;
+               return_VALUE(-ENODEV);
 
        obj = (union acpi_object *)buffer.pointer;
 
        if (obj && obj->type == ACPI_TYPE_BUFFER)
                *edid = obj;
        else {
-               printk(KERN_ERR PREFIX "Invalid _DDC data\n");
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid _DDC data\n"));
                status = -EFAULT;
                kfree(obj);
        }
 
-       return status;
+       return_VALUE(status);
 }
 
 /* bus */
@@ -409,14 +417,15 @@ acpi_video_bus_set_POST(struct acpi_video_bus *video, unsigned long option)
        union acpi_object arg0 = { ACPI_TYPE_INTEGER };
        struct acpi_object_list args = { 1, &arg0 };
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_set_POST");
 
        arg0.integer.value = option;
 
-       status = acpi_evaluate_integer(video->device->handle, "_SPD", &args, &tmp);
+       status = acpi_evaluate_integer(video->handle, "_SPD", &args, &tmp);
        if (ACPI_SUCCESS(status))
                status = tmp ? (-EINVAL) : (AE_OK);
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -424,9 +433,11 @@ acpi_video_bus_get_POST(struct acpi_video_bus *video, unsigned long *id)
 {
        int status;
 
-       status = acpi_evaluate_integer(video->device->handle, "_GPD", NULL, id);
+       ACPI_FUNCTION_TRACE("acpi_video_bus_get_POST");
 
-       return status;
+       status = acpi_evaluate_integer(video->handle, "_GPD", NULL, id);
+
+       return_VALUE(status);
 }
 
 static int
@@ -434,11 +445,12 @@ acpi_video_bus_POST_options(struct acpi_video_bus *video,
                            unsigned long *options)
 {
        int status;
+       ACPI_FUNCTION_TRACE("acpi_video_bus_POST_options");
 
-       status = acpi_evaluate_integer(video->device->handle, "_VPO", NULL, options);
+       status = acpi_evaluate_integer(video->handle, "_VPO", NULL, options);
        *options &= 3;
 
-       return status;
+       return_VALUE(status);
 }
 
 /*
@@ -469,6 +481,7 @@ acpi_video_bus_DOS(struct acpi_video_bus *video, int bios_flag, int lcd_flag)
        union acpi_object arg0 = { ACPI_TYPE_INTEGER };
        struct acpi_object_list args = { 1, &arg0 };
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_DOS");
 
        if (bios_flag < 0 || bios_flag > 3 || lcd_flag < 0 || lcd_flag > 1) {
                status = -1;
@@ -476,10 +489,10 @@ acpi_video_bus_DOS(struct acpi_video_bus *video, int bios_flag, int lcd_flag)
        }
        arg0.integer.value = (lcd_flag << 2) | bios_flag;
        video->dos_setting = arg0.integer.value;
-       acpi_evaluate_object(video->device->handle, "_DOS", &args, NULL);
+       acpi_evaluate_object(video->handle, "_DOS", &args, NULL);
 
       Failed:
-       return status;
+       return_VALUE(status);
 }
 
 /*
@@ -501,28 +514,29 @@ static void acpi_video_device_find_cap(struct acpi_video_device *device)
        union acpi_object *obj = NULL;
        struct acpi_video_device_brightness *br = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_find_cap");
 
        memset(&device->cap, 0, 4);
 
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_ADR", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_ADR", &h_dummy1))) {
                device->cap._ADR = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_BCL", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_BCL", &h_dummy1))) {
                device->cap._BCL = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_BCM", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_BCM", &h_dummy1))) {
                device->cap._BCM = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_DDC", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_DDC", &h_dummy1))) {
                device->cap._DDC = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_DCS", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_DCS", &h_dummy1))) {
                device->cap._DCS = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_DGS", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_DGS", &h_dummy1))) {
                device->cap._DGS = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(device->dev->handle, "_DSS", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_DSS", &h_dummy1))) {
                device->cap._DSS = 1;
        }
 
@@ -546,7 +560,8 @@ static void acpi_video_device_find_cap(struct acpi_video_device *device)
                                o = (union acpi_object *)&obj->package.
                                    elements[i];
                                if (o->type != ACPI_TYPE_INTEGER) {
-                                       printk(KERN_ERR PREFIX "Invalid data\n");
+                                       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                                         "Invalid data\n"));
                                        continue;
                                }
                                br->levels[count] = (u32) o->integer.value;
@@ -568,7 +583,7 @@ static void acpi_video_device_find_cap(struct acpi_video_device *device)
 
        kfree(obj);
 
-       return;
+       return_VOID;
 }
 
 /*
@@ -586,22 +601,22 @@ static void acpi_video_bus_find_cap(struct acpi_video_bus *video)
        acpi_handle h_dummy1;
 
        memset(&video->cap, 0, 4);
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_DOS", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_DOS", &h_dummy1))) {
                video->cap._DOS = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_DOD", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_DOD", &h_dummy1))) {
                video->cap._DOD = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_ROM", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_ROM", &h_dummy1))) {
                video->cap._ROM = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_GPD", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_GPD", &h_dummy1))) {
                video->cap._GPD = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_SPD", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_SPD", &h_dummy1))) {
                video->cap._SPD = 1;
        }
-       if (ACPI_SUCCESS(acpi_get_handle(video->device->handle, "_VPO", &h_dummy1))) {
+       if (ACPI_SUCCESS(acpi_get_handle(video->handle, "_VPO", &h_dummy1))) {
                video->cap._VPO = 1;
        }
 }
@@ -615,9 +630,10 @@ static int acpi_video_bus_check(struct acpi_video_bus *video)
 {
        acpi_status status = -ENOENT;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_check");
 
        if (!video)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        /* Since there is no HID, CID and so on for VGA driver, we have
         * to check well known required nodes.
@@ -641,7 +657,7 @@ static int acpi_video_bus_check(struct acpi_video_bus *video)
                status = 0;
        }
 
-       return status;
+       return_VALUE(status);
 }
 
 /* --------------------------------------------------------------------------
@@ -657,6 +673,7 @@ static int acpi_video_device_info_seq_show(struct seq_file *seq, void *offset)
        struct acpi_video_device *dev =
            (struct acpi_video_device *)seq->private;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_info_seq_show");
 
        if (!dev)
                goto end;
@@ -675,7 +692,7 @@ static int acpi_video_device_info_seq_show(struct seq_file *seq, void *offset)
        seq_printf(seq, "known by bios: %s\n", dev->flags.bios ? "yes" : "no");
 
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -692,6 +709,7 @@ static int acpi_video_device_state_seq_show(struct seq_file *seq, void *offset)
            (struct acpi_video_device *)seq->private;
        unsigned long state;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_state_seq_show");
 
        if (!dev)
                goto end;
@@ -711,7 +729,7 @@ static int acpi_video_device_state_seq_show(struct seq_file *seq, void *offset)
                seq_printf(seq, "<not supported>\n");
 
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -732,12 +750,13 @@ acpi_video_device_write_state(struct file *file,
        char str[12] = { 0 };
        u32 state = 0;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_write_state");
 
        if (!dev || count + 1 > sizeof str)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        if (copy_from_user(str, buffer, count))
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        str[count] = 0;
        state = simple_strtoul(str, NULL, 0);
@@ -746,9 +765,9 @@ acpi_video_device_write_state(struct file *file,
        status = acpi_video_device_set_state(dev, state);
 
        if (status)
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
-       return count;
+       return_VALUE(count);
 }
 
 static int
@@ -758,10 +777,11 @@ acpi_video_device_brightness_seq_show(struct seq_file *seq, void *offset)
            (struct acpi_video_device *)seq->private;
        int i;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_brightness_seq_show");
 
        if (!dev || !dev->brightness) {
                seq_printf(seq, "<not supported>\n");
-               return 0;
+               return_VALUE(0);
        }
 
        seq_printf(seq, "levels: ");
@@ -769,7 +789,7 @@ acpi_video_device_brightness_seq_show(struct seq_file *seq, void *offset)
                seq_printf(seq, " %d", dev->brightness->levels[i]);
        seq_printf(seq, "\ncurrent: %d\n", dev->brightness->curr);
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -790,18 +810,19 @@ acpi_video_device_write_brightness(struct file *file,
        unsigned int level = 0;
        int i;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_write_brightness");
 
        if (!dev || !dev->brightness || count + 1 > sizeof str)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        if (copy_from_user(str, buffer, count))
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        str[count] = 0;
        level = simple_strtoul(str, NULL, 0);
 
        if (level > 100)
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        /* validate though the list of available levels */
        for (i = 0; i < dev->brightness->count; i++)
@@ -812,7 +833,7 @@ acpi_video_device_write_brightness(struct file *file,
                        break;
                }
 
-       return count;
+       return_VALUE(count);
 }
 
 static int acpi_video_device_EDID_seq_show(struct seq_file *seq, void *offset)
@@ -823,6 +844,7 @@ static int acpi_video_device_EDID_seq_show(struct seq_file *seq, void *offset)
        int i;
        union acpi_object *edid = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_EDID_seq_show");
 
        if (!dev)
                goto out;
@@ -847,7 +869,7 @@ static int acpi_video_device_EDID_seq_show(struct seq_file *seq, void *offset)
        else
                kfree(edid);
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -862,26 +884,28 @@ static int acpi_video_device_add_fs(struct acpi_device *device)
        struct proc_dir_entry *entry = NULL;
        struct acpi_video_device *vid_dev;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_add_fs");
 
        if (!device)
-               return -ENODEV;
+               return_VALUE(-ENODEV);
 
        vid_dev = (struct acpi_video_device *)acpi_driver_data(device);
        if (!vid_dev)
-               return -ENODEV;
+               return_VALUE(-ENODEV);
 
        if (!acpi_device_dir(device)) {
                acpi_device_dir(device) = proc_mkdir(acpi_device_bid(device),
                                                     vid_dev->video->dir);
                if (!acpi_device_dir(device))
-                       return -ENODEV;
+                       return_VALUE(-ENODEV);
                acpi_device_dir(device)->owner = THIS_MODULE;
        }
 
        /* 'info' [R] */
        entry = create_proc_entry("info", S_IRUGO, acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'info' fs entry\n"));
        else {
                entry->proc_fops = &acpi_video_device_info_fops;
                entry->data = acpi_driver_data(device);
@@ -893,7 +917,8 @@ static int acpi_video_device_add_fs(struct acpi_device *device)
            create_proc_entry("state", S_IFREG | S_IRUGO | S_IWUSR,
                              acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'state' fs entry\n"));
        else {
                acpi_video_device_state_fops.write = acpi_video_device_write_state;
                entry->proc_fops = &acpi_video_device_state_fops;
@@ -906,7 +931,8 @@ static int acpi_video_device_add_fs(struct acpi_device *device)
            create_proc_entry("brightness", S_IFREG | S_IRUGO | S_IWUSR,
                              acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'brightness' fs entry\n"));
        else {
                acpi_video_device_brightness_fops.write = acpi_video_device_write_brightness;
                entry->proc_fops = &acpi_video_device_brightness_fops;
@@ -917,23 +943,25 @@ static int acpi_video_device_add_fs(struct acpi_device *device)
        /* 'EDID' [R] */
        entry = create_proc_entry("EDID", S_IRUGO, acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'brightness' fs entry\n"));
        else {
                entry->proc_fops = &acpi_video_device_EDID_fops;
                entry->data = acpi_driver_data(device);
                entry->owner = THIS_MODULE;
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_device_remove_fs(struct acpi_device *device)
 {
        struct acpi_video_device *vid_dev;
+       ACPI_FUNCTION_TRACE("acpi_video_device_remove_fs");
 
        vid_dev = (struct acpi_video_device *)acpi_driver_data(device);
        if (!vid_dev || !vid_dev->video || !vid_dev->video->dir)
-               return -ENODEV;
+               return_VALUE(-ENODEV);
 
        if (acpi_device_dir(device)) {
                remove_proc_entry("info", acpi_device_dir(device));
@@ -944,7 +972,7 @@ static int acpi_video_device_remove_fs(struct acpi_device *device)
                acpi_device_dir(device) = NULL;
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 /* video bus */
@@ -952,6 +980,7 @@ static int acpi_video_bus_info_seq_show(struct seq_file *seq, void *offset)
 {
        struct acpi_video_bus *video = (struct acpi_video_bus *)seq->private;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_info_seq_show");
 
        if (!video)
                goto end;
@@ -964,7 +993,7 @@ static int acpi_video_bus_info_seq_show(struct seq_file *seq, void *offset)
                   video->flags.post ? "yes" : "no");
 
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_info_open_fs(struct inode *inode, struct file *file)
@@ -977,6 +1006,7 @@ static int acpi_video_bus_ROM_seq_show(struct seq_file *seq, void *offset)
 {
        struct acpi_video_bus *video = (struct acpi_video_bus *)seq->private;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_ROM_seq_show");
 
        if (!video)
                goto end;
@@ -985,7 +1015,7 @@ static int acpi_video_bus_ROM_seq_show(struct seq_file *seq, void *offset)
        seq_printf(seq, "<TODO>\n");
 
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_ROM_open_fs(struct inode *inode, struct file *file)
@@ -999,6 +1029,7 @@ static int acpi_video_bus_POST_info_seq_show(struct seq_file *seq, void *offset)
        unsigned long options;
        int status;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_POST_info_seq_show");
 
        if (!video)
                goto end;
@@ -1021,7 +1052,7 @@ static int acpi_video_bus_POST_info_seq_show(struct seq_file *seq, void *offset)
        } else
                seq_printf(seq, "<not supported>\n");
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -1037,6 +1068,7 @@ static int acpi_video_bus_POST_seq_show(struct seq_file *seq, void *offset)
        int status;
        unsigned long id;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_POST_seq_show");
 
        if (!video)
                goto end;
@@ -1049,17 +1081,18 @@ static int acpi_video_bus_POST_seq_show(struct seq_file *seq, void *offset)
        seq_printf(seq, "device posted is <%s>\n", device_decode[id & 3]);
 
       end:
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_DOS_seq_show(struct seq_file *seq, void *offset)
 {
        struct acpi_video_bus *video = (struct acpi_video_bus *)seq->private;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_DOS_seq_show");
 
        seq_printf(seq, "DOS setting: <%d>\n", video->dos_setting);
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_POST_open_fs(struct inode *inode, struct file *file)
@@ -1084,21 +1117,22 @@ acpi_video_bus_write_POST(struct file *file,
        char str[12] = { 0 };
        unsigned long opt, options;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_write_POST");
 
        if (!video || count + 1 > sizeof str)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        status = acpi_video_bus_POST_options(video, &options);
        if (!ACPI_SUCCESS(status))
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        if (copy_from_user(str, buffer, count))
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        str[count] = 0;
        opt = strtoul(str, NULL, 0);
        if (opt > 3)
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        /* just in case an OEM 'forget' the motherboard... */
        options |= 1;
@@ -1106,11 +1140,11 @@ acpi_video_bus_write_POST(struct file *file,
        if (options & (1ul << opt)) {
                status = acpi_video_bus_set_POST(video, opt);
                if (!ACPI_SUCCESS(status))
-                       return -EFAULT;
+                       return_VALUE(-EFAULT);
 
        }
 
-       return count;
+       return_VALUE(count);
 }
 
 static ssize_t
@@ -1124,24 +1158,25 @@ acpi_video_bus_write_DOS(struct file *file,
        char str[12] = { 0 };
        unsigned long opt;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_write_DOS");
 
        if (!video || count + 1 > sizeof str)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        if (copy_from_user(str, buffer, count))
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        str[count] = 0;
        opt = strtoul(str, NULL, 0);
        if (opt > 7)
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
        status = acpi_video_bus_DOS(video, opt & 0x3, (opt & 0x4) >> 2);
 
        if (!ACPI_SUCCESS(status))
-               return -EFAULT;
+               return_VALUE(-EFAULT);
 
-       return count;
+       return_VALUE(count);
 }
 
 static int acpi_video_bus_add_fs(struct acpi_device *device)
@@ -1149,6 +1184,7 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
        struct proc_dir_entry *entry = NULL;
        struct acpi_video_bus *video;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_add_fs");
 
        video = (struct acpi_video_bus *)acpi_driver_data(device);
 
@@ -1156,7 +1192,7 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
                acpi_device_dir(device) = proc_mkdir(acpi_device_bid(device),
                                                     acpi_video_dir);
                if (!acpi_device_dir(device))
-                       return -ENODEV;
+                       return_VALUE(-ENODEV);
                video->dir = acpi_device_dir(device);
                acpi_device_dir(device)->owner = THIS_MODULE;
        }
@@ -1164,7 +1200,8 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
        /* 'info' [R] */
        entry = create_proc_entry("info", S_IRUGO, acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'info' fs entry\n"));
        else {
                entry->proc_fops = &acpi_video_bus_info_fops;
                entry->data = acpi_driver_data(device);
@@ -1174,7 +1211,8 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
        /* 'ROM' [R] */
        entry = create_proc_entry("ROM", S_IRUGO, acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'ROM' fs entry\n"));
        else {
                entry->proc_fops = &acpi_video_bus_ROM_fops;
                entry->data = acpi_driver_data(device);
@@ -1185,7 +1223,8 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
        entry =
            create_proc_entry("POST_info", S_IRUGO, acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'POST_info' fs entry\n"));
        else {
                entry->proc_fops = &acpi_video_bus_POST_info_fops;
                entry->data = acpi_driver_data(device);
@@ -1197,7 +1236,8 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
            create_proc_entry("POST", S_IFREG | S_IRUGO | S_IRUSR,
                              acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'POST' fs entry\n"));
        else {
                acpi_video_bus_POST_fops.write = acpi_video_bus_write_POST;
                entry->proc_fops = &acpi_video_bus_POST_fops;
@@ -1210,7 +1250,8 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
            create_proc_entry("DOS", S_IFREG | S_IRUGO | S_IRUSR,
                              acpi_device_dir(device));
        if (!entry)
-               return -ENODEV;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Unable to create 'DOS' fs entry\n"));
        else {
                acpi_video_bus_DOS_fops.write = acpi_video_bus_write_DOS;
                entry->proc_fops = &acpi_video_bus_DOS_fops;
@@ -1218,13 +1259,14 @@ static int acpi_video_bus_add_fs(struct acpi_device *device)
                entry->owner = THIS_MODULE;
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_remove_fs(struct acpi_device *device)
 {
        struct acpi_video_bus *video;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_remove_fs");
 
        video = (struct acpi_video_bus *)acpi_driver_data(device);
 
@@ -1238,7 +1280,7 @@ static int acpi_video_bus_remove_fs(struct acpi_device *device)
                acpi_device_dir(device) = NULL;
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 /* --------------------------------------------------------------------------
@@ -1252,12 +1294,13 @@ acpi_video_bus_get_one_device(struct acpi_device *device,
                              struct acpi_video_bus *video)
 {
        unsigned long device_id;
-       int status;
+       int status, result;
        struct acpi_video_device *data;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_get_one_device");
 
        if (!device || !video)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        status =
            acpi_evaluate_integer(device->handle, "_ADR", NULL, &device_id);
@@ -1265,10 +1308,11 @@ acpi_video_bus_get_one_device(struct acpi_device *device,
 
                data = kmalloc(sizeof(struct acpi_video_device), GFP_KERNEL);
                if (!data)
-                       return -ENOMEM;
+                       return_VALUE(-ENOMEM);
 
                memset(data, 0, sizeof(struct acpi_video_device));
 
+               data->handle = device->handle;
                strcpy(acpi_device_name(device), ACPI_VIDEO_DEVICE_NAME);
                strcpy(acpi_device_class(device), ACPI_VIDEO_CLASS);
                acpi_driver_data(device) = data;
@@ -1295,18 +1339,15 @@ acpi_video_bus_get_one_device(struct acpi_device *device,
                acpi_video_device_bind(video, data);
                acpi_video_device_find_cap(data);
 
-               status = acpi_install_notify_handler(device->handle,
+               status = acpi_install_notify_handler(data->handle,
                                                     ACPI_DEVICE_NOTIFY,
                                                     acpi_video_device_notify,
                                                     data);
                if (ACPI_FAILURE(status)) {
                        ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
                                          "Error installing notify handler\n"));
-                       if(data->brightness)
-                               kfree(data->brightness->levels);
-                       kfree(data->brightness);
-                       kfree(data);
-                       return -ENODEV;
+                       result = -ENODEV;
+                       goto end;
                }
 
                down(&video->sem);
@@ -1315,10 +1356,11 @@ acpi_video_bus_get_one_device(struct acpi_device *device,
 
                acpi_video_device_add_fs(device);
 
-               return 0;
+               return_VALUE(0);
        }
 
-       return -ENOENT;
+      end:
+       return_VALUE(-ENOENT);
 }
 
 /*
@@ -1361,6 +1403,7 @@ acpi_video_device_bind(struct acpi_video_bus *video,
                       struct acpi_video_device *device)
 {
        int i;
+       ACPI_FUNCTION_TRACE("acpi_video_device_bind");
 
 #define IDS_VAL(i) video->attached_array[i].value.int_val
 #define IDS_BIND(i) video->attached_array[i].bind_info
@@ -1397,15 +1440,17 @@ static int acpi_video_device_enumerate(struct acpi_video_bus *video)
        union acpi_object *dod = NULL;
        union acpi_object *obj;
 
-       status = acpi_evaluate_object(video->device->handle, "_DOD", NULL, &buffer);
+       ACPI_FUNCTION_TRACE("acpi_video_device_enumerate");
+
+       status = acpi_evaluate_object(video->handle, "_DOD", NULL, &buffer);
        if (!ACPI_SUCCESS(status)) {
-               ACPI_EXCEPTION((AE_INFO, status, "Evaluating _DOD"));
-               return status;
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Error evaluating _DOD\n"));
+               return_VALUE(status);
        }
 
        dod = (union acpi_object *)buffer.pointer;
        if (!dod || (dod->type != ACPI_TYPE_PACKAGE)) {
-               ACPI_EXCEPTION((AE_INFO, status, "Invalid _DOD data"));
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid _DOD data\n"));
                status = -EFAULT;
                goto out;
        }
@@ -1429,7 +1474,8 @@ static int acpi_video_device_enumerate(struct acpi_video_bus *video)
                obj = (union acpi_object *)&dod->package.elements[i];
 
                if (obj->type != ACPI_TYPE_INTEGER) {
-                       printk(KERN_ERR PREFIX "Invalid _DOD data\n");
+                       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                         "Invalid _DOD data\n"));
                        active_device_list[i].value.int_val =
                            ACPI_VIDEO_HEAD_INVALID;
                }
@@ -1446,8 +1492,8 @@ static int acpi_video_device_enumerate(struct acpi_video_bus *video)
        video->attached_array = active_device_list;
        video->attached_count = count;
       out:
-       kfree(buffer.pointer);
-       return status;
+       acpi_os_free(buffer.pointer);
+       return_VALUE(status);
 }
 
 /*
@@ -1472,6 +1518,7 @@ static int acpi_video_switch_output(struct acpi_video_bus *video, int event)
        unsigned long state;
        int status = 0;
 
+       ACPI_FUNCTION_TRACE("acpi_video_switch_output");
 
        list_for_each_safe(node, next, &video->video_device_list) {
                dev = container_of(node, struct acpi_video_device, entry);
@@ -1502,7 +1549,7 @@ static int acpi_video_switch_output(struct acpi_video_bus *video, int event)
                break;
        }
 
-       return status;
+       return_VALUE(status);
 }
 
 static int
@@ -1529,6 +1576,7 @@ acpi_video_bus_get_devices(struct acpi_video_bus *video,
        int status = 0;
        struct list_head *node, *next;
 
+       ACPI_FUNCTION_TRACE("acpi_video_get_devices");
 
        acpi_video_device_enumerate(video);
 
@@ -1541,12 +1589,13 @@ acpi_video_bus_get_devices(struct acpi_video_bus *video,
 
                status = acpi_video_bus_get_one_device(dev, video);
                if (ACPI_FAILURE(status)) {
-                       ACPI_EXCEPTION((AE_INFO, status, "Cant attach device"));
+                       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                         "Cant attach device\n"));
                        continue;
                }
 
        }
-       return status;
+       return_VALUE(status);
 }
 
 static int acpi_video_bus_put_one_device(struct acpi_video_device *device)
@@ -1554,9 +1603,10 @@ static int acpi_video_bus_put_one_device(struct acpi_video_device *device)
        acpi_status status;
        struct acpi_video_bus *video;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_put_one_device");
 
        if (!device || !device->video)
-               return -ENOENT;
+               return_VALUE(-ENOENT);
 
        video = device->video;
 
@@ -1565,11 +1615,14 @@ static int acpi_video_bus_put_one_device(struct acpi_video_device *device)
        up(&video->sem);
        acpi_video_device_remove_fs(device->dev);
 
-       status = acpi_remove_notify_handler(device->dev->handle,
+       status = acpi_remove_notify_handler(device->handle,
                                            ACPI_DEVICE_NOTIFY,
                                            acpi_video_device_notify);
+       if (ACPI_FAILURE(status))
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Error removing notify handler\n"));
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int acpi_video_bus_put_devices(struct acpi_video_bus *video)
@@ -1577,6 +1630,7 @@ static int acpi_video_bus_put_devices(struct acpi_video_bus *video)
        int status;
        struct list_head *node, *next;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_put_devices");
 
        list_for_each_safe(node, next, &video->video_device_list) {
                struct acpi_video_device *data =
@@ -1589,13 +1643,12 @@ static int acpi_video_bus_put_devices(struct acpi_video_bus *video)
                        printk(KERN_WARNING PREFIX
                               "hhuuhhuu bug in acpi video driver.\n");
 
-               if (data->brightness)
-                       kfree(data->brightness->levels);
                kfree(data->brightness);
+
                kfree(data);
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 /* acpi_video interface */
@@ -1615,12 +1668,14 @@ static void acpi_video_bus_notify(acpi_handle handle, u32 event, void *data)
        struct acpi_video_bus *video = (struct acpi_video_bus *)data;
        struct acpi_device *device = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_notify");
        printk("video bus notify\n");
 
        if (!video)
-               return;
+               return_VOID;
 
-       device = video->device;
+       if (acpi_bus_get_device(handle, &device))
+               return_VOID;
 
        switch (event) {
        case ACPI_VIDEO_NOTIFY_SWITCH:  /* User request that a switch occur,
@@ -1649,7 +1704,7 @@ static void acpi_video_bus_notify(acpi_handle handle, u32 event, void *data)
                break;
        }
 
-       return;
+       return_VOID;
 }
 
 static void acpi_video_device_notify(acpi_handle handle, u32 event, void *data)
@@ -1658,12 +1713,14 @@ static void acpi_video_device_notify(acpi_handle handle, u32 event, void *data)
            (struct acpi_video_device *)data;
        struct acpi_device *device = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_device_notify");
 
        printk("video device notify\n");
        if (!video_device)
-               return;
+               return_VOID;
 
-       device = video_device->dev;
+       if (acpi_bus_get_device(handle, &device))
+               return_VOID;
 
        switch (event) {
        case ACPI_VIDEO_NOTIFY_SWITCH:  /* change in status (cycle output device) */
@@ -1683,7 +1740,7 @@ static void acpi_video_device_notify(acpi_handle handle, u32 event, void *data)
                                  "Unsupported event [0x%x]\n", event));
                break;
        }
-       return;
+       return_VOID;
 }
 
 static int acpi_video_bus_add(struct acpi_device *device)
@@ -1692,16 +1749,17 @@ static int acpi_video_bus_add(struct acpi_device *device)
        acpi_status status = 0;
        struct acpi_video_bus *video = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_add");
 
        if (!device)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        video = kmalloc(sizeof(struct acpi_video_bus), GFP_KERNEL);
        if (!video)
-               return -ENOMEM;
+               return_VALUE(-ENOMEM);
        memset(video, 0, sizeof(struct acpi_video_bus));
 
-       video->device = device;
+       video->handle = device->handle;
        strcpy(acpi_device_name(device), ACPI_VIDEO_BUS_NAME);
        strcpy(acpi_device_class(device), ACPI_VIDEO_CLASS);
        acpi_driver_data(device) = video;
@@ -1721,16 +1779,12 @@ static int acpi_video_bus_add(struct acpi_device *device)
        acpi_video_bus_get_devices(video, device);
        acpi_video_bus_start_devices(video);
 
-       status = acpi_install_notify_handler(device->handle,
+       status = acpi_install_notify_handler(video->handle,
                                             ACPI_DEVICE_NOTIFY,
                                             acpi_video_bus_notify, video);
        if (ACPI_FAILURE(status)) {
                ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
                                  "Error installing notify handler\n"));
-               acpi_video_bus_stop_devices(video);
-               acpi_video_bus_put_devices(video);
-               kfree(video->attached_array);
-               acpi_video_bus_remove_fs(device);
                result = -ENODEV;
                goto end;
        }
@@ -1742,10 +1796,12 @@ static int acpi_video_bus_add(struct acpi_device *device)
               video->flags.post ? "yes" : "no");
 
       end:
-       if (result)
+       if (result) {
+               acpi_video_bus_remove_fs(device);
                kfree(video);
+       }
 
-       return result;
+       return_VALUE(result);
 }
 
 static int acpi_video_bus_remove(struct acpi_device *device, int type)
@@ -1753,17 +1809,21 @@ static int acpi_video_bus_remove(struct acpi_device *device, int type)
        acpi_status status = 0;
        struct acpi_video_bus *video = NULL;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_remove");
 
        if (!device || !acpi_driver_data(device))
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        video = (struct acpi_video_bus *)acpi_driver_data(device);
 
        acpi_video_bus_stop_devices(video);
 
-       status = acpi_remove_notify_handler(video->device->handle,
+       status = acpi_remove_notify_handler(video->handle,
                                            ACPI_DEVICE_NOTIFY,
                                            acpi_video_bus_notify);
+       if (ACPI_FAILURE(status))
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                                 "Error removing notify handler\n"));
 
        acpi_video_bus_put_devices(video);
        acpi_video_bus_remove_fs(device);
@@ -1771,7 +1831,7 @@ static int acpi_video_bus_remove(struct acpi_device *device, int type)
        kfree(video->attached_array);
        kfree(video);
 
-       return 0;
+       return_VALUE(0);
 }
 
 static int
@@ -1781,9 +1841,10 @@ acpi_video_bus_match(struct acpi_device *device, struct acpi_driver *driver)
        acpi_handle h_dummy2;
        acpi_handle h_dummy3;
 
+       ACPI_FUNCTION_TRACE("acpi_video_bus_match");
 
        if (!device || !driver)
-               return -EINVAL;
+               return_VALUE(-EINVAL);
 
        /* Since there is no HID, CID for ACPI Video drivers, we have
         * to check well known required nodes for each feature we support.
@@ -1792,25 +1853,26 @@ acpi_video_bus_match(struct acpi_device *device, struct acpi_driver *driver)
        /* Does this device able to support video switching ? */
        if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_DOD", &h_dummy1)) &&
            ACPI_SUCCESS(acpi_get_handle(device->handle, "_DOS", &h_dummy2)))
-               return 0;
+               return_VALUE(0);
 
        /* Does this device able to retrieve a video ROM ? */
        if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_ROM", &h_dummy1)))
-               return 0;
+               return_VALUE(0);
 
        /* Does this device able to configure which video head to be POSTed ? */
        if (ACPI_SUCCESS(acpi_get_handle(device->handle, "_VPO", &h_dummy1)) &&
            ACPI_SUCCESS(acpi_get_handle(device->handle, "_GPD", &h_dummy2)) &&
            ACPI_SUCCESS(acpi_get_handle(device->handle, "_SPD", &h_dummy3)))
-               return 0;
+               return_VALUE(0);
 
-       return -ENODEV;
+       return_VALUE(-ENODEV);
 }
 
 static int __init acpi_video_init(void)
 {
        int result = 0;
 
+       ACPI_FUNCTION_TRACE("acpi_video_init");
 
        /*
           acpi_dbg_level = 0xFFFFFFFF;
@@ -1819,26 +1881,27 @@ static int __init acpi_video_init(void)
 
        acpi_video_dir = proc_mkdir(ACPI_VIDEO_CLASS, acpi_root_dir);
        if (!acpi_video_dir)
-               return -ENODEV;
+               return_VALUE(-ENODEV);
        acpi_video_dir->owner = THIS_MODULE;
 
        result = acpi_bus_register_driver(&acpi_video_bus);
        if (result < 0) {
                remove_proc_entry(ACPI_VIDEO_CLASS, acpi_root_dir);
-               return -ENODEV;
+               return_VALUE(-ENODEV);
        }
 
-       return 0;
+       return_VALUE(0);
 }
 
 static void __exit acpi_video_exit(void)
 {
+       ACPI_FUNCTION_TRACE("acpi_video_exit");
 
        acpi_bus_unregister_driver(&acpi_video_bus);
 
        remove_proc_entry(ACPI_VIDEO_CLASS, acpi_root_dir);
 
-       return;
+       return_VOID;
 }
 
 module_init(acpi_video_init);