1 files changed, 117 insertions, 165 deletions

diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
index e8165c4f162..8acf53e6296 100644
--- a/drivers/acpi/fan.c
+++ b/drivers/acpi/fan.c
@@ -27,152 +27,104 @@
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/types.h>
-#include <linux/proc_fs.h>
-#include <linux/seq_file.h>
 #include <asm/uaccess.h>
+#include <linux/thermal.h>
+#include <linux/acpi.h>
 
-#include <acpi/acpi_bus.h>
-#include <acpi/acpi_drivers.h>
+#define PREFIX "ACPI: "
 
-#define ACPI_FAN_COMPONENT		0x00200000
 #define ACPI_FAN_CLASS			"fan"
-#define ACPI_FAN_DRIVER_NAME		"ACPI Fan Driver"
 #define ACPI_FAN_FILE_STATE		"state"
 
 #define _COMPONENT		ACPI_FAN_COMPONENT
-ACPI_MODULE_NAME("acpi_fan")
+ACPI_MODULE_NAME("fan");
 
-    MODULE_AUTHOR("Paul Diefenbaugh");
-MODULE_DESCRIPTION(ACPI_FAN_DRIVER_NAME);
+MODULE_AUTHOR("Paul Diefenbaugh");
+MODULE_DESCRIPTION("ACPI Fan Driver");
 MODULE_LICENSE("GPL");
 
 static int acpi_fan_add(struct acpi_device *device);
-static int acpi_fan_remove(struct acpi_device *device, int type);
+static int acpi_fan_remove(struct acpi_device *device);
+
+static const struct acpi_device_id fan_device_ids[] = {
+	{"PNP0C0B", 0},
+	{"", 0},
+};
+MODULE_DEVICE_TABLE(acpi, fan_device_ids);
+
+#ifdef CONFIG_PM_SLEEP
+static int acpi_fan_suspend(struct device *dev);
+static int acpi_fan_resume(struct device *dev);
+static struct dev_pm_ops acpi_fan_pm = {
+	.resume = acpi_fan_resume,
+	.freeze = acpi_fan_suspend,
+	.thaw = acpi_fan_resume,
+	.restore = acpi_fan_resume,
+};
+#define FAN_PM_OPS_PTR (&acpi_fan_pm)
+#else
+#define FAN_PM_OPS_PTR NULL
+#endif
 
 static struct acpi_driver acpi_fan_driver = {
-	.name = ACPI_FAN_DRIVER_NAME,
+	.name = "fan",
 	.class = ACPI_FAN_CLASS,
-	.ids = "PNP0C0B",
+	.ids = fan_device_ids,
 	.ops = {
 		.add = acpi_fan_add,
 		.remove = acpi_fan_remove,
 		},
+	.drv.pm = FAN_PM_OPS_PTR,
 };
 
-struct acpi_fan {
-	acpi_handle handle;
-};
-
-/* --------------------------------------------------------------------------
-                              FS Interface (/proc)
-   -------------------------------------------------------------------------- */
-
-static struct proc_dir_entry *acpi_fan_dir;
-
-static int acpi_fan_read_state(struct seq_file *seq, void *offset)
-{
-	struct acpi_fan *fan = seq->private;
-	int state = 0;
-
-	ACPI_FUNCTION_TRACE("acpi_fan_read_state");
-
-	if (fan) {
-		if (acpi_bus_get_power(fan->handle, &state))
-			seq_printf(seq, "status:                  ERROR\n");
-		else
-			seq_printf(seq, "status:                  %s\n",
-				   !state ? "on" : "off");
-	}
-	return_VALUE(0);
-}
-
-static int acpi_fan_state_open_fs(struct inode *inode, struct file *file)
+/* thermal cooling device callbacks */
+static int fan_get_max_state(struct thermal_cooling_device *cdev, unsigned long
+			     *state)
 {
-	return single_open(file, acpi_fan_read_state, PDE(inode)->data);
+	/* ACPI fan device only support two states: ON/OFF */
+	*state = 1;
+	return 0;
 }
 
