lkml.org 
[lkml]   [2017]   [Jan]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH v3] iio: light: acpi-als: Properly enable on ASUS Zenbooks
ASUS Zenbooks need several special ACPI calls to enable the ALS peripheral.
Otherwise, reads just return 0.

Signed-off-by: Josef Gajdusek <atx@atx.name>
---
drivers/iio/light/acpi-als.c | 60 ++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 60 insertions(+)

diff --git a/drivers/iio/light/acpi-als.c b/drivers/iio/light/acpi-als.c
index f0b47c5..43417c9 100644
--- a/drivers/iio/light/acpi-als.c
+++ b/drivers/iio/light/acpi-als.c
@@ -30,6 +30,7 @@
#include <linux/acpi.h>
#include <linux/err.h>
#include <linux/mutex.h>
+#include <linux/dmi.h>

#include <linux/iio/iio.h>
#include <linux/iio/buffer.h>
@@ -39,6 +40,9 @@
#define ACPI_ALS_DEVICE_NAME "acpi-als"
#define ACPI_ALS_NOTIFY_ILLUMINANCE 0x80

+#define ACPI_ALS_ASUS_TALS "\\_SB.PCI0.LPCB.EC0.TALS"
+#define ACPI_ALS_ASUS_ALSC "\\_SB.ATKD.ALSC"
+
ACPI_MODULE_NAME("acpi-als");

/*
@@ -170,6 +174,46 @@ static int acpi_als_read_raw(struct iio_dev *indio_dev,
return IIO_VAL_INT;
}

+static int acpi_als_quirk_asus_zenbook_add(struct acpi_als *als)
+{
+ acpi_status status;
+
+ status = acpi_execute_simple_method(NULL, ACPI_ALS_ASUS_TALS, 1);
+ if (ACPI_FAILURE(status)) {
+ ACPI_EXCEPTION((AE_INFO, status, "Error enabling ALS by %s",
+ ACPI_ALS_ASUS_TALS));
+ return -EIO;
+ }
+ status = acpi_execute_simple_method(NULL, ACPI_ALS_ASUS_ALSC, 1);
+ if (ACPI_FAILURE(status)) {
+ ACPI_EXCEPTION((AE_INFO, status, "Error enabling ALS by %s",
+ ACPI_ALS_ASUS_ALSC));
+ return -EIO;
+ }
+
+ return 0;
+}
+
+struct quirk_entry {
+ int (*add)(struct acpi_als *als);
+};
+
+static struct quirk_entry acpi_als_quirk_asus_zenbook = {
+ .add = acpi_als_quirk_asus_zenbook_add
+};
+
+static const struct dmi_system_id acpi_als_quirks[] = {
+ {
+ .ident = "ASUSTeK COMPUTER INC. UX",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "UX"),
+ },
+ .driver_data = &acpi_als_quirk_asus_zenbook,
+ },
+ {},
+};
+
static const struct iio_info acpi_als_info = {
.driver_module = THIS_MODULE,
.read_raw = acpi_als_read_raw,
@@ -180,6 +224,9 @@ static int acpi_als_add(struct acpi_device *device)
struct acpi_als *als;
struct iio_dev *indio_dev;
struct iio_buffer *buffer;
+ const struct dmi_system_id *dmiid;
+ struct quirk_entry *quirk;
+ int ret;

indio_dev = devm_iio_device_alloc(&device->dev, sizeof(*als));
if (!indio_dev)
@@ -191,6 +238,19 @@ static int acpi_als_add(struct acpi_device *device)
als->device = device;
mutex_init(&als->lock);

+ dmiid = dmi_first_match(acpi_als_quirks);
+ if (dmiid) {
+ dev_dbg(&device->dev, "Detected quirk for %s\n", dmiid->ident);
+ quirk = dmiid->driver_data;
+ ret = quirk->add(als);
+ if (ret) {
+ dev_err(&device->dev,
+ "Executing quirk for %s failed - %d",
+ dmiid->ident, ret);
+ return ret;
+ }
+ }
+
indio_dev->name = ACPI_ALS_DEVICE_NAME;
indio_dev->dev.parent = &device->dev;
indio_dev->info = &acpi_als_info;
--
2.10.2
\
 
 \ /
  Last update: 2017-01-19 20:16    [W:0.053 / U:20.796 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site