mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts
@ 2026-11-04 19:17 Krzysztof Kozlowski
  2026-11-04 19:17 ` [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges Krzysztof Kozlowski
  2026-11-05  8:07 ` [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Konrad Dybcio
  0 siblings, 2 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-11-04 19:17 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio, Abel Vesa, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel
  Cc: Krzysztof Kozlowski, Sashiko, stable

Interrupts property takes a number, not a phandle.  Passing the TLMM
phandle made most likely the interrupt as broken, which could affect
touch operation.

Reported-by: Sashiko <sashiko-bot@kernel.org>
Closes: https://sashiko.dev/#/bug/linux-759fcb27-d081-4ec4-9683-6667b54fa3dc
Fixes: c636eeb751f6 ("arm64: dts: qcom: msm8994-octagon: Add AD7147 and APDS9930 sensors")
Cc: <stable@vger.kernel.org>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
---
 arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon.dtsi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon.dtsi b/arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon.dtsi
index 7ace3540ef0a..63dd5a1a1cd9 100644
--- a/arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon.dtsi
@@ -385,7 +385,8 @@ sideinteraction: touch@2c {
 		pinctrl-0 = <&grip_default>;
 		pinctrl-1 = <&grip_sleep>;
 
-		interrupts = <&tlmm 96 IRQ_TYPE_EDGE_FALLING>;
+		interrupt-parent = <&tlmm>;
+		interrupts = <96 IRQ_TYPE_EDGE_FALLING>;
 
 		vcc-supply = <&vreg_l18a_2p85>;
 	};
-- 
2.53.0


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

* [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges
  2026-11-04 19:17 [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Krzysztof Kozlowski
@ 2026-11-04 19:17 ` Krzysztof Kozlowski
  2026-11-05  8:08   ` Konrad Dybcio
  2026-11-05 13:17   ` Abel Vesa
  2026-11-05  8:07 ` [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Konrad Dybcio
  1 sibling, 2 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-11-04 19:17 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio, Abel Vesa, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel
  Cc: Krzysztof Kozlowski, Sashiko

TLMM on Eliza SoC has 185 GPIOs and UFS_RESET line, thus the last number
in gpio-ranges, representing the number of pins in pinctrl, should be
186 so the last pins can be configured in pinctrl.

Reported-by: Sashiko <sashiko-bot@kernel.org>
Closes: https://sashiko.dev/#/bug/linux-0e88535a-02d3-4a42-9f3c-ca4f9bdd3b11
Fixes: af20af39fc09 ("arm64: dts: qcom: Introduce Eliza Soc base dtsi")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
---
 arch/arm64/boot/dts/qcom/eliza.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/eliza.dtsi b/arch/arm64/boot/dts/qcom/eliza.dtsi
index a8042291bd07..df7426134722 100644
--- a/arch/arm64/boot/dts/qcom/eliza.dtsi
+++ b/arch/arm64/boot/dts/qcom/eliza.dtsi
@@ -3577,7 +3577,7 @@ tlmm: pinctrl@f100000 {
 			interrupt-controller;
 			#interrupt-cells = <2>;
 
-			gpio-ranges = <&tlmm 0 0 184>;
+			gpio-ranges = <&tlmm 0 0 186>;
 			wakeup-parent = <&pdc>;
 
 			pcie0_default_state: pcie0-default-state {
-- 
2.53.0


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

* Re: [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts
  2026-11-04 19:17 [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Krzysztof Kozlowski
  2026-11-04 19:17 ` [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges Krzysztof Kozlowski
@ 2026-11-05  8:07 ` Konrad Dybcio
  2026-11-05  8:29   ` Krzysztof Kozlowski
  1 sibling, 1 reply; 6+ messages in thread
From: Konrad Dybcio @ 2026-11-05  8:07 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Bjorn Andersson, Konrad Dybcio, Abel Vesa,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel
  Cc: Sashiko, stable

On 11/4/26 8:17 PM, Krzysztof Kozlowski wrote:
> Interrupts property takes a number, not a phandle.  Passing the TLMM
> phandle made most likely the interrupt as broken, which could affect
> touch operation.
> 
> Reported-by: Sashiko <sashiko-bot@kernel.org>
> Closes: https://sashiko.dev/#/bug/linux-759fcb27-d081-4ec4-9683-6667b54fa3dc
> Fixes: c636eeb751f6 ("arm64: dts: qcom: msm8994-octagon: Add AD7147 and APDS9930 sensors")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> ---

fwiw

https://lore.kernel.org/linux-arm-msm/20260911-topic-lumia_drop_captouch-v1-1-6f92f1f3c6d3@oss.qualcomm.com/

Konrad

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

* Re: [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges
  2026-11-04 19:17 ` [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges Krzysztof Kozlowski
@ 2026-11-05  8:08   ` Konrad Dybcio
  2026-11-05 13:17   ` Abel Vesa
  1 sibling, 0 replies; 6+ messages in thread
From: Konrad Dybcio @ 2026-11-05  8:08 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Bjorn Andersson, Konrad Dybcio, Abel Vesa,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel
  Cc: Sashiko

On 11/4/26 8:17 PM, Krzysztof Kozlowski wrote:
> TLMM on Eliza SoC has 185 GPIOs and UFS_RESET line, thus the last number
> in gpio-ranges, representing the number of pins in pinctrl, should be
> 186 so the last pins can be configured in pinctrl.
> 
> Reported-by: Sashiko <sashiko-bot@kernel.org>
> Closes: https://sashiko.dev/#/bug/linux-0e88535a-02d3-4a42-9f3c-ca4f9bdd3b11
> Fixes: af20af39fc09 ("arm64: dts: qcom: Introduce Eliza Soc base dtsi")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> ---

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

Konrad

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

* Re: [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts
  2026-11-05  8:07 ` [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Konrad Dybcio
@ 2026-11-05  8:29   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-11-05  8:29 UTC (permalink / raw)
  To: Konrad Dybcio, Bjorn Andersson, Konrad Dybcio, Abel Vesa,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel
  Cc: Sashiko, stable

On 05/11/2026 09:07, Konrad Dybcio wrote:
> On 11/4/26 8:17 PM, Krzysztof Kozlowski wrote:
>> Interrupts property takes a number, not a phandle.  Passing the TLMM
>> phandle made most likely the interrupt as broken, which could affect
>> touch operation.
>>
>> Reported-by: Sashiko <sashiko-bot@kernel.org>
>> Closes: https://sashiko.dev/#/bug/linux-759fcb27-d081-4ec4-9683-6667b54fa3dc
>> Fixes: c636eeb751f6 ("arm64: dts: qcom: msm8994-octagon: Add AD7147 and APDS9930 sensors")
>> Cc: <stable@vger.kernel.org>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>> ---
> 
> fwiw
> 
> https://lore.kernel.org/linux-arm-msm/20260911-topic-lumia_drop_captouch-v1-1-6f92f1f3c6d3@oss.qualcomm.com/

This solves the problem :)

Best regards,
Krzysztof

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

* Re: [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges
  2026-11-04 19:17 ` [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges Krzysztof Kozlowski
  2026-11-05  8:08   ` Konrad Dybcio
@ 2026-11-05 13:17   ` Abel Vesa
  1 sibling, 0 replies; 6+ messages in thread
From: Abel Vesa @ 2026-11-05 13:17 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Bjorn Andersson, Konrad Dybcio, Abel Vesa, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Gustave Monce,
	Dmitry Baryshkov, linux-arm-msm, devicetree, linux-kernel,
	Sashiko

On 26-11-04 20:17:02, Krzysztof Kozlowski wrote:
> TLMM on Eliza SoC has 185 GPIOs and UFS_RESET line, thus the last number
> in gpio-ranges, representing the number of pins in pinctrl, should be
> 186 so the last pins can be configured in pinctrl.
> 
> Reported-by: Sashiko <sashiko-bot@kernel.org>
> Closes: https://sashiko.dev/#/bug/linux-0e88535a-02d3-4a42-9f3c-ca4f9bdd3b11
> Fixes: af20af39fc09 ("arm64: dts: qcom: Introduce Eliza Soc base dtsi")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com>

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

end of thread, other threads:[~2026-11-05 13:17 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-04 19:17 [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Krzysztof Kozlowski
2026-11-04 19:17 ` [PATCH 2/2] arm64: dts: qcom: eliza: Correct number of TLMM pins in gpio-ranges Krzysztof Kozlowski
2026-11-05  8:08   ` Konrad Dybcio
2026-11-05 13:17   ` Abel Vesa
2026-11-05  8:07 ` [PATCH 1/2] arm64: dts: qcom: msm8994-octagon: Fix side touch interrupts Konrad Dybcio
2026-11-05  8:29   ` Krzysztof Kozlowski

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®