From 58ac7aa0c308d8b7e38e92840de59da7a39834d8 Mon Sep 17 00:00:00 2001 From: David Woodhouse Date: Tue, 10 Aug 2010 23:44:05 +0100 Subject: Add Lenovo ideapad driver Signed-off-by: David Woodhouse --- drivers/platform/x86/Kconfig | 7 + drivers/platform/x86/Makefile | 1 + drivers/platform/x86/ideapad_acpi.c | 282 ++++++++++++++++++++++++++++++++++++ 3 files changed, 290 insertions(+) create mode 100644 drivers/platform/x86/ideapad_acpi.c diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig index 79baa6368f79..044f430f3b43 100644 --- a/drivers/platform/x86/Kconfig +++ b/drivers/platform/x86/Kconfig @@ -219,6 +219,13 @@ config SONYPI_COMPAT ---help--- Build the sonypi driver compatibility code into the sony-laptop driver. +config IDEAPAD_ACPI + tristate "Lenovo IdeaPad ACPI Laptop Extras" + depends on ACPI + depends on RFKILL + help + This is a driver for the rfkill switches on Lenovo IdeaPad netbooks. + config THINKPAD_ACPI tristate "ThinkPad ACPI Laptop Extras" depends on ACPI diff --git a/drivers/platform/x86/Makefile b/drivers/platform/x86/Makefile index 4744c7744ffa..85fb2b84f57e 100644 --- a/drivers/platform/x86/Makefile +++ b/drivers/platform/x86/Makefile @@ -15,6 +15,7 @@ obj-$(CONFIG_ACERHDF) += acerhdf.o obj-$(CONFIG_HP_WMI) += hp-wmi.o obj-$(CONFIG_TC1100_WMI) += tc1100-wmi.o obj-$(CONFIG_SONY_LAPTOP) += sony-laptop.o +obj-$(CONFIG_IDEAPAD_ACPI) += ideapad_acpi.o obj-$(CONFIG_THINKPAD_ACPI) += thinkpad_acpi.o obj-$(CONFIG_FUJITSU_LAPTOP) += fujitsu-laptop.o obj-$(CONFIG_PANASONIC_LAPTOP) += panasonic-laptop.o diff --git a/drivers/platform/x86/ideapad_acpi.c b/drivers/platform/x86/ideapad_acpi.c new file mode 100644 index 000000000000..ae744caf4726 --- /dev/null +++ b/drivers/platform/x86/ideapad_acpi.c @@ -0,0 +1,282 @@ +/* + * ideapad_acpi.c - Lenovo IdeaPad ACPI Extras + * + * Copyright © 2010 Intel Corporation + * Copyright © 2010 David Woodhouse + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA + * 02110-1301, USA. + */ + +#include +#include +#include +#include +#include +#include +#include + +#define IDEAPAD_DEV_CAMERA 0 +#define IDEAPAD_DEV_WLAN 1 +#define IDEAPAD_DEV_BLUETOOTH 2 +#define IDEAPAD_DEV_3G 3 +#define IDEAPAD_DEV_KILLSW 4 + +static struct rfkill *ideapad_rfkill[5]; + +static const char *ideapad_rfk_names[] = { + "ideapad_camera", "ideapad_wlan", "ideapad_bluetooth", "ideapad_3g", "ideapad_rfkill" +}; +static const int ideapad_rfk_types[] = { + 0, RFKILL_TYPE_WLAN, RFKILL_TYPE_BLUETOOTH, RFKILL_TYPE_WWAN, RFKILL_TYPE_WLAN +}; + +static int ideapad_dev_exists(int device) +{ + acpi_status status; + union acpi_object in_param; + struct acpi_object_list input = { 1, &in_param }; + struct acpi_buffer output; + union acpi_object out_obj; + + output.length = sizeof(out_obj); + output.pointer = &out_obj; + + in_param.type = ACPI_TYPE_INTEGER; + in_param.integer.value = device + 1; + + status = acpi_evaluate_object(NULL, "\\_SB_.DECN", &input, &output); + if (ACPI_FAILURE(status)) { + printk(KERN_WARNING "IdeaPAD \\_SB_.DECN method failed %d. Is this an IdeaPAD?\n", status); + return -ENODEV; + } + if (out_obj.type != ACPI_TYPE_INTEGER) { + printk(KERN_WARNING "IdeaPAD \\_SB_.DECN method returned unexpected type\n"); + return -ENODEV; + } + return out_obj.integer.value; +} + +static int ideapad_dev_get_state(int device) +{ + acpi_status status; + union acpi_object in_param; + struct acpi_object_list input = { 1, &in_param }; + struct acpi_buffer output; + union acpi_object out_obj; + + output.length = sizeof(out_obj); + output.pointer = &out_obj; + + in_param.type = ACPI_TYPE_INTEGER; + in_param.integer.value = device + 1; + + status = acpi_evaluate_object(NULL, "\\_SB_.GECN", &input, &output); + if (ACPI_FAILURE(status)) { + printk(KERN_WARNING "IdeaPAD \\_SB_.GECN method failed %d\n", status); + return -ENODEV; + } + if (out_obj.type != ACPI_TYPE_INTEGER) { + printk(KERN_WARNING "IdeaPAD \\_SB_.GECN method returned unexpected type\n"); + return -ENODEV; + } + return out_obj.integer.value; +} + +static int ideapad_dev_set_state(int device, int state) +{ + acpi_status status; + union acpi_object in_params[2]; + struct acpi_object_list input = { 2, in_params }; + + in_params[0].type = ACPI_TYPE_INTEGER; + in_params[0].integer.value = device + 1; + in_params[1].type = ACPI_TYPE_INTEGER; + in_params[1].integer.value = state; + + status = acpi_evaluate_object(NULL, "\\_SB_.SECN", &input, NULL); + if (ACPI_FAILURE(status)) { + printk(KERN_WARNING "IdeaPAD \\_SB_.SECN method failed %d\n", status); + return -ENODEV; + } + return 0; +} +static ssize_t show_ideapad_cam(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + int state = ideapad_dev_get_state(IDEAPAD_DEV_CAMERA); + if (state < 0) + return state; + + return sprintf(buf, "%d\n", state); +} + +static ssize_t store_ideapad_cam(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + int ret, state; + + if (!count) + return 0; + if (sscanf(buf, "%i", &state) != 1) + return -EINVAL; + ret = ideapad_dev_set_state(IDEAPAD_DEV_CAMERA, state); + if (ret < 0) + return ret; + return count; +} + +static DEVICE_ATTR(camera_power, 0644, show_ideapad_cam, store_ideapad_cam); + +static int ideapad_rfk_set(void *data, bool blocked) +{ + int device = (unsigned long)data; + + if (device == IDEAPAD_DEV_KILLSW) + return -EINVAL; + return ideapad_dev_set_state(device, !blocked); +} + +static struct rfkill_ops ideapad_rfk_ops = { + .set_block = ideapad_rfk_set, +}; + +static void ideapad_sync_rfk_state(void) +{ + int hw_blocked = !ideapad_dev_get_state(IDEAPAD_DEV_KILLSW); + int i; + + rfkill_set_hw_state(ideapad_rfkill[IDEAPAD_DEV_KILLSW], hw_blocked); + for (i = IDEAPAD_DEV_WLAN; i < IDEAPAD_DEV_KILLSW; i++) + if (ideapad_rfkill[i]) + rfkill_set_hw_state(ideapad_rfkill[i], hw_blocked); + if (hw_blocked) + return; + + for (i = IDEAPAD_DEV_WLAN; i < IDEAPAD_DEV_KILLSW; i++) + if (ideapad_rfkill[i]) + rfkill_set_sw_state(ideapad_rfkill[i], !ideapad_dev_get_state(i)); +} + +static int ideapad_register_rfkill(struct acpi_device *device, int dev) +{ + int ret; + + ideapad_rfkill[dev] = rfkill_alloc(ideapad_rfk_names[dev], &device->dev, + ideapad_rfk_types[dev], &ideapad_rfk_ops, + (void *)(long)dev); + if (!ideapad_rfkill[dev]) + return -ENOMEM; + + ret = rfkill_register(ideapad_rfkill[dev]); + if (ret) { + rfkill_destroy(ideapad_rfkill[dev]); + return ret; + } + return 0; +} + +static void ideapad_unregister_rfkill(int dev) +{ + if (!ideapad_rfkill[dev]) + return; + + rfkill_unregister(ideapad_rfkill[dev]); + rfkill_destroy(ideapad_rfkill[dev]); +} + +static const struct acpi_device_id ideapad_device_ids[] = { + { "VPC2004", 0}, + { "", 0}, +}; +MODULE_DEVICE_TABLE(acpi, ideapad_device_ids); + +static int ideapad_acpi_add(struct acpi_device *device) +{ + int i; + int devs_present[5]; + + for (i = IDEAPAD_DEV_CAMERA; i < IDEAPAD_DEV_KILLSW; i++) { + devs_present[i] = ideapad_dev_exists(i); + if (devs_present[i] < 0) + return devs_present[i]; + } + + /* The hardware switch is always present */ + devs_present[IDEAPAD_DEV_KILLSW] = 1; + + if (devs_present[IDEAPAD_DEV_CAMERA]) { + int ret = device_create_file(&device->dev, &dev_attr_camera_power); + if (ret) + return ret; + } + + for (i = IDEAPAD_DEV_WLAN; i <= IDEAPAD_DEV_KILLSW; i++) { + if (!devs_present[i]) + continue; + + ideapad_register_rfkill(device, i); + } + ideapad_sync_rfk_state(); + return 0; +} + +static int ideapad_acpi_remove(struct acpi_device *device, int type) +{ + int i; + device_remove_file(&device->dev, &dev_attr_camera_power); + for (i = 0; i < 5; i++) + ideapad_unregister_rfkill(i); + return 0; +} + +static void ideapad_acpi_notify(struct acpi_device *device, u32 event) +{ + ideapad_sync_rfk_state(); +} + +static struct acpi_driver ideapad_acpi_driver = { + .name = "ideapad_acpi", + .class = "IdeaPad", + .ids = ideapad_device_ids, + .ops.add = ideapad_acpi_add, + .ops.remove = ideapad_acpi_remove, + .ops.notify = ideapad_acpi_notify, + .owner = THIS_MODULE, +}; + + +static int __init ideapad_acpi_module_init(void) +{ + acpi_bus_register_driver(&ideapad_acpi_driver); + + return 0; +} + + +static void __exit ideapad_acpi_module_exit(void) +{ + acpi_bus_unregister_driver(&ideapad_acpi_driver); + +} + +MODULE_AUTHOR("David Woodhouse "); +MODULE_DESCRIPTION("IdeaPad ACPI Extras"); +MODULE_LICENSE("GPL"); + +module_init(ideapad_acpi_module_init); +module_exit(ideapad_acpi_module_exit); -- cgit v1.2.3-59-g8ed1b From ce326329d72238196d61312d1cf1556ebaf94da7 Mon Sep 17 00:00:00 2001 From: David Woodhouse Date: Wed, 11 Aug 2010 17:59:35 +0100 Subject: ideapad: Stop using global variables Signed-off-by: David Woodhouse --- drivers/platform/x86/ideapad_acpi.c | 92 +++++++++++++++++++++++-------------- 1 file changed, 58 insertions(+), 34 deletions(-) diff --git a/drivers/platform/x86/ideapad_acpi.c b/drivers/platform/x86/ideapad_acpi.c index ae744caf4726..8a89902e5d55 100644 --- a/drivers/platform/x86/ideapad_acpi.c +++ b/drivers/platform/x86/ideapad_acpi.c @@ -34,13 +34,19 @@ #define IDEAPAD_DEV_3G 3 #define IDEAPAD_DEV_KILLSW 4 -static struct rfkill *ideapad_rfkill[5]; - -static const char *ideapad_rfk_names[] = { - "ideapad_camera", "ideapad_wlan", "ideapad_bluetooth", "ideapad_3g", "ideapad_rfkill" +struct ideapad_private { + struct rfkill *rfk[5]; }; -static const int ideapad_rfk_types[] = { - 0, RFKILL_TYPE_WLAN, RFKILL_TYPE_BLUETOOTH, RFKILL_TYPE_WWAN, RFKILL_TYPE_WLAN + +static struct { + char *name; + int type; +} ideapad_rfk_data[] = { + /* camera has no rfkill */ + { "ideapad_wlan", RFKILL_TYPE_WLAN }, + { "ideapad_bluetooth", RFKILL_TYPE_BLUETOOTH }, + { "ideapad_3g", RFKILL_TYPE_WWAN }, + { "ideapad_killsw", RFKILL_TYPE_WLAN } }; static int ideapad_dev_exists(int device) @@ -155,48 +161,52 @@ static struct rfkill_ops ideapad_rfk_ops = { .set_block = ideapad_rfk_set, }; -static void ideapad_sync_rfk_state(void) +static void ideapad_sync_rfk_state(struct acpi_device *adevice) { + struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); int hw_blocked = !ideapad_dev_get_state(IDEAPAD_DEV_KILLSW); int i; - rfkill_set_hw_state(ideapad_rfkill[IDEAPAD_DEV_KILLSW], hw_blocked); + rfkill_set_hw_state(priv->rfk[IDEAPAD_DEV_KILLSW], hw_blocked); for (i = IDEAPAD_DEV_WLAN; i < IDEAPAD_DEV_KILLSW; i++) - if (ideapad_rfkill[i]) - rfkill_set_hw_state(ideapad_rfkill[i], hw_blocked); + if (priv->rfk[i]) + rfkill_set_hw_state(priv->rfk[i], hw_blocked); if (hw_blocked) return; for (i = IDEAPAD_DEV_WLAN; i < IDEAPAD_DEV_KILLSW; i++) - if (ideapad_rfkill[i]) - rfkill_set_sw_state(ideapad_rfkill[i], !ideapad_dev_get_state(i)); + if (priv->rfk[i]) + rfkill_set_sw_state(priv->rfk[i], !ideapad_dev_get_state(i)); } -static int ideapad_register_rfkill(struct acpi_device *device, int dev) +static int ideapad_register_rfkill(struct acpi_device *adevice, int dev) { + struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); int ret; - ideapad_rfkill[dev] = rfkill_alloc(ideapad_rfk_names[dev], &device->dev, - ideapad_rfk_types[dev], &ideapad_rfk_ops, - (void *)(long)dev); - if (!ideapad_rfkill[dev]) + priv->rfk[dev] = rfkill_alloc(ideapad_rfk_data[dev-1].name, &adevice->dev, + ideapad_rfk_data[dev-1].type, &ideapad_rfk_ops, + (void *)(long)dev); + if (!priv->rfk[dev]) return -ENOMEM; - ret = rfkill_register(ideapad_rfkill[dev]); + ret = rfkill_register(priv->rfk[dev]); if (ret) { - rfkill_destroy(ideapad_rfkill[dev]); + rfkill_destroy(priv->rfk[dev]); return ret; } return 0; } -static void ideapad_unregister_rfkill(int dev) +static void ideapad_unregister_rfkill(struct acpi_device *adevice, int dev) { - if (!ideapad_rfkill[dev]) + struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); + + if (!priv->rfk[dev]) return; - rfkill_unregister(ideapad_rfkill[dev]); - rfkill_destroy(ideapad_rfkill[dev]); + rfkill_unregister(priv->rfk[dev]); + rfkill_destroy(priv->rfk[dev]); } static const struct acpi_device_id ideapad_device_ids[] = { @@ -205,10 +215,11 @@ static const struct acpi_device_id ideapad_device_ids[] = { }; MODULE_DEVICE_TABLE(acpi, ideapad_device_ids); -static int ideapad_acpi_add(struct acpi_device *device) +static int ideapad_acpi_add(struct acpi_device *adevice) { int i; int devs_present[5]; + struct ideapad_private *priv; for (i = IDEAPAD_DEV_CAMERA; i < IDEAPAD_DEV_KILLSW; i++) { devs_present[i] = ideapad_dev_exists(i); @@ -219,34 +230,47 @@ static int ideapad_acpi_add(struct acpi_device *device) /* The hardware switch is always present */ devs_present[IDEAPAD_DEV_KILLSW] = 1; + priv = kzalloc(sizeof(*priv), GFP_KERNEL); + if (!priv) + return -ENOMEM; + if (devs_present[IDEAPAD_DEV_CAMERA]) { - int ret = device_create_file(&device->dev, &dev_attr_camera_power); - if (ret) + int ret = device_create_file(&adevice->dev, &dev_attr_camera_power); + if (ret) { + kfree(priv); return ret; + } } + dev_set_drvdata(&adevice->dev, priv); for (i = IDEAPAD_DEV_WLAN; i <= IDEAPAD_DEV_KILLSW; i++) { if (!devs_present[i]) continue; - ideapad_register_rfkill(device, i); + ideapad_register_rfkill(adevice, i); } - ideapad_sync_rfk_state(); + ideapad_sync_rfk_state(adevice); return 0; } -static int ideapad_acpi_remove(struct acpi_device *device, int type) +static int ideapad_acpi_remove(struct acpi_device *adevice, int type) { + struct ideapad_private *priv = dev_get_drvdata(&adevice->dev); int i; - device_remove_file(&device->dev, &dev_attr_camera_power); - for (i = 0; i < 5; i++) - ideapad_unregister_rfkill(i); + + device_remove_file(&adevice->dev, &dev_attr_camera_power); + + for (i = IDEAPAD_DEV_WLAN; i <= IDEAPAD_DEV_KILLSW; i++) + ideapad_unregister_rfkill(adevice, i); + + dev_set_drvdata(&adevice->dev, NULL); + kfree(priv); return 0; } -static void ideapad_acpi_notify(struct acpi_device *device, u32 event) +static void ideapad_acpi_notify(struct acpi_device *adevice, u32 event) { - ideapad_sync_rfk_state(); + ideapad_sync_rfk_state(adevice); } static struct acpi_driver ideapad_acpi_driver = { -- cgit v1.2.3-59-g8ed1b From 2016e4a0a1c49b68c9d820e28dadab7080c45d1b Mon Sep 17 00:00:00 2001 From: David Woodhouse Date: Wed, 11 Aug 2010 17:59:51 +0100 Subject: ideapad: Only allow camera state to be set to 0 or 1 Signed-off-by: David Woodhouse --- drivers/platform/x86/ideapad_acpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/x86/ideapad_acpi.c b/drivers/platform/x86/ideapad_acpi.c index 8a89902e5d55..798496353e8c 100644 --- a/drivers/platform/x86/ideapad_acpi.c +++ b/drivers/platform/x86/ideapad_acpi.c @@ -140,7 +140,7 @@ static ssize_t store_ideapad_cam(struct device *dev, return 0; if (sscanf(buf, "%i", &state) != 1) return -EINVAL; - ret = ideapad_dev_set_state(IDEAPAD_DEV_CAMERA, state); + ret = ideapad_dev_set_state(IDEAPAD_DEV_CAMERA, !!state); if (ret < 0) return ret; return count; -- cgit v1.2.3-59-g8ed1b From 54b9b308fa39e5147e2edde3492d8650b957be6c Mon Sep 17 00:00:00 2001 From: David Woodhouse Date: Fri, 13 Aug 2010 00:56:54 +0100 Subject: Call acpi_video_register() in intel_opregion_init() failure path If i915 opregion is present, the acpi_video driver doesn't register itself immediately; it defers that until the i915 opregion code is done. But if that *fails*, the acpi_video driver was never getting registered. And thus I have no backlight support on my Lenovo IdeaPad S10-3. Call acpi_video_register() on the failure path, and it works again. Signed-off-by: David Woodhouse Acked-by: Jesse Barnes Acked-by: Matthew Garrett --- drivers/gpu/drm/i915/i915_opregion.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/i915_opregion.c b/drivers/gpu/drm/i915/i915_opregion.c index 8fcc75c1aa28..d1bf92b99788 100644 --- a/drivers/gpu/drm/i915/i915_opregion.c +++ b/drivers/gpu/drm/i915/i915_opregion.c @@ -535,6 +535,7 @@ int intel_opregion_init(struct drm_device *dev, int resume) err_out: iounmap(opregion->header); opregion->header = NULL; + acpi_video_register(); return err; } -- cgit v1.2.3-59-g8ed1b