diff mbox series

arm64: dts: qcom: c630: keep both touchpad devices enabled

Message ID 20210101022451.21882-1-shawn.guo@linaro.org (mailing list archive)
State Superseded
Headers show
Series arm64: dts: qcom: c630: keep both touchpad devices enabled | expand

Commit Message

Shawn Guo Jan. 1, 2021, 2:24 a.m. UTC
Indicated by AML code in ACPI table, the touchpad in-use could be found
on two possible slave addresses on &i2c3, i.e. hid@15 and hid@2c.  And
which one is in-use can be determined by reading another address on the
I2C bus.  Unfortunately, for DT boot, there is currently no support in
firmware to make this check and patch DT accordingly.  This results in
a non-functional touchpad on those C630 devices with hid@2c.

As i2c-hid driver will stop probing the device if there is nothing on
the slave address, we can actually keep both devices enabled in DT, and
i2c-hid driver will only probe the existing one.  The only problem is
that we cannot set up pinctrl in both device nodes, as two devices with
the same pinctrl will cause pin conflict that makes the second device
fail to probe.  Let's move the pinctrl state up to parent node to solve
this problem.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
 arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

Comments

Konrad Dybcio Jan. 1, 2021, 2:32 p.m. UTC | #1
Hi Shawn,


it seems like by doing so you're overwriting the I2C host pinctrl, so (unless that's intentional) the more correct fix would be:


pinctrl-0 = <&qup_i2c3_default &i2c3_hid_active>;


Konrad
Shawn Guo Jan. 1, 2021, 3:01 p.m. UTC | #2
On Fri, Jan 1, 2021 at 10:32 PM Konrad Dybcio
<konrad.dybcio@somainline.org> wrote:
>
> Hi Shawn,
>
> it seems like by doing so you're overwriting the I2C host pinctrl, so (unless that's intentional) the more correct fix would be:
>
> pinctrl-0 = <&qup_i2c3_default &i2c3_hid_active>;

Ha, yes, I missed that there is already some default pinctrl state set
up in sdm845.dtsi.  I will fix it with v2.

Thanks for spotting it, Konrad!

Shawn
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
index dcfe66f169f9..802966e11152 100644
--- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
+++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
@@ -353,6 +353,8 @@ 
 &i2c3 {
 	status = "okay";
 	clock-frequency = <400000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c3_hid_active>;
 
 	tsel: hid@15 {
 		compatible = "hid-over-i2c";
@@ -360,9 +362,6 @@ 
 		hid-descr-addr = <0x1>;
 
 		interrupts-extended = <&tlmm 37 IRQ_TYPE_LEVEL_HIGH>;
-
-		pinctrl-names = "default";
-		pinctrl-0 = <&i2c3_hid_active>;
 	};
 
 	tsc2: hid@2c {
@@ -371,11 +370,6 @@ 
 		hid-descr-addr = <0x20>;
 
 		interrupts-extended = <&tlmm 37 IRQ_TYPE_LEVEL_HIGH>;
-
-		pinctrl-names = "default";
-		pinctrl-0 = <&i2c3_hid_active>;
-
-		status = "disabled";
 	};
 };