diff mbox

Input: ALPS - fix TrackStick Y axis handling for SS5 hardware

Message ID 1483447529-13565-1-git-send-email-linux-kernel@PaulSD.com (mailing list archive)
State Accepted
Headers show

Commit Message

Paul Donohue Jan. 3, 2017, 12:45 p.m. UTC
A minus character was lost in commit 23fce365, causing the Y axis to be
inverted for SS5 TrackStick events.  (Pushing the TrackStick up caused
the pointer to move down, and vice versa.)  Restore the lost minus.

Signed-off-by: Paul Donohue <linux-kernel@PaulSD.com>
---
 drivers/input/mouse/alps.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Dmitry Torokhov Jan. 3, 2017, 6:41 p.m. UTC | #1
On Tue, Jan 03, 2017 at 07:45:29AM -0500, Paul Donohue wrote:
> A minus character was lost in commit 23fce365, causing the Y axis to be
> inverted for SS5 TrackStick events.  (Pushing the TrackStick up caused
> the pointer to move down, and vice versa.)  Restore the lost minus.
> 
> Signed-off-by: Paul Donohue <linux-kernel@PaulSD.com>

Applied, thank you.

> ---
>  drivers/input/mouse/alps.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/input/mouse/alps.h b/drivers/input/mouse/alps.h
> index cde6f4b..6d279aa 100644
> --- a/drivers/input/mouse/alps.h
> +++ b/drivers/input/mouse/alps.h
> @@ -114,7 +114,7 @@ enum SS4_PACKET_ID {
>  				 (_b[1] & 0x7F)		\
>  				)
>  
> -#define SS4_TS_Y_V2(_b)		(s8)(				\
> +#define SS4_TS_Y_V2(_b)		-(s8)(				\
>  				 ((_b[3] & 0x01) << 7) |	\
>  				 (_b[2] & 0x7F)		\
>  				)
> -- 
> 2.9.3
>
diff mbox

Patch

diff --git a/drivers/input/mouse/alps.h b/drivers/input/mouse/alps.h
index cde6f4b..6d279aa 100644
--- a/drivers/input/mouse/alps.h
+++ b/drivers/input/mouse/alps.h
@@ -114,7 +114,7 @@  enum SS4_PACKET_ID {
 				 (_b[1] & 0x7F)		\
 				)
 
-#define SS4_TS_Y_V2(_b)		(s8)(				\
+#define SS4_TS_Y_V2(_b)		-(s8)(				\
 				 ((_b[3] & 0x01) << 7) |	\
 				 (_b[2] & 0x7F)		\
 				)