summaryrefslogtreecommitdiffstats
path: root/drivers/platform
diff options
context:
space:
mode:
authorCorentin Chary <corentincj@iksaif.net>2010-01-13 21:55:44 +0100
committerCorentin Chary <corentincj@iksaif.net>2010-02-28 19:35:11 +0100
commit6358bf2c4c309efc7c3cbc36466c32108c12c456 (patch)
tree88c2cefbb1a12402e4a0292ffb3c32d9477831ad /drivers/platform
parentaa9df930d6eabbd8f2439eca6b2f77f81ce425f2 (diff)
downloadlinux-6358bf2c4c309efc7c3cbc36466c32108c12c456.tar.bz2
asus-laptop: stop using read_status and store_status for GPS
Signed-off-by: Corentin Chary <corentincj@iksaif.net>
Diffstat (limited to 'drivers/platform')
-rw-r--r--drivers/platform/x86/asus-laptop.c56
1 files changed, 35 insertions, 21 deletions
diff --git a/drivers/platform/x86/asus-laptop.c b/drivers/platform/x86/asus-laptop.c
index cee751e301a6..ea51d7e4df21 100644
--- a/drivers/platform/x86/asus-laptop.c
+++ b/drivers/platform/x86/asus-laptop.c
@@ -331,26 +331,9 @@ static int write_acpi_int(acpi_handle handle, const char *method, int val)
return write_acpi_int_ret(handle, method, val, NULL);
}
-static int read_gps_status(struct asus_laptop *asus)
-{
- unsigned long long status;
- acpi_status rv = AE_OK;
-
- rv = acpi_evaluate_integer(gps_status_handle, NULL, NULL, &status);
- if (ACPI_FAILURE(rv))
- pr_warning("Error reading GPS status\n");
- else
- return status ? 1 : 0;
-
- return (asus->status & GPS_ON) ? 1 : 0;
-}
-
/* Generic LED functions */
static int read_status(struct asus_laptop *asus, int mask)
{
- if (mask == GPS_ON)
- return read_gps_status(asus);
-
return (asus->status & mask) ? 1 : 0;
}
@@ -979,21 +962,52 @@ static ssize_t store_lslvl(struct device *dev, struct device_attribute *attr,
/*
* GPS
+ * TODO: use rfkill
*/
+static int asus_gps_status(struct asus_laptop *asus)
+{
+ unsigned long long status;
+ acpi_status rv = AE_OK;
+
+ rv = acpi_evaluate_integer(gps_status_handle, NULL, NULL, &status);
+ if (ACPI_FAILURE(rv)) {
+ pr_warning("Error reading GPS status\n");
+ return -ENODEV;
+ }
+ return !!status;
+}
+
+static int asus_gps_switch(struct asus_laptop *asus, int status)
+{
+ acpi_handle handle = status ? gps_on_handle : gps_off_handle;
+
+ if (write_acpi_int(handle, NULL, 0x02))
+ return -ENODEV;
+ return 0;
+}
+
static ssize_t show_gps(struct device *dev,
struct device_attribute *attr, char *buf)
{
struct asus_laptop *asus = dev_get_drvdata(dev);
- return sprintf(buf, "%d\n", read_status(asus, GPS_ON));
+ return sprintf(buf, "%d\n", asus_gps_status(asus));
}
static ssize_t store_gps(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
- struct asus_laptop *asus = dev_get_drvdata(dev);
+ struct asus_laptop *asus = dev_get_drvdata(dev);
+ int rv, value;
+ int ret;
- return store_status(asus, buf, count, NULL, GPS_ON);
+ rv = parse_arg(buf, count, &value);
+ if (rv <= 0)
+ return -EINVAL;
+ ret = asus_gps_switch(asus, !!value);
+ if (ret)
+ return ret;
+ return rv;
}
/*
@@ -1451,7 +1465,7 @@ static int __devinit asus_acpi_init(struct asus_laptop *asus)
set_light_sens_level(asus, asus->light_level);
/* GPS is on by default */
- write_status(asus, NULL, 1, GPS_ON);
+ asus_gps_switch(asus, 1);
return result;
}