]> git.baikalelectronics.ru Git - kernel.git/commit
pinctrl: baytrail: Replace WARN with dev_info_once when setting direct-irq pin to...
authorHans de Goede <hdegoede@redhat.com>
Wed, 1 Jan 2020 14:52:43 +0000 (15:52 +0100)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 9 Jan 2020 11:12:20 +0000 (13:12 +0200)
commitb485443e8e97059670a67b0febb5f9f6ed3512d7
tree7c3d5692debd00181b5b9bc84fc47a41d10f3d94
parent221414181448666a6b5d1dd134f17591f3faf001
pinctrl: baytrail: Replace WARN with dev_info_once when setting direct-irq pin to output

Suspending Goodix touchscreens requires changing the interrupt pin to
output before sending them a power-down command. Followed by wiggling
the interrupt pin to wake the device up, after which it is put back
in input mode.

On Cherry Trail device the interrupt pin is listed as a GpioInt ACPI
resource so we can do this without problems as long as we release the
IRQ before changing the pin to output mode.

On Bay Trail devices with a Goodix touchscreen direct-irq mode is used
in combination with listing the pin as a normal GpioIo resource. This
works fine, but this triggers the WARN in byt_gpio_set_direction-s output
path because direct-irq support is enabled on the pin.

This commit replaces the WARN call with a dev_info_once call, fixing a
bunch of WARN splats in dmesg on each suspend/resume cycle.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/pinctrl/intel/pinctrl-baytrail.c