Commit a4b5a279 authored by Ike Panhc's avatar Ike Panhc Committed by Matthew Garrett

ideapad: add markups, unify comments and return result when init

1. Add markups on init and exit functions
2. Unify the comments in the same style
3. Return result when module initial
Signed-off-by: default avatarIke Panhc <ike.pan@canonical.com>
Signed-off-by: default avatarMatthew Garrett <mjg@redhat.com>
parent f63409ae
/* /*
* ideapad_acpi.c - Lenovo IdeaPad ACPI Extras * ideapad-laptop.c - Lenovo IdeaPad ACPI Extras
* *
* Copyright © 2010 Intel Corporation * Copyright © 2010 Intel Corporation
* Copyright © 2010 David Woodhouse <dwmw2@infradead.org> * Copyright © 2010 David Woodhouse <dwmw2@infradead.org>
...@@ -168,8 +168,10 @@ static int write_ec_cmd(acpi_handle handle, int cmd, unsigned long data) ...@@ -168,8 +168,10 @@ static int write_ec_cmd(acpi_handle handle, int cmd, unsigned long data)
pr_err("timeout in write_ec_cmd\n"); pr_err("timeout in write_ec_cmd\n");
return -1; return -1;
} }
/* the above is ACPI helpers */
/*
* camera power
*/
static ssize_t show_ideapad_cam(struct device *dev, static ssize_t show_ideapad_cam(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
...@@ -203,6 +205,9 @@ static ssize_t store_ideapad_cam(struct device *dev, ...@@ -203,6 +205,9 @@ static ssize_t store_ideapad_cam(struct device *dev,
static DEVICE_ATTR(camera_power, 0644, show_ideapad_cam, store_ideapad_cam); static DEVICE_ATTR(camera_power, 0644, show_ideapad_cam, store_ideapad_cam);
/*
* Rfkill
*/
static int ideapad_rfk_set(void *data, bool blocked) static int ideapad_rfk_set(void *data, bool blocked)
{ {
int device = (unsigned long)data; int device = (unsigned long)data;
...@@ -235,7 +240,8 @@ static void ideapad_sync_rfk_state(struct acpi_device *adevice) ...@@ -235,7 +240,8 @@ static void ideapad_sync_rfk_state(struct acpi_device *adevice)
rfkill_set_hw_state(priv->rfk[i], hw_blocked); rfkill_set_hw_state(priv->rfk[i], hw_blocked);
} }
static int ideapad_register_rfkill(struct acpi_device *adevice, int dev) static int __devinit ideapad_register_rfkill(struct acpi_device *adevice,
int dev)
{ {
struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); struct ideapad_private *priv = dev_get_drvdata(&adevice->dev);
int ret; int ret;
...@@ -271,7 +277,8 @@ static int ideapad_register_rfkill(struct acpi_device *adevice, int dev) ...@@ -271,7 +277,8 @@ static int ideapad_register_rfkill(struct acpi_device *adevice, int dev)
return 0; return 0;
} }
static void ideapad_unregister_rfkill(struct acpi_device *adevice, int dev) static void __devexit ideapad_unregister_rfkill(struct acpi_device *adevice,
int dev)
{ {
struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); struct ideapad_private *priv = dev_get_drvdata(&adevice->dev);
...@@ -326,7 +333,6 @@ static void ideapad_platform_exit(void) ...@@ -326,7 +333,6 @@ static void ideapad_platform_exit(void)
&ideapad_attribute_group); &ideapad_attribute_group);
platform_device_unregister(ideapad_priv->platform_device); platform_device_unregister(ideapad_priv->platform_device);
} }
/* the above is platform device */
/* /*
* input device * input device
...@@ -386,15 +392,17 @@ static void ideapad_input_report(unsigned long scancode) ...@@ -386,15 +392,17 @@ static void ideapad_input_report(unsigned long scancode)
{ {
sparse_keymap_report_event(ideapad_priv->inputdev, scancode, 1, true); sparse_keymap_report_event(ideapad_priv->inputdev, scancode, 1, true);
} }
/* the above is input device */
/*
* module init/exit
*/
static const struct acpi_device_id ideapad_device_ids[] = { static const struct acpi_device_id ideapad_device_ids[] = {
{ "VPC2004", 0}, { "VPC2004", 0},
{ "", 0}, { "", 0},
}; };
MODULE_DEVICE_TABLE(acpi, ideapad_device_ids); MODULE_DEVICE_TABLE(acpi, ideapad_device_ids);
static int ideapad_acpi_add(struct acpi_device *adevice) static int __devinit ideapad_acpi_add(struct acpi_device *adevice)
{ {
int ret, i, cfg; int ret, i, cfg;
struct ideapad_private *priv; struct ideapad_private *priv;
...@@ -432,7 +440,7 @@ static int ideapad_acpi_add(struct acpi_device *adevice) ...@@ -432,7 +440,7 @@ static int ideapad_acpi_add(struct acpi_device *adevice)
return ret; return ret;
} }
static int ideapad_acpi_remove(struct acpi_device *adevice, int type) static int __devexit ideapad_acpi_remove(struct acpi_device *adevice, int type)
{ {
struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); struct ideapad_private *priv = dev_get_drvdata(&adevice->dev);
int i; int i;
...@@ -478,19 +486,14 @@ static struct acpi_driver ideapad_acpi_driver = { ...@@ -478,19 +486,14 @@ static struct acpi_driver ideapad_acpi_driver = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
}; };
static int __init ideapad_acpi_module_init(void) static int __init ideapad_acpi_module_init(void)
{ {
acpi_bus_register_driver(&ideapad_acpi_driver); return acpi_bus_register_driver(&ideapad_acpi_driver);
return 0;
} }
static void __exit ideapad_acpi_module_exit(void) static void __exit ideapad_acpi_module_exit(void)
{ {
acpi_bus_unregister_driver(&ideapad_acpi_driver); acpi_bus_unregister_driver(&ideapad_acpi_driver);
} }
MODULE_AUTHOR("David Woodhouse <dwmw2@infradead.org>"); MODULE_AUTHOR("David Woodhouse <dwmw2@infradead.org>");
......
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