WIP: SDL3 #533

Closed
madwind wants to merge 64 commits from SDL3 into master
9 changed files with 100 additions and 128 deletions
Showing only changes of commit f4e7482312 - Show all commits

View File

@ -49,14 +49,6 @@ namespace Ryujinx.Audio.Backends.SDL3
private static bool IsSupportedInternal() private static bool IsSupportedInternal()
{ {
var devices = SDL_GetAudioPlaybackDevices(out int count);
var joystickIDs = new int[count];
Marshal.Copy(devices, joystickIDs, 0, count);
for (int i = 0; i < count; ++i)
{
Console.WriteLine(joystickIDs[i]);
}
var device = OpenStream(SampleFormat.PcmInt16, Constants.TargetSampleRate, Constants.ChannelCountMax, var device = OpenStream(SampleFormat.PcmInt16, Constants.TargetSampleRate, Constants.ChannelCountMax,
Constants.TargetSampleCount, null); Constants.TargetSampleCount, null);
@ -147,11 +139,9 @@ namespace Ryujinx.Audio.Backends.SDL3
return 0; return 0;
} }
Console.WriteLine(SDL_GetAudioDeviceName(device));
bool isValid = false; bool isValid = false;
if (SDL_GetAudioDeviceFormat(device, out SDL_AudioSpec got, out int i)) if (SDL_GetAudioDeviceFormat(device, out SDL_AudioSpec got, out int i))
{ {
Console.WriteLine(got.freq);
isValid = got.format == desired.format && got.freq == desired.freq && isValid = got.format == desired.format && got.freq == desired.freq &&
got.channels == desired.channels; got.channels == desired.channels;
} }

View File

