mbox series

[RESEND,0/1] Add support for touch screens using the General Touch ST6001S controller.

Message ID ac6ad041-c57f-2850-b244-a332c980f154@virgin.net (mailing list archive)
Headers show
Series Add support for touch screens using the General Touch ST6001S controller. | expand

Message

Gareth Randall Oct. 3, 2021, 9:52 p.m. UTC
Add support for touch screens using the General Touch ST6001S
controller, as found in the GPEG model AOD22WZ-ST monitor.
This controller can output the ELO 10-byte protocol,
but requires different initialisation.


However, I would like some advice from the maintainers:

Where my patch contains the following lines, I have set them to match a 
22 inch screen (GPEG model AOD22WZ-ST) that I have been working on, 
based on me moving my finger and watching the position of the mouse 
pointer in X-windows. Are the maintainers satisfied with this or should 
there also be an option to specify them as kernel parameters?

+       // Values taken from a GPEG model AOD22WZ-ST monitor
+       input_set_abs_params(dev, ABS_X, 1365, 5828, 0, 0);
+       // max and min inverted because screen axis is inverted
+       input_set_abs_params(dev, ABS_Y, 5013, 2260, 0, 0);

Note also that the /bin/inputattach command must be extended to support 
using the newly added elo->id value (id=4). I can post my changes to 
this to assist those reviewing this patch but have not here as I'm 
trying to stick to the patch email format.

Thank you for your consideration.


  drivers/input/touchscreen/elo.c | 58 +++++++++++++++++++++++++++++++++
  1 file changed, 58 insertions(+)