ARM: dts: qcom: msm8974-samsung-lt03lte: enable wacom digitizer + touchkeys#32
ARM: dts: qcom: msm8974-samsung-lt03lte: enable wacom digitizer + touchkeys#32vrajashkr wants to merge 4 commits into
Conversation
samsung-lt03lte has 2 touch-sensitive keys on the front of the device for Menu and Back. Both these keys are managed by the Atmel Maxtouch driver and hardware. This change adds the necessary DTS changes to enable this functionality. Signed-off-by: Vishwas Rajashekar <dev@vrajashkr.com>
|
Testing the Touchkeys with evtest: |
|
Testing the Wacom digitizer with the passive stylus: |
| power-source = <PM8941_GPIO_S3>; | ||
| }; | ||
|
|
||
| stylus_insert_pin: stylus-insert-state { |
There was a problem hiding this comment.
It doesn't appear to be used in the mainline driver, but was used in the old downstream driver so I had added it to keep the pins in the right shape earlier.
I've removed this now. Thanks for the feedback!
|
|
||
| stylus_default_state: stylus-default-state { | ||
| pdct-pins { | ||
| pins = "gpio145"; |
There was a problem hiding this comment.
this gpio is not referenced in the node above? No clue if "pdct" is important - whatever that is
There was a problem hiding this comment.
If I'm not mistaken, pdct is short for pen detect. There was some conversation about it in the mailing list on the driver patch. I believe it was ultimately removed from the driver. I've removed it too.
Thanks!
|
Fwiw the latest revision of the patch series is v7 https://lore.kernel.org/all/20260501124002.132540-1-hendrik-noack@gmx.de/ Would be good to pick that one now, and prefix the patches with "FROMLIST v7 " in the first line of the commit message of those two commits/patches. |
Add bindings for Wacom W9002 and two Wacom W9007 variants which can be found in tablets. W9002, W9007A LT03, and W9007A V1 differ in the length of the return message containing coordinates, distance, pressure and button status. Co-developed-by: Ferass El Hafidi <funderscore@postmarketos.org> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org> Signed-off-by: Hendrik Noack <hendrik-noack@gmx.de> Acked-by: Conor Dooley <conor.dooley@microchip.com> Link: https://patch.msgid.link/20260528074818.12151-2-hendrik-noack@gmx.de Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Add driver for Wacom W9002 and two Wacom W9007A variants. These are penabled touchscreens supporting passive Wacom Pens and use I2C. Co-developed-by: Ferass El Hafidi <funderscore@postmarketos.org> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org> Signed-off-by: Hendrik Noack <hendrik-noack@gmx.de> Link: https://patch.msgid.link/20260528074818.12151-3-hendrik-noack@gmx.de Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This change enables support for the Wacom digitizer (passive stylus) for samsung-lt03lte. This includes the digitizer node and the fixed regulator. Signed-off-by: Vishwas Rajashekar <dev@vrajashkr.com>
6907371 to
7d93f85
Compare
Sorry it took me a while to get back to this. Fortunately, it looks like the patch has been accepted and the commits are present in linux-next at the time of writing. I've fetched the patches from there and prefixed them with FROMGIT instead (I came across a few commits with this prefix in the commit history). Let me know in case I should be using a different prefix. |
|
Tested with a fresh kernel build: Seems to be working :) |
For samsung-lt03lte, enables the wacom digitizer (passive stylus) and the front touchkeys.
The Wacom W9007 driver has been backported from an active kernel patch series.ref: https://patchew.org/linux/20260319095303.19927-1-hendrik-noack@gmx.de/20260319095303.19927-2-hendrik-noack@gmx.de/The Wacom W9007 driver patches have been taken from linux-next:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=185cd1d8cb7f0fd30d868fa12d294bba5997b7c7
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=acaefbacc997ae79527a5ade8720221dfe17407e