-static ssize_t
-acpi_fan_write_state(struct file *file, const char __user * buffer,
-		     size_t count, loff_t * ppos)
+static int fan_get_cur_state(struct thermal_cooling_device *cdev, unsigned long
+			     *state)
 {
-	int result = 0;
-	struct seq_file *m = (struct seq_file *)file->private_data;
-	struct acpi_fan *fan = (struct acpi_fan *)m->private;
-	char state_string[12] = { '\0' };
-
-	ACPI_FUNCTION_TRACE("acpi_fan_write_state");
+	struct acpi_device *device = cdev->devdata;
+	int result;
+	int acpi_state = ACPI_STATE_D0;
 
-	if (!fan || (count > sizeof(state_string) - 1))
-		return_VALUE(-EINVAL);
-
-	if (copy_from_user(state_string, buffer, count))
-		return_VALUE(-EFAULT);
-
-	state_string[count] = '\0';
+	if (!device)
+		return -EINVAL;
 
-	result = acpi_bus_set_power(fan->handle,
-				    simple_strtoul(state_string, NULL, 0));
+	result = acpi_bus_update_power(device->handle, &acpi_state);
 	if (result)
-		return_VALUE(result);
+		return result;
 
-	return_VALUE(count);
+	*state = (acpi_state == ACPI_STATE_D3_COLD ? 0 :
+		 (acpi_state == ACPI_STATE_D0 ? 1 : -1));
+	return 0;
 }
 
-static struct file_operations acpi_fan_state_ops = {
-	.open = acpi_fan_state_open_fs,
-	.read = seq_read,
-	.write = acpi_fan_write_state,
-	.llseek = seq_lseek,
-	.release = single_release,
-	.owner = THIS_MODULE,
-};
-
-static int acpi_fan_add_fs(struct acpi_device *device)
+static int
+fan_set_cur_state(struct thermal_cooling_device *cdev, unsigned long state)
 {
-	struct proc_dir_entry *entry = NULL;
-
-	ACPI_FUNCTION_TRACE("acpi_fan_add_fs");
+	struct acpi_device *device = cdev->devdata;
+	int result;
 
-	if (!device)
-		return_VALUE(-EINVAL);
-
-	if (!acpi_device_dir(device)) {
-		acpi_device_dir(device) = proc_mkdir(acpi_device_bid(device),
-						     acpi_fan_dir);
-		if (!acpi_device_dir(device))
-			return_VALUE(-ENODEV);
-		acpi_device_dir(device)->owner = THIS_MODULE;
-	}
+	if (!device || (state != 0 && state != 1))
+		return -EINVAL;
 
-	/* 'status' [R/W] */
-	entry = create_proc_entry(ACPI_FAN_FILE_STATE,
-				  S_IFREG | S_IRUGO | S_IWUSR,
-				  acpi_device_dir(device));
-	if (!entry)
-		ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
-				  "Unable to create '%s' fs entry\n",
-				  ACPI_FAN_FILE_STATE));
-	else {
-		entry->proc_fops = &acpi_fan_state_ops;
-		entry->data = acpi_driver_data(device);
-		entry->owner = THIS_MODULE;
-	}
+	result = acpi_bus_set_power(device->handle,
+				state ? ACPI_STATE_D0 : ACPI_STATE_D3_COLD);
 
-	return_VALUE(0);
+	return result;
 }
 
