Hi,
On 21-01-17 18:33, Dmitry Torokhov wrote:
Hi Hans,
On Sun, Jan 01, 2017 at 10:30:04PM +0100, Hans de Goede wrote:
On some x86 tablets we cannot directly access the GPIOs as they are
claimed by the ACPI tables, so first try using acpi_bus_set_power
and if that works use that instead of directly accessing the GPIO.
This does not belong to individual drivers but either to various buses
(spi, i2c, etc), or to the ACPI power domain code.
I agree with you on this, and upon checking it turns out that the i2c
core already connects the acpi power-domain to the struct device for
the i2c client.
So all we need to do is not touch the gpio when ACPI is doing the
powermanagent I'll send a new version for this.
Regards,
Hans
Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
---
drivers/input/touchscreen/silead.c | 32 +++++++++++++++++++++++++-------
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/drivers/input/touchscreen/silead.c b/drivers/input/touchscreen/silead.c
index f502c84..d3dd9bb 100644
--- a/drivers/input/touchscreen/silead.c
+++ b/drivers/input/touchscreen/silead.c
@@ -30,6 +30,7 @@
#include <linux/pm.h>
#include <linux/irq.h>
+#include <acpi/acpi_bus.h>
#include <asm/unaligned.h>
#define SILEAD_TS_NAME "silead_ts"
@@ -71,6 +72,7 @@ enum silead_ts_power {
struct silead_ts_data {
struct i2c_client *client;
+ bool acpi_power;
struct gpio_desc *gpio_power;
struct input_dev *input;
char fw_name[64];
@@ -125,7 +127,14 @@ static void silead_ts_set_power(struct i2c_client *client,
{
struct silead_ts_data *data = i2c_get_clientdata(client);
- if (data->gpio_power) {
+ if (data->acpi_power) {
+#ifdef CONFIG_ACPI
+ int ret = acpi_bus_set_power(ACPI_HANDLE(&client->dev),
+ state ? ACPI_STATE_D0 : ACPI_STATE_D3);
+ if (ret)
+ dev_err(&client->dev, "Power error %d\n", ret);
+#endif
+ } else if (data->gpio_power) {
gpiod_set_value_cansleep(data->gpio_power, state);
msleep(SILEAD_POWER_SLEEP);
}
@@ -465,12 +474,21 @@ static int silead_ts_probe(struct i2c_client *client,
if (client->irq <= 0)
return -ENODEV;
- /* Power GPIO pin */
- data->gpio_power = devm_gpiod_get_optional(dev, "power", GPIOD_OUT_LOW);
- if (IS_ERR(data->gpio_power)) {
- if (PTR_ERR(data->gpio_power) != -EPROBE_DEFER)
- dev_err(dev, "Shutdown GPIO request failed\n");
- return PTR_ERR(data->gpio_power);
+ /* Power handling, try ACPI companion method first then GPIO */
+#ifdef CONFIG_ACPI
+ if (ACPI_COMPANION(&client->dev) &&
+ acpi_bus_set_power(ACPI_HANDLE(&client->dev), ACPI_STATE_D3) == 0)
+ data->acpi_power = true;
+ else
+#endif
+ {
+ data->gpio_power = devm_gpiod_get_optional(dev, "power",
+ GPIOD_OUT_LOW);
+ if (IS_ERR(data->gpio_power)) {
+ if (PTR_ERR(data->gpio_power) != -EPROBE_DEFER)
+ dev_err(dev, "Power GPIO request failed\n");
+ return PTR_ERR(data->gpio_power);
+ }
}
error = silead_ts_setup(client);
--
2.9.3
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
