From 5c002ba0b984086f5cbbc541854c17a44e67ab40 Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Tue, 27 Jan 2026 13:52:06 -0800 Subject: [PATCH] Fixed the NVIDIA SHIELD Controller showing up twice on macOS Fixes https://github.com/libsdl-org/SDL/issues/14911 (cherry picked from commit 1fe6bf49827d18f00c774ced2f0c2346f54a277c) --- src/joystick/apple/SDL_mfijoystick.m | 11 ++++++++++- src/joystick/apple/SDL_mfijoystick_c.h | 1 + 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/src/joystick/apple/SDL_mfijoystick.m b/src/joystick/apple/SDL_mfijoystick.m index aaae7a8938..45bf0f25fb 100644 --- a/src/joystick/apple/SDL_mfijoystick.m +++ b/src/joystick/apple/SDL_mfijoystick.m @@ -162,6 +162,13 @@ static bool IsControllerSwitchJoyConPair(GCController *controller) } return false; } +static bool IsControllerNVIDIASHIELD(GCController *controller) +{ + if ([controller.vendorName hasPrefix:@"NVIDIA Controller"]) { + return true; + } + return false; +} static bool IsControllerStadia(GCController *controller) { if ([controller.vendorName hasPrefix:@"Stadia"]) { @@ -343,6 +350,7 @@ static bool IOS_AddMFIJoystickDevice(SDL_JoystickDeviceItem *device, GCControlle device->is_ps5 = IsControllerPS5(controller); device->is_switch_pro = IsControllerSwitchPro(controller); device->is_switch_joycon_pair = IsControllerSwitchJoyConPair(controller); + device->is_shield = IsControllerNVIDIASHIELD(controller); device->is_stadia = IsControllerStadia(controller); device->is_backbone_one = IsControllerBackboneOne(controller); device->is_switch_joyconL = IsControllerSwitchJoyConL(controller); @@ -354,6 +362,7 @@ static bool IOS_AddMFIJoystickDevice(SDL_JoystickDeviceItem *device, GCControlle (device->is_ps5 && HIDAPI_IsDeviceTypePresent(SDL_GAMEPAD_TYPE_PS5)) || (device->is_switch_pro && HIDAPI_IsDeviceTypePresent(SDL_GAMEPAD_TYPE_NINTENDO_SWITCH_PRO)) || (device->is_switch_joycon_pair && HIDAPI_IsDevicePresent(USB_VENDOR_NINTENDO, USB_PRODUCT_NINTENDO_SWITCH_JOYCON_PAIR, 0, "")) || + (device->is_shield && HIDAPI_IsDevicePresent(USB_VENDOR_NVIDIA, USB_PRODUCT_NVIDIA_SHIELD_CONTROLLER_V104, 0, "")) || (device->is_stadia && HIDAPI_IsDevicePresent(USB_VENDOR_GOOGLE, USB_PRODUCT_GOOGLE_STADIA_CONTROLLER, 0, "")) || (device->is_switch_joyconL && HIDAPI_IsDevicePresent(USB_VENDOR_NINTENDO, USB_PRODUCT_NINTENDO_SWITCH_JOYCON_LEFT, 0, "")) || (device->is_switch_joyconR && HIDAPI_IsDevicePresent(USB_VENDOR_NINTENDO, USB_PRODUCT_NINTENDO_SWITCH_JOYCON_RIGHT, 0, "")) || @@ -492,7 +501,7 @@ static bool IOS_AddMFIJoystickDevice(SDL_JoystickDeviceItem *device, GCControlle subtype = 4; #ifdef DEBUG_CONTROLLER_PROFILE - NSLog(@"Elements used:\n", controller.vendorName); + NSLog(@"Elements used:\n"); for (id key in device->buttons) { NSLog(@"\tButton: %@ (%s)\n", key, elements[key].analog ? "analog" : "digital"); } diff --git a/src/joystick/apple/SDL_mfijoystick_c.h b/src/joystick/apple/SDL_mfijoystick_c.h index 967ac0ff1c..6e275c00a2 100644 --- a/src/joystick/apple/SDL_mfijoystick_c.h +++ b/src/joystick/apple/SDL_mfijoystick_c.h @@ -53,6 +53,7 @@ typedef struct joystick_hwdata bool is_switch_joycon_pair; bool is_switch_joyconL; bool is_switch_joyconR; + bool is_shield; bool is_stadia; bool is_backbone_one; int is_siri_remote;