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

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

 



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@xxxxxxxxx>)
> > > 
> > > 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@xxxxxxxxx>)
> > > 
> > > 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@xxxxxxxxx>)
> > > 
> > > 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@xxxxxxxxx>
> 
> FYI: I do not see a dingle email from Paul, only your replies.

Patches were sent to linux-input@xxxxxxxxxxxxxxx ML. Maybe some spam 
filter ate emails? Btw I added you to receivers of my last sent email.

-- 
Pali Rohár
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.


[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