Dear Jonathan Cameron, > On 12/14/2012 01:46 AM, Marek Vasut wrote: > > This patch implements support for sampling of a touchscreen into > > the MXS LRADC driver. The LRADC block allows configuring some of > > it's channels into special mode where they either output the drive > > voltage or sample it, allowing it to operate a 4-wire or 5-wire > > resistive touchscreen. > > > > In case the touchscreen mode is enabled, the LRADC slot #7 is > > reserved for touchscreen only, therefore it is not possible to > > sample 8 LRADC channels at time, but only 7 channels. > > > > The touchscreen controller is configured such that the PENDOWN event > > disables touchscreen interrupts and triggers execution of worker > > thread, which then polls the touchscreen controller for X, Y and > > Pressure values. This reduces the overhead of interrupt-driven > > operation. Upon the PENUP event, the worker thread re-enables the > > PENDOWN detection interrupt and exits. > > I still have a few reservations about this patch. Firstly > I would love to see a more generic approach to this as I outlined > before. Still we can't have everything we want sometimes :) > Also, this is a lot of input related code in a driver that isn't > in the input subsystem. Really up to Dmitry on whether he is > happy with this, or whether he insists on an mfd. > > So all in all, with reservations I'll add this to the IIO tree > IF Dmitry is happy with it and there are no other issues raised > in the meantime. Understood, I'll wait for Dmitry. > Jonathan > > p.s. If anyone has time, I'd also like to get this out of staging > in the coming cycle. [...] -- 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