Hi Dmitry,
On Mon, Jul 23, 2012 at 09:54:00AM -0700, Dmitry Torokhov wrote:
> On Sun, Jul 22, 2012 at 05:02:19PM +0200, simon.budig@xxxxxxxxxxxxxxxxx wrote:
> > From: Simon Budig <simon.budig@xxxxxxxxxxxxxxxxx>
> >
> > This is a driver for the EDT "Polytouch" family of touch controllers
> > based on the FocalTech FT5x06 line of chips.
> >
> > Signed-off-by: Simon Budig <simon.budig@xxxxxxxxxxxxxxxxx>
> > ---
> >
> > This is a new iteration of the driver for the edt ft5x06 based
> > polytouch series of touchscreens.
> >
> > This incorporates the changes from Dmitry (Thanks!) as well as the
> > remaining suggestions from Henrik: The default parameter stuff has
> > been factored out to a macro and the accessors for the touch data
> > has been made a bit less confusing... :)
> >
> > I have tested it on real hardware (with a slighty older kernel) and
> > everything seems to work properly.
> >
> > There is a lot of stuff from Dmitry in here, so his Signed-off-by line
> > probably is required as well.
> >
> > Thanks for helping to improving this driver.
> >
>
> Henrik,
>
> Are you OK with the driver in the current form? I think it is in good
> shape and should be applied; any additional improvements could go on top
> a separate patches.
It looks all good to me,
Reviewed-by: Henrik Rydberg <rydberg@xxxxxxxxxxx>
Thanks, Simon, for all your work.
Henrik
--
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
