From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: "Lothar Waßmann" <LW@KARO-electronics.de>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Henrik Rydberg" <rydberg@euromail.se>
Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
Markus Pargmann <mpa@pengutronix.de>,
Maxime Ripard <maxime.ripard@free-electrons.com>
Subject: [PATCH v2 0/4] input: ft5x06: Fix userspace reported maximum value
Date: Thu, 13 Nov 2014 15:06:54 +0100 [thread overview]
Message-ID: <1415887618-29844-1-git-send-email-maxime.ripard@free-electrons.com> (raw)
Hi,
The current ft5x06 reports to the user-space that its maximum
coordinates are, on both X and Y, way higher than what could be
actually usable on the screen (in my case, 5759x1151 instead of
480x800).
This causes trouble on some userspace stacks that then try to re-scale
these coordinates back to the framebuffer resolution, like QT does.
Use the of_touchscreen code to find the real touchscreen limits in the
DT case, and report that to the userspace.
Thanks,
Maxime
Changes from v1:
- Do not use input_set_abs_params anymore to avoid enabling an axis
that was not enabled by the driver itself in of_touchscreen
- Emit a warning when an axis is set in the DT but not enabled in
the driver
- Remove EV_SYN from the edt-ft5x06 driver
Maxime Ripard (4):
input: touchscreen: of: Use input_set_abs_params
input: touchscreen: of: Register multitouch axes
input: ft5x06: Allow to set the maximum axes value through the DT
input: edt-ft5x06: Remove EV_SYN event report
drivers/input/touchscreen/edt-ft5x06.c | 6 ++-
drivers/input/touchscreen/of_touchscreen.c | 62 +++++++++++++++++++++++++-----
2 files changed, 57 insertions(+), 11 deletions(-)
--
2.1.1
next reply other threads:[~2014-11-13 14:10 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-13 14:06 Maxime Ripard [this message]
2014-11-13 14:06 ` [PATCH v2 1/4] input: touchscreen: of: Use input_set_abs_params Maxime Ripard
2014-11-13 14:06 ` [PATCH v2 2/4] input: touchscreen: of: Register multitouch axes Maxime Ripard
2014-11-13 14:06 ` [PATCH v2 3/4] input: ft5x06: Allow to set the maximum axes value through the DT Maxime Ripard
2014-11-13 14:06 ` [PATCH v2 4/4] input: edt-ft5x06: Remove EV_SYN event report Maxime Ripard
2015-02-23 21:49 ` [PATCH v2 0/4] input: ft5x06: Fix userspace reported maximum value Markus Pargmann
2015-02-24 16:55 ` Maxime Ripard
2015-02-26 15:26 ` Markus Pargmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1415887618-29844-1-git-send-email-maxime.ripard@free-electrons.com \
--to=maxime.ripard@free-electrons.com \
--cc=LW@KARO-electronics.de \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mpa@pengutronix.de \
--cc=rydberg@euromail.se \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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®