Re: [PATCH v8] input: Add Synaptics NavPoint (PXA27x SSP/SPI) driver

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

On Wednesday, May 09, 2012 10:09:55 PM Paul Parsons wrote:
> Hello Dmitry,
> 
> --- On Wed, 9/5/12, Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> > The driver looks excellent now, I have just a few nits:
> > 
> > - you do not need separate mutex and counter; it is
> > perfectly fine to
> >   use input device's ones;
> > 
> > - we don't really need min/max for coordinates: the defaults
> > are good
> >   for your device and if more devices come with
> > different limits one
> >   could adjust limits via EVIOCSABS ioctls.
> > 
> > I also took libertu if rearranging the code a bit so it
> > flows more like
> > the rest of the input drivers. Could you please tell me if
> > the patch
> > below breaks your device or if it still works?
> 
> It still works! Thanks.
> 
> Will you apply a combined patch (my patch v8 + your patch),
> or should I submit them as patch v9?

If you have not made any changes to the patch I sent then there is no need
to send v9, I'll fold everything together on my end.

Thanks.

-- 
Dmitry