|| ||Christopher Heiny <email@example.com> |
|| ||Dmitry Torokhov <firstname.lastname@example.org> |
|| ||[RFC PATCH 0/1] input/touchscreen: Synaptics Touchscreen Driver |
|| ||Mon, 22 Mar 2010 19:07:37 -0700|
|| ||Jean Delvare <email@example.com>,
Linux Kernel <firstname.lastname@example.org>,
Linux Input <email@example.com>,
Christopher Heiny <firstname.lastname@example.org>,
Allie Xiong <email@example.com>,
William Manson <firstname.lastname@example.org>|
|| ||Article, Thread
This patch adds an initial driver supporting Synaptics ClearPad
touchscreens that use the RMI4 protocol, as defined here:
This patch addresses most of the feedback relating to our previous submission.
Any elements not acted on (such as creation of an 'rmi' bus on the kernel
bus architecture) are captured as TODOs in the code, and will be implemented
in a future submission. In addition:
- We think we've resolved all the whitespace issues. For Kconfig,
we used the indent pattern used by other modules. For all other
sources, we've converted to 8-space hard TAB characters.
- Some concerns were expressed regarding our I2C implementation.
Dmitry recommended we contact Jean Delvare for further review,
so we are cc'ing him on this patch.
This version of the driver does not support all features of the RMI4
protocol yet. We felt it more important to start with a solid
implementation of the basic functionality, and then build on that base
to support additional features (such as gestures, proximity detection,
capacitive buttons, and so on).
Additionally, as this is our team's first venture into the wonderful
world of kernel submissions, we figured it was important to start out
initially with a fairly simple codebase, just in case we're doing
something horribly wrong.
The significant files in this version of the driver are:
Implements the basic core of the RMI4 protocol
including self-discovery of RMI4 functions. This
also implements RMI4 function 0x01, which provides
generic device control.
Implements basic RMI4 function 0x11 (2D sensor)
features, including multitouch up to 10 fingers.
The major driver functions (mod_init, mod_exit,
and so on).
An additional file is included, but will eventually be changed/dropped
Provides platform setup for development. This will
be replaced in the future by a generic approach to
specifying platform parameters.
More detailed descriptions can be found in each file's comments.
We chose to use multiple source files because it makes it easy to add
support for additional RMI4 functions with minimal impact to existing
functions. Additionally, this supports future changes to the
configuration to allow you to compile in only those RMI4 functions that
you need for your particular device.
There are two existing drivers for similar Synaptics devices in the
current kernel tree (excluding the PS/2 touchpad driver). These are:
A driver for the Exeda 15mm touchpad, written by Mike Rapoport
<email@example.com> and Igor Grinberg <firstname.lastname@example.org>
A driver for the HTC Dream ClearPad, written by Arve HjÃ¸nnevÃ¥g
We have not extended these drivers for a couple of reasons. First, the
two drivers are specific to particular Synaptics products, and it is our
desire to produce a general solution that takes advantage of the 'self
describing' features of products that use the RMI protocol.
Second, and more importantly, is that the existing drivers are written
for an older version of the RMI protocol (aka RMI3) that is being
retired in favor of a more recent and substantially changed version (aka
RMI4). Most currently shipping Synaptics ClearPads speak the RMI4
protocol, and it will be Synaptics protocol of choice going forward. In
almost all respects, RMI4 is not backward compatible with RMI3, making
it impractical to support both versions in the same driver.
Comments and other feedback on this driver are welcomed.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to email@example.com
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/