@ -151,7 +151,6 @@ namespace Ryujinx.Audio.Backends.SDL3
public override void QueueBuffer(AudioBuffer buffer) public override void QueueBuffer(AudioBuffer buffer)
{ {
Console.WriteLine(buffer);
EnsureAudioStreamSetup(buffer); EnsureAudioStreamSetup(buffer);
if (_outputStream != 0) if (_outputStream != 0)

View File

@ -0,0 +1,4 @@
namespace Ryujinx.Input.SDL3
{
public record GamepadInfo(string driverId, nint gamepadHandle);
}

View File

@ -60,13 +60,13 @@ namespace Ryujinx.Input.SDL3
private float _triggerThreshold; private float _triggerThreshold;
public SDL3Gamepad(nint gamepadHandle, string driverId) public SDL3Gamepad(GamepadInfo gamepadInfo)
{ {
_gamepadHandle = gamepadHandle; _gamepadHandle = gamepadInfo.gamepadHandle;
_buttonsUserMapping = new List<ButtonMappingEntry>(20); _buttonsUserMapping = new List<ButtonMappingEntry>(20);
Name = SDL_GetGamepadName(_gamepadHandle); Name = SDL_GetGamepadName(_gamepadHandle);
Id = driverId; Id = gamepadInfo.driverId;
Features = GetFeaturesFlag(); Features = GetFeaturesFlag();
_triggerThreshold = 0.0f; _triggerThreshold = 0.0f;
@ -181,8 +181,7 @@ namespace Ryujinx.Input.SDL3
private static Vector3 RadToDegree(Vector3 rad) => rad * (180 / MathF.PI); private static Vector3 RadToDegree(Vector3 rad) => rad * (180 / MathF.PI);
//TODO: miss constant SDL_STANDARD_GRAVITY 9.80665f private static Vector3 GsToMs2(Vector3 gs) => gs / SDL_STANDARD_GRAVITY;
private static Vector3 GsToMs2(Vector3 gs) => gs / 9.80665f;
public void SetConfiguration(InputConfig configuration) public void SetConfiguration(InputConfig configuration)
{ {

View File

@ -4,7 +4,6 @@ using Ryujinx.SDL3.Common;
using System; using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.Linq; using System.Linq;
using System.Runtime.InteropServices;
using System.Text; using System.Text;
using System.Threading; using System.Threading;
using static SDL3.SDL; using static SDL3.SDL;
@ -13,7 +12,7 @@ namespace Ryujinx.Input.SDl3
{ {
public class SDL3GamepadDriver : IGamepadDriver public class SDL3GamepadDriver : IGamepadDriver
{ {
private readonly Dictionary<uint, string> _gamepadsInstanceIdsMapping; private readonly Dictionary<uint, GamepadInfo> _gamepadsInstanceIdsMapping;
private readonly List<string> _gamepadsIds; private readonly List<string> _gamepadsIds;
private readonly Lock _lock = new(); private readonly Lock _lock = new();
@ -35,7 +34,7 @@ namespace Ryujinx.Input.SDl3
public SDL3GamepadDriver() public SDL3GamepadDriver()
{ {
_gamepadsInstanceIdsMapping = new Dictionary<uint, string>(); _gamepadsInstanceIdsMapping = new Dictionary<uint, GamepadInfo>();
_gamepadsIds = new List<string>(); _gamepadsIds = new List<string>();
SDL3Driver.Instance.Initialize(); SDL3Driver.Instance.Initialize();
@ -43,20 +42,19 @@ namespace Ryujinx.Input.SDl3
SDL3Driver.Instance.OnJoystickDisconnected += HandleJoyStickDisconnected; SDL3Driver.Instance.OnJoystickDisconnected += HandleJoyStickDisconnected;
SDL3Driver.Instance.OnJoyBatteryUpdated += HandleJoyBatteryUpdated; SDL3Driver.Instance.OnJoyBatteryUpdated += HandleJoyBatteryUpdated;
IntPtr joystickArray = SDL_GetJoysticks(out int count); // IntPtr joystickArray = SDL_GetJoysticks(out int count);
//
var joystickIDs = new int[count]; // var joystickIDs = new int[count];
Marshal.Copy(joystickArray, joystickIDs, 0, count); // Marshal.Copy(joystickArray, joystickIDs, 0, count);
//
for (int i = 0; i < count; i++) // for (int i = 0; i < count; i++)
{ // {
HandleJoyStickConnected((uint)joystickIDs[i]); // HandleJoyStickConnected((uint)joystickIDs[i]);
} // }
} }
private string GenerateGamepadId(uint joystickIndex) private string GenerateGamepadId(uint joystickIndex)
{ {
int bufferSize = 33; int bufferSize = 33;
Span<byte> pszGUID = stackalloc byte[bufferSize]; Span<byte> pszGUID = stackalloc byte[bufferSize];
SDL_GUIDToString(SDL_GetJoystickGUIDForID(joystickIndex), pszGUID, bufferSize); SDL_GUIDToString(SDL_GetJoystickGUIDForID(joystickIndex), pszGUID, bufferSize);
@ -82,23 +80,24 @@ namespace Ryujinx.Input.SDl3
return id; return id;
} }
private uint GetJoystickIndexByGamepadId(string id) private GamepadInfo GetJoystickIndexByGamepadId(string id)
{ {
lock (_lock) lock (_lock)
{ {
return _gamepadsInstanceIdsMapping.FirstOrDefault(x=>x.Value == id).Key; return _gamepadsInstanceIdsMapping.FirstOrDefault(x => x.Value.driverId == id).Value;
} }
} }
private void HandleJoyStickDisconnected(uint joystickInstanceId) private void HandleJoyStickDisconnected(uint joystickInstanceId)
{ {
bool joyConPairDisconnected = false; bool joyConPairDisconnected = false;
if (!_gamepadsInstanceIdsMapping.Remove(joystickInstanceId, out string id)) if (!_gamepadsInstanceIdsMapping.Remove(joystickInstanceId, out GamepadInfo gamepadInfo))
return; return;
lock (_lock) lock (_lock)
{ {
_gamepadsIds.Remove(id); _gamepadsIds.Remove(gamepadInfo.driverId);
SDL_CloseGamepad(gamepadInfo.gamepadHandle);
if (!SDL3JoyConPair.IsCombinable(_gamepadsInstanceIdsMapping)) if (!SDL3JoyConPair.IsCombinable(_gamepadsInstanceIdsMapping))
{ {
_gamepadsIds.Remove(SDL3JoyConPair.Id); _gamepadsIds.Remove(SDL3JoyConPair.Id);
@ -106,62 +105,58 @@ namespace Ryujinx.Input.SDl3
} }
} }
OnGamepadDisconnected?.Invoke(id); OnGamepadDisconnected?.Invoke(gamepadInfo.driverId);
if (joyConPairDisconnected) if (joyConPairDisconnected)
{ {
OnGamepadDisconnected?.Invoke(SDL3JoyConPair.Id); OnGamepadDisconnected?.Invoke(SDL3JoyConPair.Id);
} }
} }
private void HandleJoyStickConnected(uint joystickInstanceId) private void HandleJoyStickConnected(uint gamepadInstanceId)
{ {
bool joyConPairConnected = false; bool joyConPairConnected = false;
if (_gamepadsInstanceIdsMapping.ContainsKey(gamepadInstanceId))
if (SDL_IsGamepad(joystickInstanceId))
{ {
if (_gamepadsInstanceIdsMapping.ContainsKey(joystickInstanceId)) // Sometimes a JoyStick connected event fires after the app starts even though it was connected before
// so it is rejected to avoid doubling the entries.
return;
}
string id = GenerateGamepadId(gamepadInstanceId);
if (id == null)
{
return;
}
if (_gamepadsInstanceIdsMapping.TryAdd(gamepadInstanceId, new GamepadInfo(id, SDL_OpenGamepad(gamepadInstanceId))))
{
lock (_lock)
{ {
// Sometimes a JoyStick connected event fires after the app starts even though it was connected before if (gamepadInstanceId <= _gamepadsIds.FindLastIndex(_ => true))
// so it is rejected to avoid doubling the entries. {
return; // _gamepadsIds.Insert(joystickDeviceId, id);
}
else
_gamepadsIds.Add(id);
if (SDL3JoyConPair.IsCombinable(_gamepadsInstanceIdsMapping))
{
_gamepadsIds.Remove(SDL3JoyConPair.Id);
_gamepadsIds.Add(SDL3JoyConPair.Id);
joyConPairConnected = true;
}
} }
string id = GenerateGamepadId(joystickInstanceId); OnGamepadConnected?.Invoke(id);
if (id == null) if (joyConPairConnected)
{ {
return; OnGamepadConnected?.Invoke(SDL3JoyConPair.Id);
}
if (_gamepadsInstanceIdsMapping.TryAdd(joystickInstanceId, id))
{
lock (_lock)
{
if (joystickInstanceId <= _gamepadsIds.FindLastIndex(_ => true))
{
// _gamepadsIds.Insert(joystickDeviceId, id);
}
else
_gamepadsIds.Add(id);
if (SDL3JoyConPair.IsCombinable(_gamepadsInstanceIdsMapping))
{
_gamepadsIds.Remove(SDL3JoyConPair.Id);
_gamepadsIds.Add(SDL3JoyConPair.Id);
joyConPairConnected = true;
}
}
OnGamepadConnected?.Invoke(id);
if (joyConPairConnected)
{
OnGamepadConnected?.Invoke(SDL3JoyConPair.Id);
}
} }
} }
} }
private void HandleJoyBatteryUpdated(uint joystickDeviceId, SDL_JoyBatteryEvent joyBatteryEvent) private void HandleJoyBatteryUpdated(uint joystickDeviceId, SDL_JoyBatteryEvent joyBatteryEvent)
{ {
Logger.Info?.Print(LogClass.Hid, Logger.Info?.Print(LogClass.Hid,
$"{SDL_GetGamepadNameForID(joystickDeviceId)}, Battery percent: {joyBatteryEvent.percent}"); $"{SDL_GetGamepadNameForID(joystickDeviceId)}, Battery percent: {joyBatteryEvent.percent}");
@ -205,27 +200,18 @@ namespace Ryujinx.Input.SDl3
} }
} }
var instanceId = GetJoystickIndexByGamepadId(id); var gamepadInfo = GetJoystickIndexByGamepadId(id);
if (gamepadInfo == null)
if (instanceId == nint.Zero)
{
return null;
}
nint gamepadHandle = SDL_OpenGamepad(instanceId);
if (gamepadHandle == nint.Zero)
{ {
return null; return null;
} }
Console.WriteLine(SDL_GetGamepadName(gamepadHandle)); if (SDL3JoyCon.IsJoyCon(gamepadInfo.gamepadHandle))
if (SDL_GetGamepadName(gamepadHandle).StartsWith(SDL3JoyCon.Prefix))
{ {
return new SDL3JoyCon(gamepadHandle, id); return new SDL3JoyCon(gamepadInfo);
} }
return new SDL3Gamepad(gamepadHandle, id); return new SDL3Gamepad(gamepadInfo);
} }
} }
} }

View File

@ -61,27 +61,27 @@ namespace Ryujinx.Input.SDL3
public GamepadFeaturesFlag Features { get; } public GamepadFeaturesFlag Features { get; }
private nint _gamepadHandle; private nint _gamepadHandle;
private enum JoyConType private enum JoyConType
{ {
Left, Right Left, Right
} }
public const string Prefix = "Nintendo Switch Joy-Con";
public const string LeftName = "Nintendo Switch Joy-Con (L)"; public const string LeftName = "Nintendo Switch Joy-Con (L)";
public const string RightName = "Nintendo Switch Joy-Con (R)"; public const string RightName = "Nintendo Switch Joy-Con (R)";
private readonly JoyConType _joyConType; private readonly JoyConType _joyConType;
public SDL3JoyCon(nint gamepadHandle, string driverId) public SDL3JoyCon(GamepadInfo gamepadInfo)
{ {
_gamepadHandle = gamepadHandle; _gamepadHandle = gamepadInfo.gamepadHandle;
_buttonsUserMapping = new List<ButtonMappingEntry>(10); _buttonsUserMapping = new List<ButtonMappingEntry>(10);
Name = SDL_GetGamepadName(_gamepadHandle); Name = SDL_GetGamepadName(_gamepadHandle);
Id = driverId; Id = gamepadInfo.driverId;
Features = GetFeaturesFlag(); Features = GetFeaturesFlag();
Console.WriteLine(Name+": "+Features);
// Enable motion tracking // Enable motion tracking
if (Features.HasFlag(GamepadFeaturesFlag.Motion)) if (Features.HasFlag(GamepadFeaturesFlag.Motion))
{ {
@ -118,7 +118,6 @@ namespace Ryujinx.Input.SDL3
private GamepadFeaturesFlag GetFeaturesFlag() private GamepadFeaturesFlag GetFeaturesFlag()
{ {
GamepadFeaturesFlag result = GamepadFeaturesFlag.None; GamepadFeaturesFlag result = GamepadFeaturesFlag.None;
if (SDL_GamepadHasSensor(_gamepadHandle, SDL_SensorType.SDL_SENSOR_ACCEL) && if (SDL_GamepadHasSensor(_gamepadHandle, SDL_SensorType.SDL_SENSOR_ACCEL) &&
SDL_GamepadHasSensor(_gamepadHandle, SDL_SensorType.SDL_SENSOR_GYRO)) SDL_GamepadHasSensor(_gamepadHandle, SDL_SensorType.SDL_SENSOR_GYRO))
{ {
@ -141,8 +140,7 @@ namespace Ryujinx.Input.SDL3
{ {
if (disposing && _gamepadHandle != nint.Zero) if (disposing && _gamepadHandle != nint.Zero)
{ {
SDL_CloseGamepad(_gamepadHandle); // SDL_CloseGamepad(_gamepadHandle);
_gamepadHandle = nint.Zero; _gamepadHandle = nint.Zero;
} }
} }
@ -210,9 +208,7 @@ namespace Ryujinx.Input.SDL3
} }
private static Vector3 RadToDegree(Vector3 rad) => rad * (180 / MathF.PI); private static Vector3 RadToDegree(Vector3 rad) => rad * (180 / MathF.PI);
private static Vector3 GsToMs2(Vector3 gs) => gs / SDL_STANDARD_GRAVITY;
//TODO: miss constant SDL_STANDARD_GRAVITY 9.80665f
private static Vector3 GsToMs2(Vector3 gs) => gs / 9.80665f;
public void SetConfiguration(InputConfig configuration) public void SetConfiguration(InputConfig configuration)
{ {
@ -416,5 +412,11 @@ namespace Ryujinx.Input.SDL3
// } // }
return SDL_GetGamepadButton(_gamepadHandle, button); return SDL_GetGamepadButton(_gamepadHandle, button);
} }
public static bool IsJoyCon(IntPtr gamepadHandle)
{
var gamepadName = SDL_GetGamepadName(gamepadHandle);
return gamepadName is LeftName or RightName;
}
} }
} }

View File

@ -101,41 +101,32 @@ namespace Ryujinx.Input.SDL3
right.SetTriggerThreshold(triggerThreshold); right.SetTriggerThreshold(triggerThreshold);
} }
public static bool IsCombinable(Dictionary<uint, string> gamepadsInstanceIdsMapping) public static bool IsCombinable(Dictionary<uint, GamepadInfo> gamepadsInstanceIdsMapping)
{ {
(uint leftIndex, uint rightIndex) = DetectJoyConPair(gamepadsInstanceIdsMapping); (GamepadInfo leftGamepadInfo, GamepadInfo rightGamepadInfo) = DetectJoyConPair(gamepadsInstanceIdsMapping);
return leftIndex != 0 && rightIndex != 0; return leftGamepadInfo != null && rightGamepadInfo != null;
} }
private static (uint leftInstance, uint rightInstance) DetectJoyConPair( private static (GamepadInfo leftGamepadInfo, GamepadInfo rightGamepadInfo) DetectJoyConPair(
Dictionary<uint, string> gamepadsInstanceIdsMapping) Dictionary<uint, GamepadInfo> gamepadsInstanceIdsMapping)
{ {
var leftInstance = gamepadsInstanceIdsMapping var leftGamepadInfo = gamepadsInstanceIdsMapping
.FirstOrDefault(item => SDL_GetGamepadNameForID(item.Key) == SDL3JoyCon.LeftName).Key; .FirstOrDefault(item => SDL_GetGamepadNameForID(item.Key) == SDL3JoyCon.LeftName).Value;
var rightInstance = gamepadsInstanceIdsMapping var rightGamepadInfo = gamepadsInstanceIdsMapping
.FirstOrDefault(item => SDL_GetGamepadNameForID(item.Key) == SDL3JoyCon.RightName).Key; .FirstOrDefault(item => SDL_GetGamepadNameForID(item.Key) == SDL3JoyCon.RightName).Value;
return (leftInstance, rightInstance); return (leftGamepadInfo, rightGamepadInfo);
} }
public static IGamepad GetGamepad(Dictionary<uint, string> gamepadsInstanceIdsMapping) public static IGamepad GetGamepad(Dictionary<uint, GamepadInfo> gamepadsInstanceIdsMapping)
{ {
(uint leftInstance, uint rightInstance) = DetectJoyConPair(gamepadsInstanceIdsMapping); (GamepadInfo leftGamepadInfo, GamepadInfo rightGamepadInfo) = DetectJoyConPair(gamepadsInstanceIdsMapping);
if (leftInstance == 0 || rightInstance == 0) if (leftGamepadInfo == null || rightGamepadInfo == null)
{ {
return null; return null;
} }
nint leftGamepadHandle = SDL_OpenGamepad(leftInstance); return new SDL3JoyConPair(new SDL3JoyCon(leftGamepadInfo), new SDL3JoyCon(rightGamepadInfo));
nint rightGamepadHandle = SDL_OpenGamepad(rightInstance);
if (leftGamepadHandle == nint.Zero || rightGamepadHandle == nint.Zero)
{
return null;
}
return new SDL3JoyConPair(new SDL3JoyCon(leftGamepadHandle, gamepadsInstanceIdsMapping[leftInstance]),
new SDL3JoyCon(rightGamepadHandle, gamepadsInstanceIdsMapping[rightInstance]));
} }
} }
} }

