From 6f1aa78e04545ba895dc1aa111db91e55ddf7fe5 Mon Sep 17 00:00:00 2001
From: walle <[EMAIL REDACTED]>
Date: Mon, 9 Feb 2026 21:13:18 +0800
Subject: [PATCH] fixed trigger axes formula: data * 257 - 32768 (same as PS4)
---
src/joystick/hidapi/SDL_hidapi_gamesir.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/joystick/hidapi/SDL_hidapi_gamesir.c b/src/joystick/hidapi/SDL_hidapi_gamesir.c
index 5f4d54049c4b2..cfb92bea9b58f 100644
--- a/src/joystick/hidapi/SDL_hidapi_gamesir.c
+++ b/src/joystick/hidapi/SDL_hidapi_gamesir.c
@@ -696,17 +696,17 @@ static void HIDAPI_DriverGameSir_HandleStatePacket(SDL_Joystick *joystick, SDL_D
}
// Handle trigger axes
- // Protocol: L2 (payload byte 12) - analog left trigger 0-255, 0 = released, 255 = pressed
- // R2 (payload byte 13) - analog right trigger 0-255, 0 = released, 255 = pressed
- // SDL range: 0-32767 (0 = released, 32767 = fully pressed)
- // Linear mapping: 0-255 -> 0-32767
+ // Protocol: L2 (payload byte 12) - analog left trigger 0-255, 0 = released, 255 = fully pressed
+ // R2 (payload byte 13) - analog right trigger 0-255, 0 = released, 255 = fully pressed
+ // SDL range: -32768 to 32767 (-32768 = released, 32767 = fully pressed)
+ // Linear mapping: 0-255 -> -32768..32767, formula: data * 257 - 32768 (same as PS4)
if (last[12] != data[12]) {
- axis = (Sint16)(((int)data[12] * 255) - 32767);
+ axis = ((int)data[12] * 257) - 32768;
SDL_SendJoystickAxis(timestamp, joystick, SDL_GAMEPAD_AXIS_LEFT_TRIGGER, axis);
}
if (last[13] != data[13]) {
- axis = (Sint16)(((int)data[13] * 255) - 32767);
+ axis = ((int)data[13] * 257) - 32768;
SDL_SendJoystickAxis(timestamp, joystick, SDL_GAMEPAD_AXIS_RIGHT_TRIGGER, axis);
}
}