mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing
@ 2026-07-28 11:03 Komal Bajaj
  2026-07-28 11:09 ` Konrad Dybcio
  2026-07-31  8:31 ` Bartosz Golaszewski
  0 siblings, 2 replies; 3+ messages in thread
From: Komal Bajaj @ 2026-07-28 11:03 UTC (permalink / raw)
  To: Bartosz Golaszewski, Bjorn Andersson, Linus Walleij, Konrad Dybcio
  Cc: linux-arm-msm, linux-gpio, linux-kernel, Komal Bajaj

The intr_target_width field sets the mask width used when writing
target processor into interrupt config register and deciding which
processor receives summary interrupt for a given GPIO.

Without it, pinctrl driver defaults to a 3-bit mask. On Shikra, this
field is 4 bits wide, which could corrupt adjacent bits and mis-route
interrupts. Set intr_target_width = 4 to match the hardware.

Fixes: 9db68ec534c5 ("pinctrl: qcom: Add Shikra pinctrl driver")
Signed-off-by: Komal Bajaj <komal.bajaj@oss.qualcomm.com>
---
 drivers/pinctrl/qcom/pinctrl-shikra.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pinctrl/qcom/pinctrl-shikra.c b/drivers/pinctrl/qcom/pinctrl-shikra.c
index 0fc98369948c..55aec2f675e6 100644
--- a/drivers/pinctrl/qcom/pinctrl-shikra.c
+++ b/drivers/pinctrl/qcom/pinctrl-shikra.c
@@ -44,6 +44,7 @@
 		.intr_status_bit = 0,		\
 		.intr_wakeup_enable_bit = 7,	\
 		.intr_wakeup_present_bit = 6,	\
+		.intr_target_width = 4,		\
 		.intr_target_bit = 8,		\
 		.intr_target_kpss_val = 3,	\
 		.intr_raw_status_bit = 4,	\

---
base-commit: 0d33d21e47d9dc66f91e44da3fc9220c74d93df7
change-id: 20260728-shikra-pinctrl-intr-width-982cc47c24c1

Best regards,
-- 
Komal Bajaj <komal.bajaj@oss.qualcomm.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing
  2026-07-28 11:03 [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing Komal Bajaj
@ 2026-07-28 11:09 ` Konrad Dybcio
  2026-07-31  8:31 ` Bartosz Golaszewski
  1 sibling, 0 replies; 3+ messages in thread
From: Konrad Dybcio @ 2026-07-28 11:09 UTC (permalink / raw)
  To: Komal Bajaj, Bartosz Golaszewski, Bjorn Andersson, Linus Walleij
  Cc: linux-arm-msm, linux-gpio, linux-kernel

On 7/28/26 1:03 PM, Komal Bajaj wrote:
> The intr_target_width field sets the mask width used when writing
> target processor into interrupt config register and deciding which
> processor receives summary interrupt for a given GPIO.
> 
> Without it, pinctrl driver defaults to a 3-bit mask. On Shikra, this
> field is 4 bits wide, which could corrupt adjacent bits and mis-route
> interrupts. Set intr_target_width = 4 to match the hardware.
> 
> Fixes: 9db68ec534c5 ("pinctrl: qcom: Add Shikra pinctrl driver")
> Signed-off-by: Komal Bajaj <komal.bajaj@oss.qualcomm.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing
  2026-07-28 11:03 [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing Komal Bajaj
  2026-07-28 11:09 ` Konrad Dybcio
@ 2026-07-31  8:31 ` Bartosz Golaszewski
  1 sibling, 0 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2026-07-31  8:31 UTC (permalink / raw)
  To: Bartosz Golaszewski, Bjorn Andersson, Linus Walleij,
	Konrad Dybcio, Komal Bajaj
  Cc: Bartosz Golaszewski, linux-arm-msm, linux-gpio, linux-kernel


On Tue, 28 Jul 2026 16:33:08 +0530, Komal Bajaj wrote:
> The intr_target_width field sets the mask width used when writing
> target processor into interrupt config register and deciding which
> processor receives summary interrupt for a given GPIO.
> 
> Without it, pinctrl driver defaults to a 3-bit mask. On Shikra, this
> field is 4 bits wide, which could corrupt adjacent bits and mis-route
> interrupts. Set intr_target_width = 4 to match the hardware.
> 
> [...]

Applied, thanks!

[1/1] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing
      https://git.kernel.org/brgl/c/1e7b04c12c077e8829991833a9aa2cb3bdacab61

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-07-31  8:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-07-28 11:03 [PATCH] pinctrl: qcom: shikra: Fix intr_target_width for summary interrupt routing Komal Bajaj
2026-07-28 11:09 ` Konrad Dybcio
2026-07-31  8:31 ` Bartosz Golaszewski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®