-static int acpi_fan_remove_fs(struct acpi_device *device)
-{
-	ACPI_FUNCTION_TRACE("acpi_fan_remove_fs");
-
-	if (acpi_device_dir(device)) {
-		remove_proc_entry(ACPI_FAN_FILE_STATE, acpi_device_dir(device));
-		remove_proc_entry(acpi_device_bid(device), acpi_fan_dir);
-		acpi_device_dir(device) = NULL;
-	}
-
-	return_VALUE(0);
-}
+static const struct thermal_cooling_device_ops fan_cooling_ops = {
+	.get_max_state = fan_get_max_state,
+	.get_cur_state = fan_get_cur_state,
+	.set_cur_state = fan_set_cur_state,
+};
 
 /* --------------------------------------------------------------------------
                                  Driver Interface
@@ -181,94 +133,94 @@ static int acpi_fan_remove_fs(struct acpi_device *device)
 static int acpi_fan_add(struct acpi_device *device)
 {
 	int result = 0;
-	struct acpi_fan *fan = NULL;
-	int state = 0;
-
-	ACPI_FUNCTION_TRACE("acpi_fan_add");
+	struct thermal_cooling_device *cdev;
 
 	if (!device)
-		return_VALUE(-EINVAL);
+		return -EINVAL;
 
-	fan = kmalloc(sizeof(struct acpi_fan), GFP_KERNEL);
-	if (!fan)
-		return_VALUE(-ENOMEM);
-	memset(fan, 0, sizeof(struct acpi_fan));
-
-	fan->handle = device->handle;
 	strcpy(acpi_device_name(device), "Fan");
 	strcpy(acpi_device_class(device), ACPI_FAN_CLASS);
-	acpi_driver_data(device) = fan;
 
-	result = acpi_bus_get_power(fan->handle, &state);
+	result = acpi_bus_update_power(device->handle, NULL);
 	if (result) {
-		ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
-				  "Error reading power state\n"));
+		printk(KERN_ERR PREFIX "Setting initial power state\n");
 		goto end;
 	}
 
-	result = acpi_fan_add_fs(device);
-	if (result)
+	cdev = thermal_cooling_device_register("Fan", device,
+						&fan_cooling_ops);
+	if (IS_ERR(cdev)) {
+		result = PTR_ERR(cdev);
 		goto end;
+	}
+
+	dev_dbg(&device->dev, "registered as cooling_device%d\n", cdev->id);
+
+	device->driver_data = cdev;
+	result = sysfs_create_link(&device->dev.kobj,
+				   &cdev->device.kobj,
+				   "thermal_cooling");
+	if (result)
+		dev_err(&device->dev, "Failed to create sysfs link "
+			"'thermal_cooling'\n");
+
+	result = sysfs_create_link(&cdev->device.kobj,
+				   &device->dev.kobj,
+				   "device");
+	if (result)
+		dev_err(&device->dev, "Failed to create sysfs link "
+			"'device'\n");
 
 	printk(KERN_INFO PREFIX "%s [%s] (%s)\n",
 	       acpi_device_name(device), acpi_device_bid(device),
 	       !device->power.state ? "on" : "off");
 
-      end:
-	if (result)
-		kfree(fan);
-
-	return_VALUE(result);
+end:
+	return result;
 }
 
-static int acpi_fan_remove(struct acpi_device *device, int type)
+static int acpi_fan_remove(struct acpi_device *device)
 {
-	struct acpi_fan *fan = NULL;
-
-	ACPI_FUNCTION_TRACE("acpi_fan_remove");
-
-	if (!device || !acpi_driver_data(device))
-		return_VALUE(-EINVAL);
+	struct thermal_cooling_device *cdev;
 
-	fan = (struct acpi_fan *)acpi_driver_data(device);
+	if (!device)
+		return -EINVAL;
 
-	acpi_fan_remove_fs(device);
+	cdev =  acpi_driver_data(device);
+	if (!cdev)
+		return -EINVAL;
 
-	kfree(fan);
+	sysfs_remove_link(&device->dev.kobj, "thermal_cooling");
+	sysfs_remove_link(&cdev->device.kobj, "device");
+	thermal_cooling_device_unregister(cdev);
 
-	return_VALUE(0);
+	return 0;
 }
 
-static int __init acpi_fan_init(void)
+#ifdef CONFIG_PM_SLEEP
+static int acpi_fan_suspend(struct device *dev)
 {
-	int result = 0;
-
-	ACPI_FUNCTION_TRACE("acpi_fan_init");
+	if (!dev)
+		return -EINVAL;
 
-	acpi_fan_dir = proc_mkdir(ACPI_FAN_CLASS, acpi_root_dir);
-	if (!acpi_fan_dir)
-		return_VALUE(-ENODEV);
-	acpi_fan_dir->owner = THIS_MODULE;
+	acpi_bus_set_power(to_acpi_device(dev)->handle, ACPI_STATE_D0);
 
-	result = acpi_bus_register_driver(&acpi_fan_driver);
-	if (result < 0) {
-		remove_proc_entry(ACPI_FAN_CLASS, acpi_root_dir);
-		return_VALUE(-ENODEV);
-	}
-
-	return_VALUE(0);
+	return AE_OK;
 }
 
-static void __exit acpi_fan_exit(void)
+static int acpi_fan_resume(struct device *dev)
 {
-	ACPI_FUNCTION_TRACE("acpi_fan_exit");
+	int result;
 
-	acpi_bus_unregister_driver(&acpi_fan_driver);
+	if (!dev)
+		return -EINVAL;
 
-	remove_proc_entry(ACPI_FAN_CLASS, acpi_root_dir);
+	result = acpi_bus_update_power(to_acpi_device(dev)->handle, NULL);
+	if (result)
+		printk(KERN_ERR PREFIX "Error updating fan power state\n");
 
-	return_VOID;
+	return result;
 }
+#endif
 
-module_init(acpi_fan_init);
-module_exit(acpi_fan_exit);
+module_acpi_driver(acpi_fan_driver);