Re: [PATCH v4 1/2] input: touchscreen: introduce tsc2005 driver

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

 



* Aaro Koskinen <aaro.koskinen@xxxxxxxxx> [100419 07:47]:
> From: Lauri Leukkunen <lauri.leukkunen@xxxxxxxxx>
> 
> Introduce a driver for the Texas Instruments TSC2005 touchscreen
> controller (http://focus.ti.com/docs/prod/folders/print/tsc2005.html).
> 
> The patch is based on a driver by Lauri Leukkunen, with modifications
> by David Brownell, Phil Carmody, Imre Deak, Hiroshi DOYU, Ari Kauppi,
> Tony Lindgren, Jarkko Nikula, Eero Nurkkala and Roman Tereshonkov.
> 
> Signed-off-by: Lauri Leukkunen <lauri.leukkunen@xxxxxxxxx>
> [aaro.koskinen@xxxxxxxxx: patch description, rebasing & cleanup]
> Signed-off-by: Aaro Koskinen <aaro.koskinen@xxxxxxxxx>
> Cc: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx>
> Cc: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx>
> Cc: Imre Deak <imre.deak@xxxxxxxxx>
> Cc: Hiroshi DOYU <Hiroshi.DOYU@xxxxxxxxx>
> Cc: Ari Kauppi <Ext-Ari.Kauppi@xxxxxxxxx>
> Cc: Tony Lindgren <tony@xxxxxxxxxxx>
> Cc: Jarkko Nikula <jhnikula@xxxxxxxxx>
> Cc: Eero Nurkkala <ext-eero.nurkkala@xxxxxxxxx>
> Cc: Roman Tereshonkov <roman.tereshonkov@xxxxxxxxx>
> ---
> 
> v4: Refreshed the patches against the current mainline
> (13bd8e4673d527a9e48f41956b11d391e7c2cfe0). No other changes as no
> comments were received.
> 
> The v3 has some major modifications. I have tried to clean up and simplify
> the driver. All action takes place in thread context, SPI transfers are
> synchronous, the spinlock is removed, and there are no long sections
> with interrupts disabled. The driver should be now SMP safe.
> 
> Also the following review comments are addressed:
> 	- Use del_timer_sync() instead of del_timer().
> 	- Use mod_timer() always instead of add_timer().
> 	- Use attribute_group.
> 	- Delete filtering and averaging which belong to upper layers.
> 	- Delete pen_down sysfs file.
> 
> The previous version and comments can be seen here:
> http://marc.info/?t=125734115600004&r=1&w=2

Any news on this? Would be nice to get this merged in as it sounds
like the N900 display will finally be usable in the mainline kernel
too.

Regards,

Tony
--
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