diff --git a/Assets/Tests/InputSystem/APIVerificationTests.cs b/Assets/Tests/InputSystem/APIVerificationTests.cs index 63aef7d580..c96b435404 100644 --- a/Assets/Tests/InputSystem/APIVerificationTests.cs +++ b/Assets/Tests/InputSystem/APIVerificationTests.cs @@ -183,6 +183,7 @@ internal static bool IgnoreTypeForDocsByName(string fullName) #if UNITY_EDITOR_OSX fullName == typeof(UnityEngine.InputSystem.XInput.XboxGamepadMacOS).FullName || fullName == typeof(UnityEngine.InputSystem.XInput.XboxOneGampadMacOSWireless).FullName || + fullName == typeof(UnityEngine.InputSystem.XInput.XboxGamepadMacOSWireless).FullName || #endif #if UNITY_EDITOR_WIN fullName == typeof(UnityEngine.InputSystem.XInput.XInputControllerWindows).FullName || diff --git a/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XInputSupport.cs b/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XInputSupport.cs index 6e760a2393..3fd4ba959c 100644 --- a/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XInputSupport.cs +++ b/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XInputSupport.cs @@ -28,7 +28,18 @@ public static void Initialize() InputSystem.RegisterLayout( matches: new InputDeviceMatcher().WithInterface("HID") .WithProduct("Xbox.*Wired Controller")); + + // Matching older controllers that have different View and Share buttons than the newer Xbox Series + // controllers. + // Reported inhttps://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1264 InputSystem.RegisterLayout( + matches: new InputDeviceMatcher().WithInterface("HID") + .WithCapability("vendorId", 0x045E) + .WithCapability("productId", 0x02E0)); + + // This layout is for all the other Xbox One or Series controllers that have the same View and Share buttons. + // Reported in https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-385 + InputSystem.RegisterLayout( matches: new InputDeviceMatcher().WithInterface("HID") .WithProduct("Xbox.*Wireless Controller")); #endif diff --git a/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XboxGamepadMacOS.cs b/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XboxGamepadMacOS.cs index ab43ddbaec..e83aa099e1 100644 --- a/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XboxGamepadMacOS.cs +++ b/Packages/com.unity.inputsystem/InputSystem/Plugins/XInput/XboxGamepadMacOS.cs @@ -111,7 +111,7 @@ internal struct XInputControllerWirelessOSXState : IInputStateTypeInfo public enum Button { Start = 11, - Select = 10, + Select = 16, LeftThumbstickPress = 13, RightThumbstickPress = 14, LeftShoulder = 6, @@ -194,6 +194,98 @@ public XInputControllerWirelessOSXState WithDpad(byte value) leftStickY = 32767 }; } + + [StructLayout(LayoutKind.Explicit)] + internal struct XInputControllerWirelessOSXStateV2 : IInputStateTypeInfo + { + public static FourCC kFormat => new FourCC('H', 'I', 'D'); + + public enum Button + { + Start = 11, + Select = 10, + LeftThumbstickPress = 13, + RightThumbstickPress = 14, + LeftShoulder = 6, + RightShoulder = 7, + A = 0, + B = 1, + X = 3, + Y = 4, + } + [FieldOffset(0)] + private byte padding; + + [InputControl(name = "leftStick", layout = "Stick", format = "VC2S")] + [InputControl(name = "leftStick/x", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5")] + [InputControl(name = "leftStick/left", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0,clampMax=0.5,invert")] + [InputControl(name = "leftStick/right", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0.5,clampMax=1")] + [InputControl(name = "leftStick/y", offset = 2, format = "USHT", parameters = "invert,normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5")] + [InputControl(name = "leftStick/up", offset = 2, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0,clampMax=0.5,invert")] + [InputControl(name = "leftStick/down", offset = 2, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0.5,clampMax=1,invert=false")] + [FieldOffset(1)] public ushort leftStickX; + [FieldOffset(3)] public ushort leftStickY; + + [InputControl(name = "rightStick", layout = "Stick", format = "VC2S")] + [InputControl(name = "rightStick/x", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5")] + [InputControl(name = "rightStick/left", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0,clampMax=0.5,invert")] + [InputControl(name = "rightStick/right", offset = 0, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0.5,clampMax=1")] + [InputControl(name = "rightStick/y", offset = 2, format = "USHT", parameters = "invert,normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5")] + [InputControl(name = "rightStick/up", offset = 2, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0,clampMax=0.5,invert")] + [InputControl(name = "rightStick/down", offset = 2, format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=1,normalizeZero=0.5,clamp=1,clampMin=0.5,clampMax=1,invert=false")] + [FieldOffset(5)] public ushort rightStickX; + [FieldOffset(7)] public ushort rightStickY; + + [InputControl(name = "leftTrigger", format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=0.01560998")] + [FieldOffset(9)] public ushort leftTrigger; + [InputControl(name = "rightTrigger", format = "USHT", parameters = "normalize,normalizeMin=0,normalizeMax=0.01560998")] + [FieldOffset(11)] public ushort rightTrigger; + + [InputControl(name = "dpad", format = "BIT", layout = "Dpad", sizeInBits = 4, defaultState = 8)] + [InputControl(name = "dpad/up", format = "BIT", layout = "DiscreteButton", parameters = "minValue=8,maxValue=2,nullValue=0,wrapAtValue=9", bit = 0, sizeInBits = 4)] + [InputControl(name = "dpad/right", format = "BIT", layout = "DiscreteButton", parameters = "minValue=2,maxValue=4", bit = 0, sizeInBits = 4)] + [InputControl(name = "dpad/down", format = "BIT", layout = "DiscreteButton", parameters = "minValue=4,maxValue=6", bit = 0, sizeInBits = 4)] + [InputControl(name = "dpad/left", format = "BIT", layout = "DiscreteButton", parameters = "minValue=6, maxValue=8", bit = 0, sizeInBits = 4)] + [FieldOffset(13)] + public byte dpad; + + [InputControl(name = "start", bit = (uint)Button.Start, displayName = "Start")] + [InputControl(name = "select", bit = (uint)Button.Select, displayName = "Select")] + [InputControl(name = "leftStickPress", bit = (uint)Button.LeftThumbstickPress)] + [InputControl(name = "rightStickPress", bit = (uint)Button.RightThumbstickPress)] + [InputControl(name = "leftShoulder", bit = (uint)Button.LeftShoulder)] + [InputControl(name = "rightShoulder", bit = (uint)Button.RightShoulder)] + [InputControl(name = "buttonSouth", bit = (uint)Button.A, displayName = "A")] + [InputControl(name = "buttonEast", bit = (uint)Button.B, displayName = "B")] + [InputControl(name = "buttonWest", bit = (uint)Button.X, displayName = "X")] + [InputControl(name = "buttonNorth", bit = (uint)Button.Y, displayName = "Y")] + + [FieldOffset(14)] + public uint buttons; + + public FourCC format => kFormat; + + public XInputControllerWirelessOSXStateV2 WithButton(Button button) + { + Debug.Assert((int)button < 32, $"Expected button < 32, so we fit into the 32 bit wide bitmask"); + buttons |= 1U << (int)button; + return this; + } + + public XInputControllerWirelessOSXStateV2 WithDpad(byte value) + { + dpad = value; + return this; + } + + public static XInputControllerWirelessOSXStateV2 defaultState => new XInputControllerWirelessOSXStateV2 + { + rightStickX = 32767, + rightStickY = 32767, + leftStickX = 32767, + leftStickY = 32767 + }; + } } namespace UnityEngine.InputSystem.XInput { @@ -223,5 +315,22 @@ public class XboxGamepadMacOS : XInputController public class XboxOneGampadMacOSWireless : XInputController { } + + /// + /// A wireless Xbox One or Xbox Series Gamepad connected to a macOS computer. + /// + /// + /// An Xbox One/Series wireless gamepad connected to a mac using Bluetooth. + /// The reason this is different from is that some Xbox Controllers have + /// different View and Share button bit mapping. So we need to use a different layout for those controllers. It seems + /// that some Xbox One and Xbox Series controller share the same mappings so this combines them all. + /// Note: only the latest version of Xbox One wireless gamepads support Bluetooth. Older models only work + /// with a proprietary Xbox wireless protocol, and cannot be used on a Mac. + /// Unlike wired controllers, bluetooth-cabable Xbox One controllers do not need a custom driver to work on macOS. + /// + [InputControlLayout(displayName = "Wireless Xbox Controller", stateType = typeof(XInputControllerWirelessOSXStateV2), hideInUI = true)] + public class XboxGamepadMacOSWireless : XInputController + { + } } #endif // UNITY_STANDALONE_OSX || UNITY_EDITOR_OSX