Added ACPI resource handler for hotplug operations. The handler, acpi_set_shp_device(), sets device resource information to a hotplug request, which is consumed by the CPU and memory handlers. For setting the device resource information, acpi_scan_shp_devices() walks the acpi_device tree from a target device, and calls .resource of ACPI drivers. For hot-add, acpi_set_shp_device() is called right after the ACPI bus handler so that it can walk through new acpi_device objects. For hot-delete, it is called at the begging of the validate phase so that other validate handlers can use the device resource information for their validations. Signed-off-by: Toshi Kani <toshi.kani@xxxxxx> --- drivers/acpi/Makefile | 1 + drivers/acpi/bus.c | 1 + drivers/acpi/internal.h | 1 + drivers/acpi/shp_resource.c | 86 +++++++++++++++++++++++++++++++++++++++++++ include/acpi/acpi_bus.h | 4 ++ 5 files changed, 93 insertions(+) create mode 100644 drivers/acpi/shp_resource.c diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile index 2a4502b..205be23 100644 --- a/drivers/acpi/Makefile +++ b/drivers/acpi/Makefile @@ -34,6 +34,7 @@ acpi-$(CONFIG_ACPI_SLEEP) += proc.o acpi-y += bus.o glue.o acpi-y += scan.o acpi-y += resource.o +acpi-y += shp_resource.o acpi-y += processor_core.o acpi-y += ec.o acpi-$(CONFIG_ACPI_DOCK) += dock.o diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c index 31a1911..69b5edb 100644 --- a/drivers/acpi/bus.c +++ b/drivers/acpi/bus.c @@ -1236,6 +1236,7 @@ static int __init acpi_init(void) acpi_sleep_proc_init(); acpi_wakeup_device_init(); acpi_shp_init(); + acpi_shp_res_init(); return 0; } diff --git a/drivers/acpi/internal.h b/drivers/acpi/internal.h index 3c407cd..51aa740 100644 --- a/drivers/acpi/internal.h +++ b/drivers/acpi/internal.h @@ -26,6 +26,7 @@ int init_acpi_device_notify(void); int acpi_scan_init(void); int acpi_sysfs_init(void); +void acpi_shp_res_init(void); #ifdef CONFIG_DEBUG_FS extern struct dentry *acpi_debugfs_dir; diff --git a/drivers/acpi/shp_resource.c b/drivers/acpi/shp_resource.c new file mode 100644 index 0000000..51ab968 --- /dev/null +++ b/drivers/acpi/shp_resource.c @@ -0,0 +1,86 @@ +/* + * shp_resource.c - Setup system device hot-plug resource information + * + * Copyright (C) 2012 Hewlett-Packard Development Company, L.P. + * Toshi Kani <toshi.kani@xxxxxx> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/init.h> +#include <linux/module.h> +#include <linux/device.h> +#include <linux/list.h> +#include <linux/acpi.h> +#include <acpi/sys_hotplug.h> + +#include "internal.h" + +static int +acpi_set_shp_device(struct acpi_device *device, struct shp_request *req) +{ + int ret; + + if (!device->driver) { + dev_dbg(&device->dev, "driver not bound\n"); + return 0; + } + + if (!device->driver->ops.resource) + return 0; + + ret = device->driver->ops.resource(device, req); + if (ret) { + dev_err(&device->dev, "ops.resource failed (%d)\n", ret); + return -EINVAL; + } + + return 0; +} + +static int +acpi_scan_shp_devices(struct acpi_device *device, struct shp_request *req) +{ + struct acpi_device *child = NULL; + + if (acpi_set_shp_device(device, req)) + return 0; + + list_for_each_entry(child, &device->children, node) + acpi_scan_shp_devices(child, req); + + return 0; +} + +static int acpi_set_shp_resources(struct shp_request *req, int rollback) +{ + acpi_handle handle = (acpi_handle) req->handle; + struct acpi_device *device = NULL; + + if (rollback) + return 0; + + /* only handle hot-plug operation */ + if (!shp_is_hotplug_op(req->operation)) + return 0; + + if (acpi_bus_get_device(handle, &device)) { + acpi_handle_err(handle, "acpi_bus_get_device failed\n"); + return -EINVAL; + } + + acpi_scan_shp_devices(device, req); + + return 0; +} + +void __init acpi_shp_res_init(void) +{ + shp_register_handler(SHP_ADD_EXECUTE, acpi_set_shp_resources, + SHP_ACPI_RES_ADD_EXECUTE_ORDER); + shp_register_handler(SHP_DEL_VALIDATE, acpi_set_shp_resources, + SHP_ACPI_RES_DEL_VALIDATE_ORDER); +} diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h index 7ced5dc..6bf002e 100644 --- a/include/acpi/acpi_bus.h +++ b/include/acpi/acpi_bus.h @@ -27,6 +27,7 @@ #define __ACPI_BUS_H__ #include <linux/device.h> +#include <acpi/sys_hotplug.h> #include <acpi/acpi.h> @@ -94,6 +95,8 @@ typedef int (*acpi_op_start) (struct acpi_device * device); typedef int (*acpi_op_bind) (struct acpi_device * device); typedef int (*acpi_op_unbind) (struct acpi_device * device); typedef void (*acpi_op_notify) (struct acpi_device * device, u32 event); +typedef int (*acpi_op_resource) (struct acpi_device *device, + struct shp_request *shp_req); struct acpi_bus_ops { u32 acpi_op_add:1; @@ -107,6 +110,7 @@ struct acpi_device_ops { acpi_op_bind bind; acpi_op_unbind unbind; acpi_op_notify notify; + acpi_op_resource resource; }; #define ACPI_DRIVER_ALL_NOTIFY_EVENTS 0x1 /* system AND device events */ -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html