We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] Patch to src/joystick/hidapi/SDL_hidapi_switch.c which applies to SDL2 2.0.12

SDL_hidapi_switch.c.patch (text/plain), 3.27 KB, created by bluenaxela+sdl on 2020-03-22 07:04:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: bluenaxela+sdl

Created: 2020-03-22 07:04:57 UTC

Size: 3.27 KB

patch

obsolete

>*** ../SDL2-2.0.12/src/joystick/hidapi/SDL_hidapi_switch.c	2020-03-10 18:36:18.000000000 -0700
>--- src/joystick/hidapi/SDL_hidapi_switch.c	2020-03-21 23:50:26.072267300 -0700
>***************
>*** 524,530 ****
>      return WriteSubcommand(ctx, k_eSwitchSubcommandIDs_SetPlayerLights, &led_data, sizeof(led_data), NULL);
>  }
>  
>! static SDL_bool LoadStickCalibration(SDL_DriverSwitch_Context *ctx)
>  {
>      Uint8 *pStickCal;
>      size_t stick, axis;
>--- 524,530 ----
>      return WriteSubcommand(ctx, k_eSwitchSubcommandIDs_SetPlayerLights, &led_data, sizeof(led_data), NULL);
>  }
>  
>! static SDL_bool LoadStickCalibration(SDL_DriverSwitch_Context *ctx, Uint8 input_mode)
>  {
>      Uint8 *pStickCal;
>      size_t stick, axis;
>***************
>*** 577,583 ****
>          }
>      }
>  
>!     if (ctx->m_bUsingBluetooth) {
>          for (stick = 0; stick < 2; ++stick) {
>              for(axis = 0; axis < 2; ++axis) {
>                  ctx->m_StickExtents[stick].axis[axis].sMin = (Sint16)(SDL_MIN_SINT16 * 0.5f);
>--- 577,583 ----
>          }
>      }
>  
>!     if (input_mode == k_eSwitchInputReportIDs_SimpleControllerState) {
>          for (stick = 0; stick < 2; ++stick) {
>              for(axis = 0; axis < 2; ++axis) {
>                  ctx->m_StickExtents[stick].axis[axis].sMin = (Sint16)(SDL_MIN_SINT16 * 0.5f);
>***************
>*** 720,726 ****
>              ctx->m_bUsingBluetooth = SDL_TRUE;
>          }
>  
>!         if (!LoadStickCalibration(ctx)) {
>              SDL_SetError("Couldn't load stick calibration");
>              goto error;
>          }
>--- 720,743 ----
>              ctx->m_bUsingBluetooth = SDL_TRUE;
>          }
>  
>!         /* Determine the desired input mode (needed before loading stick calibration) */
>!         if (ctx->m_bUsingBluetooth) {
>!             input_mode = k_eSwitchInputReportIDs_SimpleControllerState;
>!         }
>!         else {
>!             input_mode = k_eSwitchInputReportIDs_FullControllerState;
>!         }
>! 
>!         /* The official Nintendo Switch Pro Controller supports FullControllerState over bluetooth
>!          * just fine. We really should use that, or else the epowerlevel code in
>!          * HandleFullControllerState is completely pointless. We need full state if we want battery
>!          * level and we only care about battery level over bluetooth anyway.
>!          */
>!         if (device->vendor_id == 0x057e && device->product_id == 0x2009) {
>!             input_mode = k_eSwitchInputReportIDs_FullControllerState;
>!         }
>! 
>!         if (!LoadStickCalibration(ctx, input_mode)) {
>              SDL_SetError("Couldn't load stick calibration");
>              goto error;
>          }
>***************
>*** 730,741 ****
>              goto error;
>          }
>  
>!         /* Set the desired input mode */
>!         if (ctx->m_bUsingBluetooth) {
>!             input_mode = k_eSwitchInputReportIDs_SimpleControllerState;
>!         } else {
>!             input_mode = k_eSwitchInputReportIDs_FullControllerState;
>!         }
>          if (!SetInputMode(ctx, input_mode)) {
>              SDL_SetError("Couldn't set input mode");
>              goto error;
>--- 747,753 ----
>              goto error;
>          }
>  
>!         /* Set desired input mode */
>          if (!SetInputMode(ctx, input_mode)) {
>              SDL_SetError("Couldn't set input mode");
>              goto error;

Actions: View | Diff

Attachments on bug 5051: 4275

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.