Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-09 Thread Grygorii Strashko
On 10/07/2017 07:42 PM, Linus Walleij wrote: > On Tue, Oct 3, 2017 at 7:00 PM, Grygorii Strashko > wrote: > >> New GPIO IRQs are allocated and mapped dynamically by default when >> GPIO IRQ infrastructure is used by cherryview-pinctrl driver. >> This causes issues on some Intel platforms [1][2]

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-07 Thread Linus Walleij
On Tue, Oct 3, 2017 at 7:00 PM, Grygorii Strashko wrote: > New GPIO IRQs are allocated and mapped dynamically by default when > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > This causes issues on some Intel platforms [1][2] with broken BIOS which > hardcodes Linux IRQ numbers in

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-06 Thread Chris Gorman
Hello all, Apologies if you are receiving this acknowledgement a second time. I have tested this patch on an Intel_Strago chromebook (Acer cb3-532) and it works to enable the keyboard. I tested it on 4.14.0-rc3. Let me know if you want me to test the patch on the stable branch. Tested-by: Chri

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-06 Thread Mika Westerberg
On Wed, Oct 04, 2017 at 09:42:49AM +0300, Mika Westerberg wrote: > On Tue, Oct 03, 2017 at 12:00:49PM -0500, Grygorii Strashko wrote: > > New GPIO IRQs are allocated and mapped dynamically by default when > > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > > This causes issues on so

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-04 Thread Chris Gorman
Tested on 4.14.0-rc3 and this works for me. Tested by: Chris Gorman On Tue, Oct 3, 2017 at 1:00 PM, Grygorii Strashko wrote: > New GPIO IRQs are allocated and mapped dynamically by default when > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > This causes issues on some Intel pl

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-04 Thread Mika Westerberg
On Wed, Oct 04, 2017 at 12:07:34PM -0400, Chris Gorman wrote: > Tested-by: Chris Gorman I guess you sent this because of the "Tested-by", right? It is enough if you reply to the original thread (the one posted by Grygorii yesterday) and say something like: Works for me, Tested-by: ... Then

[PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-04 Thread Chris Gorman
From: Grygorii Strashko New GPIO IRQs are allocated and mapped dynamically by default when GPIO IRQ infrastructure is used by cherryview-pinctrl driver. This causes issues on some Intel platforms [1] with broken BIOS which hardcodes Linux IRQ numbers in their ACPI tables instead of GPIO number.

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Mika Westerberg
On Tue, Oct 03, 2017 at 12:00:49PM -0500, Grygorii Strashko wrote: > New GPIO IRQs are allocated and mapped dynamically by default when > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > This causes issues on some Intel platforms [1][2] with broken BIOS which > hardcodes Linux IRQ nu

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Mika Westerberg
On Tue, Oct 03, 2017 at 08:54:30PM +0300, Andy Shevchenko wrote: > I'm not sure about this approach. It might be some other broken BIOS > discovered with some other driver (like pinctrl-baytrail.c as an > hypothetical example). Let's deal that separately if it ever happens. > Alternative is to de

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Grygorii Strashko
On 10/03/2017 12:54 PM, Andy Shevchenko wrote: > On Tue, Oct 3, 2017 at 8:00 PM, Grygorii Strashko > wrote: >> New GPIO IRQs are allocated and mapped dynamically by default when >> GPIO IRQ infrastructure is used by cherryview-pinctrl driver. >> This causes issues on some Intel platforms [1][2]

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Andy Shevchenko
On Tue, Oct 3, 2017 at 8:00 PM, Grygorii Strashko wrote: > New GPIO IRQs are allocated and mapped dynamically by default when > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > This causes issues on some Intel platforms [1][2] with broken BIOS which > hardcodes Linux IRQ numbers in

[PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Grygorii Strashko
New GPIO IRQs are allocated and mapped dynamically by default when GPIO IRQ infrastructure is used by cherryview-pinctrl driver. This causes issues on some Intel platforms [1][2] with broken BIOS which hardcodes Linux IRQ numbers in their ACPI tables. On such platforms cherryview-pinctrl driver sh