* [PATCH] pinctrl: intel: Fix Jasperlake hostown offset
@ 2020-11-10 22:49 Evan Green
[not found] ` <CAHp75VcfmJ6-cqCsZ6BjbghGDt+w-AbxGxLoWG61VVF2Knor-Q@mail.gmail.com>
2020-11-11 22:14 ` Andy Shevchenko
0 siblings, 2 replies; 5+ messages in thread
From: Evan Green @ 2020-11-10 22:49 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Evan Green, stable, Linus Walleij, Mika Westerberg, linux-gpio,
linux-kernel
GPIOs that attempt to use interrupts get thwarted with a message like:
"pin 161 cannot be used as IRQ" (for instance with SD_CD). This is because
the JSL_HOSTSW_OWN offset is incorrect, so every GPIO looks like it's
owned by ACPI.
Signed-off-by: Evan Green <evgreen@chromium.org>
Fixes: e278dcb7048b1 ("pinctrl: intel: Add Intel Jasper Lake pin
controller support")
Cc: stable@vger.kernel.org
---
drivers/pinctrl/intel/pinctrl-jasperlake.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/intel/pinctrl-jasperlake.c b/drivers/pinctrl/intel/pinctrl-jasperlake.c
index 9bd0e8e6310c3..283698cf0dc7d 100644
--- a/drivers/pinctrl/intel/pinctrl-jasperlake.c
+++ b/drivers/pinctrl/intel/pinctrl-jasperlake.c
@@ -16,7 +16,7 @@
#define JSL_PAD_OWN 0x020
#define JSL_PADCFGLOCK 0x080
-#define JSL_HOSTSW_OWN 0x0b0
+#define JSL_HOSTSW_OWN 0x0c0
#define JSL_GPI_IS 0x100
#define JSL_GPI_IE 0x120
--
2.26.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] pinctrl: intel: Fix Jasperlake hostown offset
[not found] ` <CAHp75VcfmJ6-cqCsZ6BjbghGDt+w-AbxGxLoWG61VVF2Knor-Q@mail.gmail.com>
@ 2020-11-11 0:03 ` Evan Green
2020-11-11 12:16 ` Andy Shevchenko
0 siblings, 1 reply; 5+ messages in thread
From: Evan Green @ 2020-11-11 0:03 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Andy Shevchenko, stable, Linus Walleij, Mika Westerberg,
linux-gpio, linux-kernel
On Tue, Nov 10, 2020 at 3:48 PM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
>
>
> On Wednesday, November 11, 2020, Evan Green <evgreen@chromium.org> wrote:
>>
>> GPIOs that attempt to use interrupts get thwarted with a message like:
>> "pin 161 cannot be used as IRQ" (for instance with SD_CD). This is because
>> the JSL_HOSTSW_OWN offset is incorrect, so every GPIO looks like it's
>> owned by ACPI.
>
>
> Funny, I have created a similar patch few hours ago. Are you sure this is enough? In mine I have also padcfglock updated. But I have to confirm that, that’s why I didn’t send it out.
Oh weird! I didn't check padcfglock since it didn't happen to be
involved in the bug I was tracking down. I was trying to clean out
some skeletons in my kernel closet [1] and debugged it down to this.
If you want to smash the two patches together I'm fine with that. Let
me know, and CC me if you do post something.
-Evan
[1] https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/master/overlay-dedede/sys-kernel/chromeos-kernel-5_4/files/0001-CHROMIUM-pinctrl-intel-Allow-pin-as-IRQ-even-in-ACPI.patch
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pinctrl: intel: Fix Jasperlake hostown offset
2020-11-11 0:03 ` Evan Green
@ 2020-11-11 12:16 ` Andy Shevchenko
0 siblings, 0 replies; 5+ messages in thread
From: Andy Shevchenko @ 2020-11-11 12:16 UTC (permalink / raw)
To: Evan Green
Cc: Andy Shevchenko, stable, Linus Walleij, Mika Westerberg,
linux-gpio, linux-kernel
On Tue, Nov 10, 2020 at 04:03:57PM -0800, Evan Green wrote:
> On Tue, Nov 10, 2020 at 3:48 PM Andy Shevchenko
> <andy.shevchenko@gmail.com> wrote:
> > On Wednesday, November 11, 2020, Evan Green <evgreen@chromium.org> wrote:
> >>
> >> GPIOs that attempt to use interrupts get thwarted with a message like:
> >> "pin 161 cannot be used as IRQ" (for instance with SD_CD). This is because
> >> the JSL_HOSTSW_OWN offset is incorrect, so every GPIO looks like it's
> >> owned by ACPI.
> >
> >
> > Funny, I have created a similar patch few hours ago. Are you sure this is enough? In mine I have also padcfglock updated. But I have to confirm that, that’s why I didn’t send it out.
>
> Oh weird! I didn't check padcfglock since it didn't happen to be
> involved in the bug I was tracking down. I was trying to clean out
> some skeletons in my kernel closet [1] and debugged it down to this.
>
> If you want to smash the two patches together I'm fine with that. Let
> me know, and CC me if you do post something.
Can you test that 0x90 is correct value for padcfglock offset?
> [1] https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/master/overlay-dedede/sys-kernel/chromeos-kernel-5_4/files/0001-CHROMIUM-pinctrl-intel-Allow-pin-as-IRQ-even-in-ACPI.patch
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pinctrl: intel: Fix Jasperlake hostown offset
2020-11-10 22:49 [PATCH] pinctrl: intel: Fix Jasperlake hostown offset Evan Green
[not found] ` <CAHp75VcfmJ6-cqCsZ6BjbghGDt+w-AbxGxLoWG61VVF2Knor-Q@mail.gmail.com>
@ 2020-11-11 22:14 ` Andy Shevchenko
2020-11-11 23:16 ` Evan Green
1 sibling, 1 reply; 5+ messages in thread
From: Andy Shevchenko @ 2020-11-11 22:14 UTC (permalink / raw)
To: Evan Green
Cc: Andy Shevchenko, stable, Linus Walleij, Mika Westerberg,
linux-gpio, linux-kernel
On Tue, Nov 10, 2020 at 02:49:49PM -0800, Evan Green wrote:
> GPIOs that attempt to use interrupts get thwarted with a message like:
> "pin 161 cannot be used as IRQ" (for instance with SD_CD). This is because
> the JSL_HOSTSW_OWN offset is incorrect, so every GPIO looks like it's
Simply HOSTSW_OWN, and same spelling in the subject, please.
> owned by ACPI.
> Signed-off-by: Evan Green <evgreen@chromium.org>
> Fixes: e278dcb7048b1 ("pinctrl: intel: Add Intel Jasper Lake pin
> controller support")
It must be one line, and put it first in the tag block
> Cc: stable@vger.kernel.org
This is second one...
Fixes: ...
Cc: ...
SoB: ...
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pinctrl: intel: Fix Jasperlake hostown offset
2020-11-11 22:14 ` Andy Shevchenko
@ 2020-11-11 23:16 ` Evan Green
0 siblings, 0 replies; 5+ messages in thread
From: Evan Green @ 2020-11-11 23:16 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Andy Shevchenko, stable, Linus Walleij, Mika Westerberg,
linux-gpio, LKML
On Wed, Nov 11, 2020 at 2:13 PM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> On Tue, Nov 10, 2020 at 02:49:49PM -0800, Evan Green wrote:
> > GPIOs that attempt to use interrupts get thwarted with a message like:
> > "pin 161 cannot be used as IRQ" (for instance with SD_CD). This is because
> > the JSL_HOSTSW_OWN offset is incorrect, so every GPIO looks like it's
>
> Simply HOSTSW_OWN, and same spelling in the subject, please.
>
> > owned by ACPI.
>
>
> > Signed-off-by: Evan Green <evgreen@chromium.org>
>
> > Fixes: e278dcb7048b1 ("pinctrl: intel: Add Intel Jasper Lake pin
> > controller support")
>
> It must be one line, and put it first in the tag block
>
>
> > Cc: stable@vger.kernel.org
>
> This is second one...
>
> Fixes: ...
> Cc: ...
> SoB: ...
Thanks, will fix these things, spinning now.
-Evan
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-11-12 1:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-10 22:49 [PATCH] pinctrl: intel: Fix Jasperlake hostown offset Evan Green
[not found] ` <CAHp75VcfmJ6-cqCsZ6BjbghGDt+w-AbxGxLoWG61VVF2Knor-Q@mail.gmail.com>
2020-11-11 0:03 ` Evan Green
2020-11-11 12:16 ` Andy Shevchenko
2020-11-11 22:14 ` Andy Shevchenko
2020-11-11 23:16 ` Evan Green
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.