Message-ID: <B27AC323FA92C946BA36D2913DDC8E3701FD6F0B1A51@VMBX126.ihostexchange.net>
Date:	Fri, 4 Mar 2011 05:46:39 -0500
From:	Dudley Du <dudl@...ress.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
CC:	"rubini@...vis.unipv.it" <rubini@...l.unipv.it>,
	"olofj@...gle.com" <olofj@...gle.com>,
	"micahc@...gle.com" <micahc@...gle.com>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"David Solda (David.Solda@...ress.com)" <David.Solda@...ress.com>
Subject: RE: [PATCH] mouse: Add new i2c-based input mouse driver into input
 subsystem for Cypress i2c trackpad devices.

Hi Dmitry,

Thanks for your questions.

This new driver belongs to Input device Driver, it locates in the layer under Input Core,
It communicates directly to Cypress I2C trackpad hardware device.
Cypress I2C trackpad hardware device is a multi-touch device, it uses i2c interface to communicate with host.
This device can detect one finger movement, can detect multi-finger movements,
Also it can recognize and report multi-finger movements as predefined gestures.
So using these trackpad devices, all gesture detect are done in trackpad device, not in driver or upper layers.

In brief, the basic functions of this i2c-based input mouse driver are list as bellow:
1. connect this driver into i2c subsystem and input subsystem;
   Because Cypress trackpad devices use i2c interfaces to connect and communicate with host.
2. communicate with Cypress trackpad devices to read raw data of fingers movement and detected gestures;
3. parse received raw data from trackpad devices into system recognizes data;
4. report received finger movement data into input subsystem as cursor movement.
5. convert received finger gestures into keyboard press/release, mouse press/release and wheel operations,
   then report the gestures by reporting combined keyboard press/release, mouse press/release and wheel operations into Input subsystem;
   that's why there will be virtual keyboard, virtual wheel handling, etc in this driver.
6. Do cursor performance filter to improve cursor movement performance;
   e.g.: make cursor movement much more smooth; improve cursor moving linearity and so on.
7. Do gesture performance filter to improve gesture performance;
   e.g.: when do two fingers vertical scroll operation on a web page, make the web page be scrolled much more smooth,
         and apply acceleration on the scroll to make low finger movement corresponding to low scrolling,
         fast finger movement corresponding to fast scrolling and can be enlarged to fit user's sense.
8. report data to input subsystem.


Bellow tries to answer the questions in the mail:
1. Why there are amount of code to reinterpret data from device and convert to gestures in this driver?
  1) Trackpad devices use I2C interface to report data through I2C registers. These I2C registers are defined and organized by Cypress,
     so input subsystem and other trackpad device drivers can't read and recognize the meanings of this registers directly.
  2) also there are 2 released registers map spec that used in trackpad devices.
     So there will be two cluster code named gen1 and gen2 to reinterpret data from device.
  3) also fingers gestures are detected in trackpad device, and the gestures are reported as gesture IDs,
     These gesture IDs are all defined and only meaningfully to Cypress trackpad devices.
     That means these gesture IDs can't be recognized by input subsystem and other drivers,
     So this driver is required to reinterpret the gesture data from device, and cannot do this in user space.

2. Why give a choice between relative mode and absolute mode to report data?
  It's sure that absolute mode can supply more interface to report multi-class data and also provide more data for gesture processing.
  Currently as we know, with absolute mode, Input subsystem requires trackpad device to report absolute X and Y position data directly,
  then Input system transfers these data to mousedev driver, and mousedev driver calculates based on this absolute X and Y position data
  and convert them into X delta and Y delta data. In this processing, mousedev doesn't apply any cursor acceleration function to cursor movement
  when convert into X delta and Y delta data, so cursor movement shown in screen has no acceleration.
  That means when fast move finger in trackpad, the cursor movement in screen can't be enlarged just as we use USB mouse device.
  And in absolute mode, we are unable to add cursor acceleration function to absolution X and Y data.
  So in order to add cursor acceleration function to cursor movement, we added relative mode also be supported.
  Because in relative mode, trackpad devices report relative delta data, and based on this relative delta data,
  driver can apply cursor performance filter to this data, so cursor acceleration function can also be applied in relative mode.

3. Why add polling mode supported in this driver?
  Besides I2C bus hardware pins of VDD, Gound, SDA and SDL, Trackpad device also requires an extended GPIO pin to assert interrupts to host.
  But for some host devices, there is no extended GPIO pin reserved for this slave interrupt pin (SINT),
  So for these devices, interrupts from trackpad devices can't be supported, and only polling mode can be used.
  That's why we add polling mode in this driver, it provides compatible to those device that without extended GPIO pin supported.

Thanks.

Best Wishes,
Dudley Du
86-21-61648950


-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@...il.com] 
Sent: Friday, March 04, 2011 5:05 PM
To: Dudley Du
Cc: rubini@...vis.unipv.it; olofj@...gle.com; micahc@...gle.com; linux-input@...r.kernel.org; linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mouse: Add new i2c-based input mouse driver into input subsystem for Cypress i2c trackpad devices.

Hi Dudley,

On Fri, Mar 04, 2011 at 03:48:21AM -0500, Dudley Du wrote:
> Hi Linux-Input Maintainers,
> 
> This patch adds new i2c-based input mouse driver into input subsystem for
> Cypress i2c trackpad devices. Through this new added driver,
> Cypress i2c trackpad devices will be supported in Linux based system
> which are developed based on Cypress self-designed PSOC chipset.
> The function of this driver reads trackpad data using i2c interfaces
> and report cursor moving data and gesture combined keys to input subsystem.
> 
> This driver adds one driver main file cypress.c in drivers/input/mouse directory,
> and one header file cyapa.h in include/linux directory,
> and modified Kconfig and Makefile files in drivers/input/mouse directory to include this driver.
> 
> Attached is the patch file for this driver and bellow is the content of this driver.
> 

Could you please try describing capabilities of your device. From the
glancing at it it looks like it supports some form of multi-touch...

From the very rough code scanning there appears to be huge amount of
code that reinterprets data coming from the device and converts it into
gestures (virtual keyboard, virtual wheel hanlding, etc, etc). Such
processing does not belong to the kernel driver but rather to userspace.
Please take a look and other touchpad drivers in the kernel (synaptics,
alps, elantech, bcm, and so for) and how they are work in tandem with
synaptics X driver which provides advanced gesture handling.

I also question why one would give choise between using relative and
absolute reporting when absolute mode provides more data for gesture
processing. Also polling mode is probably unsuitable for production
driver and so I'd recommend dropping it for the mainline submission.

Thanks.

-- 
Dmitry

Powered by blists - more mailing lists