]> git.baikalelectronics.ru Git - kernel.git/commit
gpiolib: acpi: Make set-debounce-timeout failures non fatal
authorHans de Goede <hdegoede@redhat.com>
Mon, 16 Aug 2021 10:41:19 +0000 (12:41 +0200)
committerBartosz Golaszewski <brgl@bgdev.pl>
Wed, 22 Sep 2021 09:16:42 +0000 (11:16 +0200)
commit32ebc94b2ba386bba02318d79ed6b1e143f54bde
tree30d3d7cd36e892965dec03b3fb409b625ef2a232
parentfa5d4460cfe6a4ab811a36d49fe44ccb61944b70
gpiolib: acpi: Make set-debounce-timeout failures non fatal

Commit 2c28dfb4dc83 ("gpiolib: acpi: Take into account debounce settings")
made the gpiolib-acpi code call gpio_set_debounce_timeout() when requesting
GPIOs.

This in itself is fine, but it also made gpio_set_debounce_timeout()
errors fatal, causing the requesting of the GPIO to fail. This is causing
regressions. E.g. on a HP ElitePad 1000 G2 various _AEI specified GPIO
ACPI event sources specify a debouncy timeout of 20 ms, but the
pinctrl-baytrail.c only supports certain fixed values, the closest
ones being 12 or 24 ms and pinctrl-baytrail.c responds with -EINVAL
when specified a value which is not one of the fixed values.

This is causing the acpi_request_own_gpiod() call to fail for 3
ACPI event sources on the HP ElitePad 1000 G2, which in turn is causing
e.g. the battery charging vs discharging status to never get updated,
even though a charger has been plugged-in or unplugged.

Make gpio_set_debounce_timeout() errors non fatal, warning about the
failure instead, to fix this regression.

Note we should probably also fix various pinctrl drivers to just
pick the first bigger discrete value rather then returning -EINVAL but
this will need to be done on a per driver basis, where as this fix
at least gets us back to where things were before and thus restores
functionality on devices where this was lost due to
gpio_set_debounce_timeout() errors.

Fixes: 2c28dfb4dc83 ("gpiolib: acpi: Take into account debounce settings")
Depends-on: 4abb7772c880 ("gpiolib: acpi: Extract acpi_request_own_gpiod() helper")
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
drivers/gpio/gpiolib-acpi.c