Re: [PATCH v1] gpiolib: acpi: Switch to cansleep version of GPIO library call

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 13, 2018 at 6:00 PM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:

> The commit ca876c7483b6
>
>   ("gpiolib-acpi: make sure we trigger edge events at least once on boot")
>
> added a initial value check for pin which is about to be locked as IRQ.
> Unfortunately, not all GPIO drivers can do that atomically. Thus,
> switch to cansleep version of the call. Otherwise we have a warning:
>
> ...
>   WARNING: CPU: 2 PID: 1408 at drivers/gpio/gpiolib.c:2883 gpiod_get_value+0x46/0x50
> ...
>   RIP: 0010:gpiod_get_value+0x46/0x50
> ...
>
> The change tested on Intel Broxton with Whiskey Cove PMIC GPIO controller.
>
> Fixes: ca876c7483b6 ("gpiolib-acpi: make sure we trigger edge events at least once on boot")
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Cc: Hans de Goede <hdegoede@xxxxxxxxxx>
> Cc: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
> Cc: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>

Patch applied for fixes with Mika's ACK.

Yours,
Linus Walleij



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux