lkml.org 
[lkml]   [2012]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ACPI : Implement ACPI_NOTIFY_EJECT_REQUEST for ACPI container
Please ignore the patch.
I found that ACPI container driver does not need to implement EJECT routine
because it has already prepared eject file at userland.

2012/06/05 15:26, Yasuaki Ishimatsu wrote:
> Even if ACPI_NOTIFY_EJECT_REQUEST is notified to ACPI container, ACPI
> container just calls kobject_uevent(). The patch implements a function
> to remove all devices included in ACPI container.
>
> Signed-off-by: Yasuaki Ishimatsu<isimatu.yasuaki@jp.fujitsu.com>
>
> ---
> drivers/acpi/container.c | 32 ++++++++++++++++++++++++++++++--
> 1 file changed, 30 insertions(+), 2 deletions(-)
>
> Index: linux-3.5-rc1/drivers/acpi/container.c
> ===================================================================
> --- linux-3.5-rc1.orig/drivers/acpi/container.c 2012-06-05 14:01:54.000000000 +0900
> +++ linux-3.5-rc1/drivers/acpi/container.c 2012-06-05 14:17:40.940836226 +0900
> @@ -123,6 +123,8 @@ static int acpi_container_remove(struct
> acpi_status status = AE_OK;
> struct acpi_container *pc = NULL;
>
> + kobject_uevent(&device->dev.kobj, KOBJ_OFFLINE);
> +
> pc = acpi_driver_data(device);
> kfree(pc);
> return status;
> @@ -152,6 +154,32 @@ static int container_device_add(struct a
> return result;
> }
>
> +static void container_device_remove(struct acpi_device *device,
> + acpi_handle handle)
> +{
> + struct acpi_object_list arg_list;
> + union acpi_object arg;
> + int result;
> + acpi_status status;
> +
> + result = acpi_bus_trim(device, 1);
> + if (result) {
> + printk(KERN_WARNING "Removing contaier failed");
> + return;
> + }
> +
> + arg_list.count = 1;
> + arg_list.pointer =&arg;
> + arg.type = ACPI_TYPE_INTEGER;
> + arg.integer.value = 1;
> +
> + status = acpi_evaluate_object(handle, "_EJ0",&arg_list, NULL);
> + if (ACPI_FAILURE(status))
> + printk(KERN_WARNING "Eject device failed\n");
> +
> + return;
> +}
> +
> static void container_notify_cb(acpi_handle handle, u32 type, void *context)
> {
> struct acpi_device *device = NULL;
> @@ -183,13 +211,13 @@ static void container_notify_cb(acpi_han
> } else {
> if (ACPI_SUCCESS(status)) {
> /* device exist and this is a remove request */
> - kobject_uevent(&device->dev.kobj, KOBJ_OFFLINE);
> + container_device_remove(device, handle);
> }
> }
> break;
> case ACPI_NOTIFY_EJECT_REQUEST:
> if (!acpi_bus_get_device(handle,&device)&& device) {
> - kobject_uevent(&device->dev.kobj, KOBJ_OFFLINE);
> + container_device_remove(device, handle);
> }
> break;
> default:
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>




\
 
 \ /
  Last update: 2012-06-12 08:01    [W:0.059 / U:0.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site