View File

@ -8048,6 +8048,6 @@ public static unsafe partial class SDL
[UnmanagedCallConv(CallConvs = [typeof(CallConvCdecl)])] [UnmanagedCallConv(CallConvs = [typeof(CallConvCdecl)])]
public static partial int SDL_EnterAppMainCallbacks(int argc, IntPtr argv, SDL_AppInit_func appinit, SDL_AppIterate_func appiter, SDL_AppEvent_func appevent, SDL_AppQuit_func appquit); public static partial int SDL_EnterAppMainCallbacks(int argc, IntPtr argv, SDL_AppInit_func appinit, SDL_AppIterate_func appiter, SDL_AppEvent_func appevent, SDL_AppQuit_func appquit);
public const uint SDL_AUDIO_DEVICE_DEFAULT_PLAYBACK = 0xFFFFFFFFu; public const uint SDL_AUDIO_DEVICE_DEFAULT_PLAYBACK = 0xFFFFFFFFu;
public const float SDL_STANDARD_GRAVITY = 9.80665f;
} }

View File

@ -25,7 +25,8 @@ namespace Ryujinx.SDL3.Common
public static Action<Action> MainThreadDispatcher { get; set; } public static Action<Action> MainThreadDispatcher { get; set; }
private const SDL_InitFlags SdlInitFlags = SDL_InitFlags.SDL_INIT_GAMEPAD | SDL_InitFlags.SDL_INIT_AUDIO; private const SDL_InitFlags SdlInitFlags = SDL_InitFlags.SDL_INIT_GAMEPAD | SDL_InitFlags.SDL_INIT_AUDIO |
SDL_InitFlags.SDL_INIT_VIDEO;
private bool _isRunning; private bool _isRunning;
private uint _refereceCount; private uint _refereceCount;
@ -123,19 +124,18 @@ namespace Ryujinx.SDL3.Common
private void HandleSDLEvent(ref SDL_Event evnt) private void HandleSDLEvent(ref SDL_Event evnt)
{ {
var type = (SDL_EventType)evnt.type; var type = (SDL_EventType)evnt.type;
if (type == SDL_EventType.SDL_EVENT_JOYSTICK_ADDED) if (type == SDL_EventType.SDL_EVENT_GAMEPAD_ADDED)
{ {
uint instanceId = evnt.jdevice.which; uint instanceId = evnt.jdevice.which;
Logger.Debug?.Print(LogClass.Application, $"Added joystick instance id {instanceId}"); Logger.Debug?.Print(LogClass.Application, $"Added joystick instance id {instanceId}");
OnJoyStickConnected?.Invoke(instanceId); OnJoyStickConnected?.Invoke(instanceId);
} }
else if (type == SDL_EventType.SDL_EVENT_JOYSTICK_REMOVED) else if (type == SDL_EventType.SDL_EVENT_GAMEPAD_REMOVED)
{ {
uint instanceId = evnt.jdevice.which; uint instanceId = evnt.jdevice.which;
Logger.Debug?.Print(LogClass.Application, $"Removed joystick instance id {instanceId}"); Logger.Debug?.Print(LogClass.Application, $"Removed joystick instance id {instanceId}");
OnJoystickDisconnected?.Invoke(instanceId); OnJoystickDisconnected?.Invoke(instanceId);
@ -144,7 +144,8 @@ namespace Ryujinx.SDL3.Common
{ {
OnJoyBatteryUpdated?.Invoke(evnt.jbattery.which, evnt.jbattery); OnJoyBatteryUpdated?.Invoke(evnt.jbattery.which, evnt.jbattery);
} }
else if (type is >= SDL_EventType.SDL_EVENT_WINDOW_FIRST and <= SDL_EventType.SDL_EVENT_WINDOW_LAST or SDL_EventType.SDL_EVENT_MOUSE_BUTTON_DOWN else if (type is >= SDL_EventType.SDL_EVENT_WINDOW_FIRST and <= SDL_EventType.SDL_EVENT_WINDOW_LAST
or SDL_EventType.SDL_EVENT_MOUSE_BUTTON_DOWN
or SDL_EventType.SDL_EVENT_MOUSE_BUTTON_UP) or SDL_EventType.SDL_EVENT_MOUSE_BUTTON_UP)
{ {
if (_registeredWindowHandlers.TryGetValue(evnt.window.windowID, out Action<SDL_Event> handler)) if (_registeredWindowHandlers.TryGetValue(evnt.window.windowID, out Action<SDL_Event> handler))