Re: [PATCH v5 0/3] Input: ALPS - Bug fix and cleanup for SS5 hardware

["Pali Rohár" <pali.rohar@gmail.com>]

[-- Attachment #1: Type: Text/Plain, Size: 3136 bytes --]

On Sunday 13 November 2016 07:43:35 Dmitry Torokhov wrote:
> On Sat, Nov 12, 2016 at 4:29 AM, Pali Rohár <pali.rohar@gmail.com>
> wrote:
> > On Saturday 12 November 2016 02:06:40 Dmitry Torokhov wrote:
> >> On Sat, Nov 12, 2016 at 02:03:42AM +0100, Pali Rohár wrote:
> >> > On Saturday 12 November 2016 01:56:21 Dmitry Torokhov wrote:
> >> > > On Fri, Nov 11, 2016 at 02:59:09PM +0100, Pali Rohár wrote:
> >> > > > On Thursday 10 November 2016 17:44:43 Paul Donohue wrote:
> >> > > > > This series includes a bug fix and assorted cleanup for
> >> > > > > the Alps SS5 (SS4 v2) code.
> >> > > > > 
> >> > > > > Changes in v5:
> >> > > > > * Moved TrackStick decoding logic to
> >> > > > > alps_process_packet_ss4_v2() to
> >> > > > > 
> >> > > > >   make the code more consistent with other protocol
> >> > > > >   versions (Suggested by Pali Rohár
> >> > > > >   <pali.rohar@gmail.com>)
> >> > > > > 
> >> > > > > Changes in v4:
> >> > > > > * Fixed patch formatting issues
> >> > > > > * Correct casting issues in macros added in v3
> >> > > > > 
> >> > > > > Changes in v3:
> >> > > > > * Added additional code cleanup to make the code easier to
> >> > > > > understand
> >> > > > > 
> >> > > > >   (Suggested by Pali Rohár <pali.rohar@gmail.com>)
> >> > > > > 
> >> > > > > Changes in v2:
> >> > > > > * For consistency with other Alps functions, check packet
> >> > > > > bytes for
> >> > > > > 
> >> > > > >   packet type rather than adding a flag to alps_fields for
> >> > > > >   packet
> >> > > > > 
> >> > > > > type. (Suggested by Pali Rohár <pali.rohar@gmail.com>)
> >> > > > > 
> >> > > > > Paul Donohue (3):
> >> > > > >   Input: ALPS - Fix TrackStick support for SS5 hardware
> >> > > > >   Input: ALPS - Clean up TrackStick handling for SS5
> >> > > > >   hardware Input: ALPS - Clean up code for SS5 hardware
> >> > > > >  
> >> > > > >  drivers/input/mouse/alps.c | 59
> >> > > > > 
> >> > > > > +++++++++++++++++++++++++---------------------
> >> > > > > drivers/input/mouse/alps.h | 22 ++++++++++++++++-
> >> > > > > 
> >> > > > >  2 files changed, 53 insertions(+), 28 deletions(-)
> >> > > > 
> >> > > > Whole patch series looks fine.
> >> > > > You can add my Reviewed-by: Pali Rohár
> >> > > > <pali.rohar@gmail.com>
> >> > > 
> >> > > FYI: I do not see a dingle email from Paul, only your replies.
> >> > 
> >> > Patches were sent to linux-input@vger.kernel.org ML. Maybe some
> >> > spam filter ate emails? Btw I added you to receivers of my last
> >> > sent email.
> >> 
> >> Yes, I looked there as well and did not see anything but you
> >> replies.
> >> 
> >> Thanks.
> > 
> > Do you need to resend patches? Or have you already found them?
> 
> Hmm, was the latest v5? I see
> https://patchwork.kernel.org/patch/9421473/

Yes, v5 is the last.

> It looks like Paul is running his own mailserver on what looks like
> Verizon IP pool, which in this day and age likely makes gmail drop
> his mails.

Hm... looks weird... because at least I received his emails on my gmail 
account...

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]