* [PATCH] Input: silead - Add support for ACPI bus power methods
@ 2017-01-01 21:30 Hans de Goede
2017-01-21 17:33 ` Dmitry Torokhov
0 siblings, 1 reply; 3+ messages in thread
From: Hans de Goede @ 2017-01-01 21:30 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: russianneuromancer @ ya . ru, Gregor Riepl, linux-input,
Hans de Goede
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.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
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
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] Input: silead - Add support for ACPI bus power methods
2017-01-01 21:30 [PATCH] Input: silead - Add support for ACPI bus power methods Hans de Goede
@ 2017-01-21 17:33 ` Dmitry Torokhov
2017-01-22 19:54 ` Hans de Goede
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Torokhov @ 2017-01-21 17:33 UTC (permalink / raw)
To: Hans de Goede; +Cc: russianneuromancer @ ya . ru, Gregor Riepl, linux-input
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.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> 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.
--
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] Input: silead - Add support for ACPI bus power methods
2017-01-21 17:33 ` Dmitry Torokhov
@ 2017-01-22 19:54 ` Hans de Goede
0 siblings, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2017-01-22 19:54 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: russianneuromancer @ ya . ru, Gregor Riepl, linux-input
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@redhat.com>
>> ---
>> 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.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-01-22 19:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-01 21:30 [PATCH] Input: silead - Add support for ACPI bus power methods Hans de Goede
2017-01-21 17:33 ` Dmitry Torokhov
2017-01-22 19:54 ` Hans de Goede
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).