lkml.org 
[lkml]   [2022]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH AUTOSEL 4.9 1/6] ARM: dts: imx7d-sdb: config the max pressure for tsc2046
Hi!

> From: Haibo Chen <haibo.chen@nxp.com>
>
> [ Upstream commit e7c4ebe2f9cd68588eb24ba4ed122e696e2d5272 ]
>
> Use the general touchscreen method to config the max pressure for
> touch tsc2046(data sheet suggest 8 bit pressure), otherwise, for
> ABS_PRESSURE, when config the same max and min value, weston will
> meet the following issue,
>
> [17:19:39.183] event1 - ADS7846 Touchscreen: is tagged by udev as: Touchscreen
> [17:19:39.183] event1 - ADS7846 Touchscreen: kernel bug: device has min == max on ABS_PRESSURE
> [17:19:39.183] event1 - ADS7846 Touchscreen: was rejected
> [17:19:39.183] event1 - not using input device '/dev/input/event1'
>
> This will then cause the APP weston-touch-calibrator can't list touch devices.
>
> root@imx6ul7d:~# weston-touch-calibrator
> could not load cursor 'dnd-move'
> could not load cursor 'dnd-copy'
> could not load cursor 'dnd-none'
> No devices listed.
>
> And accroding to binding Doc, "ti,x-max", "ti,y-max", "ti,pressure-max"
> belong to the deprecated properties, so remove them. Also for "ti,x-min",
> "ti,y-min", "ti,x-plate-ohms", the value set in dts equal to the default
> value in driver, so are redundant, also remove here.

Did someone check the source code in 4.9? AFAICT it still tries to use
the properties, and probing seems to be different from newer kernels.

Best regards,
Pavel
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2022-10-17 13:16    [W:0.127 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site