Commit fe06fba2 authored by Alex Chiang's avatar Alex Chiang Committed by Len Brown

ACPI: dock: add struct dock_station * directly to platform device data

Instead of adding a (struct dock_station **) to our dock device's
platform data, we can add the (struct dock_station *) directly.

This change saves us some ugly casting and improves readability.

The cost of making this change is an extra 290 bytes of stack usage,
but this is an infrequently called code-path and unlikely to cause
the kernel to blow up.
Signed-off-by: default avatarAlex Chiang <achiang@hp.com>
Acked-by: default avatarShaohua Li <shaohua.li@intel.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 9751cb72
...@@ -843,8 +843,7 @@ static ssize_t show_docked(struct device *dev, ...@@ -843,8 +843,7 @@ static ssize_t show_docked(struct device *dev,
{ {
struct acpi_device *tmp; struct acpi_device *tmp;
struct dock_station *dock_station = *((struct dock_station **) struct dock_station *dock_station = dev->platform_data;
dev->platform_data);
if (ACPI_SUCCESS(acpi_bus_get_device(dock_station->handle, &tmp))) if (ACPI_SUCCESS(acpi_bus_get_device(dock_station->handle, &tmp)))
return snprintf(buf, PAGE_SIZE, "1\n"); return snprintf(buf, PAGE_SIZE, "1\n");
...@@ -858,8 +857,7 @@ static DEVICE_ATTR(docked, S_IRUGO, show_docked, NULL); ...@@ -858,8 +857,7 @@ static DEVICE_ATTR(docked, S_IRUGO, show_docked, NULL);
static ssize_t show_flags(struct device *dev, static ssize_t show_flags(struct device *dev,
struct device_attribute *attr, char *buf) struct device_attribute *attr, char *buf)
{ {
struct dock_station *dock_station = *((struct dock_station **) struct dock_station *dock_station = dev->platform_data;
dev->platform_data);
return snprintf(buf, PAGE_SIZE, "%d\n", dock_station->flags); return snprintf(buf, PAGE_SIZE, "%d\n", dock_station->flags);
} }
...@@ -872,8 +870,7 @@ static ssize_t write_undock(struct device *dev, struct device_attribute *attr, ...@@ -872,8 +870,7 @@ static ssize_t write_undock(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
{ {
int ret; int ret;
struct dock_station *dock_station = *((struct dock_station **) struct dock_station *dock_station = dev->platform_data;
dev->platform_data);
if (!count) if (!count)
return -EINVAL; return -EINVAL;
...@@ -891,8 +888,7 @@ static ssize_t show_dock_uid(struct device *dev, ...@@ -891,8 +888,7 @@ static ssize_t show_dock_uid(struct device *dev,
struct device_attribute *attr, char *buf) struct device_attribute *attr, char *buf)
{ {
unsigned long long lbuf; unsigned long long lbuf;
struct dock_station *dock_station = *((struct dock_station **) struct dock_station *dock_station = dev->platform_data;
dev->platform_data);
acpi_status status = acpi_evaluate_integer(dock_station->handle, acpi_status status = acpi_evaluate_integer(dock_station->handle,
"_UID", NULL, &lbuf); "_UID", NULL, &lbuf);
if (ACPI_FAILURE(status)) if (ACPI_FAILURE(status))
...@@ -905,8 +901,7 @@ static DEVICE_ATTR(uid, S_IRUGO, show_dock_uid, NULL); ...@@ -905,8 +901,7 @@ static DEVICE_ATTR(uid, S_IRUGO, show_dock_uid, NULL);
static ssize_t show_dock_type(struct device *dev, static ssize_t show_dock_type(struct device *dev,
struct device_attribute *attr, char *buf) struct device_attribute *attr, char *buf)
{ {
struct dock_station *dock_station = *((struct dock_station **) struct dock_station *dock_station = dev->platform_data;
dev->platform_data);
char *type; char *type;
if (dock_station->flags & DOCK_IS_DOCK) if (dock_station->flags & DOCK_IS_DOCK)
...@@ -944,20 +939,18 @@ static struct attribute_group dock_attribute_group = { ...@@ -944,20 +939,18 @@ static struct attribute_group dock_attribute_group = {
*/ */
static int dock_add(acpi_handle handle) static int dock_add(acpi_handle handle)
{ {
int ret; int ret, id;
struct dock_station *dock_station; struct dock_station ds, *dock_station;
struct platform_device *dock_device; struct platform_device *dock_device;
id = dock_station_count;
dock_device = dock_device =
platform_device_register_simple("dock", platform_device_register_data(NULL, "dock",
dock_station_count, NULL, 0); id, &ds, sizeof(ds));
if (IS_ERR(dock_device)) if (IS_ERR(dock_device))
return PTR_ERR(dock_device); return PTR_ERR(dock_device);
/* allocate & initialize the dock_station private data */ dock_station = dock_device->dev.platform_data;
dock_station = kzalloc(sizeof(*dock_station), GFP_KERNEL);
if (!dock_station)
return -ENOMEM;
dock_station->handle = handle; dock_station->handle = handle;
dock_station->dock_device = dock_device; dock_station->dock_device = dock_device;
dock_station->last_dock_time = jiffies - HZ; dock_station->last_dock_time = jiffies - HZ;
...@@ -968,9 +961,6 @@ static int dock_add(acpi_handle handle) ...@@ -968,9 +961,6 @@ static int dock_add(acpi_handle handle)
mutex_init(&dock_station->hp_lock); mutex_init(&dock_station->hp_lock);
ATOMIC_INIT_NOTIFIER_HEAD(&dock_notifier_list); ATOMIC_INIT_NOTIFIER_HEAD(&dock_notifier_list);
platform_device_add_data(dock_device, &dock_station,
sizeof(struct dock_station *));
/* we want the dock device to send uevents */ /* we want the dock device to send uevents */
dev_set_uevent_suppress(&dock_device->dev, 0); dev_set_uevent_suppress(&dock_device->dev, 0);
...@@ -1003,9 +993,6 @@ err_rmgroup: ...@@ -1003,9 +993,6 @@ err_rmgroup:
sysfs_remove_group(&dock_device->dev.kobj, &dock_attribute_group); sysfs_remove_group(&dock_device->dev.kobj, &dock_attribute_group);
err_unregister: err_unregister:
platform_device_unregister(dock_device); platform_device_unregister(dock_device);
out:
kfree(dock_station);
dock_station = NULL;
printk(KERN_ERR "%s encountered error %d\n", __func__, ret); printk(KERN_ERR "%s encountered error %d\n", __func__, ret);
return ret; return ret;
} }
...@@ -1026,13 +1013,12 @@ static int dock_remove(struct dock_station *dock_station) ...@@ -1026,13 +1013,12 @@ static int dock_remove(struct dock_station *dock_station)
list) list)
kfree(dd); kfree(dd);
list_del(&dock_station->sibling);
/* cleanup sysfs */ /* cleanup sysfs */
sysfs_remove_group(&dock_device->dev.kobj, &dock_attribute_group); sysfs_remove_group(&dock_device->dev.kobj, &dock_attribute_group);
platform_device_unregister(dock_device); platform_device_unregister(dock_device);
/* free dock station memory */
kfree(dock_station);
dock_station = NULL;
return 0; return 0;
} }
......
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