Re: [RFC PATCH 00/11] input: Synaptics RMI4 Touchscreen Driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/22/2012 05:50 AM, Linus Walleij wrote:
On Sat, Aug 18, 2012 at 12:17 AM, Christopher Heiny
<cheiny@xxxxxxxxxxxxx>  wrote:

>This patch implements a driver supporting Synaptics ClearPad and other
>touchscreen sensors that use the RMI4 protocol, as defined here:
Nice!

>This patch is against the v2.6.38 tag of Linus' kernel tree, commit
>521cb40b0c44418a4fd36dc633f575813d59a43d.  This will be our last patch against
>such an old kernel version, future patches will be against 3.x kernels.
Please use the head of the subsystem tree to do this work.
In this case, use Dmitry's git.

Currently I just cannot test this because we have no such old codebase
around.

But I will attempt to review the patch set!

Thanks for all the feedback so far - it has been quite valuable. I'm not going to try to respond to every comment, especially since most of it will be of a "Yeah - good idea" nature.

A couple of general things:


The "phys" debugfs interface is intended to tell you about the physical interface a particular RMI4 sensor is using (either SPI or I2C or whatever), including the name of the interface, the number of reads/writes done, number of bytes that have been read/written, number of errors, and so on.

We chose to to call name it phys so that the debug tools would be able to find this interface no matter what the physical connection was, even when new physical layer support (for example, SMbus) is introduced.



EARLY_SUSPEND/LATE_RESUME and other power management stuff. We're caught in a bind here. Most of our customers are using some flavor of Android. They have the expectation that our driver will (a) support the Android power management model, and (b) be contributed into the mainline kernel without change. Yes, I know these are contradictory requirements, given that Android specific features are not in the mainline.

With the upcoming rebase of the code to more modern kernels, we'll be able to eliminate a bunch of those dependencies. But the only way to eliminate them entirely would be to maintain mainline and Android versions of the driver, which would drain resources from developing core features and fixing bugs. So for now we've got a single code base. When we finally submit a patch and the only response is "everything is fine but that Android stuff", we'll probably change that policy within 48 hours (to include time needed for celebration and subsequent hangover recovery :-) ).

					Cheers,
						Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux