Commit 19593a1f authored by Aaron Lu's avatar Aaron Lu Committed by Zhang Rui

ACPI / fan: convert to platform driver

Convert ACPI fan driver to a platform driver for the purpose of phasing
out ACPI bus.
Signed-off-by: default avatarAaron Lu <aaron.lu@intel.com>
Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
parent 2bb3a2bf
...@@ -30,17 +30,14 @@ ...@@ -30,17 +30,14 @@
#include <asm/uaccess.h> #include <asm/uaccess.h>
#include <linux/thermal.h> #include <linux/thermal.h>
#include <linux/acpi.h> #include <linux/acpi.h>
#include <linux/platform_device.h>
#define PREFIX "ACPI: "
#define ACPI_FAN_CLASS "fan"
MODULE_AUTHOR("Paul Diefenbaugh"); MODULE_AUTHOR("Paul Diefenbaugh");
MODULE_DESCRIPTION("ACPI Fan Driver"); MODULE_DESCRIPTION("ACPI Fan Driver");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
static int acpi_fan_add(struct acpi_device *device); static int acpi_fan_probe(struct platform_device *pdev);
static int acpi_fan_remove(struct acpi_device *device); static int acpi_fan_remove(struct platform_device *pdev);
static const struct acpi_device_id fan_device_ids[] = { static const struct acpi_device_id fan_device_ids[] = {
{"PNP0C0B", 0}, {"PNP0C0B", 0},
...@@ -62,15 +59,14 @@ static struct dev_pm_ops acpi_fan_pm = { ...@@ -62,15 +59,14 @@ static struct dev_pm_ops acpi_fan_pm = {
#define FAN_PM_OPS_PTR NULL #define FAN_PM_OPS_PTR NULL
#endif #endif
static struct acpi_driver acpi_fan_driver = { static struct platform_driver acpi_fan_driver = {
.name = "fan", .probe = acpi_fan_probe,
.class = ACPI_FAN_CLASS, .remove = acpi_fan_remove,
.ids = fan_device_ids, .driver = {
.ops = { .name = "acpi-fan",
.add = acpi_fan_add, .acpi_match_table = fan_device_ids,
.remove = acpi_fan_remove, .pm = FAN_PM_OPS_PTR,
}, },
.drv.pm = FAN_PM_OPS_PTR,
}; };
/* thermal cooling device callbacks */ /* thermal cooling device callbacks */
...@@ -126,17 +122,15 @@ static const struct thermal_cooling_device_ops fan_cooling_ops = { ...@@ -126,17 +122,15 @@ static const struct thermal_cooling_device_ops fan_cooling_ops = {
Driver Interface Driver Interface
-------------------------------------------------------------------------- */ -------------------------------------------------------------------------- */
static int acpi_fan_add(struct acpi_device *device) static int acpi_fan_probe(struct platform_device *pdev)
{ {
int result = 0; int result = 0;
struct thermal_cooling_device *cdev; struct thermal_cooling_device *cdev;
struct acpi_device *device = ACPI_COMPANION(&pdev->dev);
strcpy(acpi_device_name(device), "Fan");
strcpy(acpi_device_class(device), ACPI_FAN_CLASS);
result = acpi_device_update_power(device, NULL); result = acpi_device_update_power(device, NULL);
if (result) { if (result) {
printk(KERN_ERR PREFIX "Setting initial power state\n"); dev_err(&pdev->dev, "Setting initial power state\n");
goto end; goto end;
} }
...@@ -147,24 +141,24 @@ static int acpi_fan_add(struct acpi_device *device) ...@@ -147,24 +141,24 @@ static int acpi_fan_add(struct acpi_device *device)
goto end; goto end;
} }
dev_dbg(&device->dev, "registered as cooling_device%d\n", cdev->id); dev_dbg(&pdev->dev, "registered as cooling_device%d\n", cdev->id);
device->driver_data = cdev; platform_set_drvdata(pdev, cdev);
result = sysfs_create_link(&device->dev.kobj, result = sysfs_create_link(&pdev->dev.kobj,
&cdev->device.kobj, &cdev->device.kobj,
"thermal_cooling"); "thermal_cooling");
if (result) if (result)
dev_err(&device->dev, "Failed to create sysfs link " dev_err(&pdev->dev, "Failed to create sysfs link "
"'thermal_cooling'\n"); "'thermal_cooling'\n");
result = sysfs_create_link(&cdev->device.kobj, result = sysfs_create_link(&cdev->device.kobj,
&device->dev.kobj, &pdev->dev.kobj,
"device"); "device");
if (result) if (result)
dev_err(&device->dev, "Failed to create sysfs link " dev_err(&pdev->dev, "Failed to create sysfs link "
"'device'\n"); "'device'\n");
printk(KERN_INFO PREFIX "%s [%s] (%s)\n", dev_info(&pdev->dev, "%s [%s] (%s)\n",
acpi_device_name(device), acpi_device_bid(device), acpi_device_name(device), acpi_device_bid(device),
!device->power.state ? "on" : "off"); !device->power.state ? "on" : "off");
...@@ -172,11 +166,11 @@ static int acpi_fan_add(struct acpi_device *device) ...@@ -172,11 +166,11 @@ static int acpi_fan_add(struct acpi_device *device)
return result; return result;
} }
static int acpi_fan_remove(struct acpi_device *device) static int acpi_fan_remove(struct platform_device *pdev)
{ {
struct thermal_cooling_device *cdev = acpi_driver_data(device); struct thermal_cooling_device *cdev = platform_get_drvdata(pdev);
sysfs_remove_link(&device->dev.kobj, "thermal_cooling"); sysfs_remove_link(&pdev->dev.kobj, "thermal_cooling");
sysfs_remove_link(&cdev->device.kobj, "device"); sysfs_remove_link(&cdev->device.kobj, "device");
thermal_cooling_device_unregister(cdev); thermal_cooling_device_unregister(cdev);
...@@ -186,7 +180,7 @@ static int acpi_fan_remove(struct acpi_device *device) ...@@ -186,7 +180,7 @@ static int acpi_fan_remove(struct acpi_device *device)
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
static int acpi_fan_suspend(struct device *dev) static int acpi_fan_suspend(struct device *dev)
{ {
acpi_device_set_power(to_acpi_device(dev), ACPI_STATE_D0); acpi_device_set_power(ACPI_COMPANION(dev), ACPI_STATE_D0);
return AE_OK; return AE_OK;
} }
...@@ -195,12 +189,12 @@ static int acpi_fan_resume(struct device *dev) ...@@ -195,12 +189,12 @@ static int acpi_fan_resume(struct device *dev)
{ {
int result; int result;
result = acpi_device_update_power(to_acpi_device(dev), NULL); result = acpi_device_update_power(ACPI_COMPANION(dev), NULL);
if (result) if (result)
printk(KERN_ERR PREFIX "Error updating fan power state\n"); dev_err(dev, "Error updating fan power state\n");
return result; return result;
} }
#endif #endif
module_acpi_driver(acpi_fan_driver); module_platform_driver(acpi_fan_driver);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment