dahjah/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-12-16 04:09:39 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

WiimoteReal/IOhidapi: Log the currently attached hid driver on Linux.

Browse Source
This commit is contained in:
Jordan Woyak 2025-10-03 18:10:42 -05:00
parent 7609220e61
commit 4d53de5d84
1 changed files with 51 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
Source/Core/Core/HW/WiimoteReal/IOhidapi.cpp
View File

@ -3,7 +3,9 @@
#include "Core/HW/WiimoteReal/IOhidapi.h"
#include <algorithm>
#if defined(__linux__)
#include <filesystem>
#endif
#include "Common/Assert.h"
#include "Common/Logging/Log.h"
@ -12,6 +14,50 @@
using namespace WiimoteCommon;
using namespace WiimoteReal;
#if defined(__linux__)
// Here we check the currently attached Linux driver just for logging purposes.
// Maybe in the future we'll be able to bind the desired HID driver somehow.
static void LogLinuxDriverName(const char* device_path)
{
namespace fs = std::filesystem;
const auto sys_device = "/sys/class/hidraw" / fs::path(device_path).filename() / "device";
// "driver" is a symlink to a path that contains the driver name.
// usually /sys/bus/hid/drivers/hid-generic (what we want)
// or this /sys/bus/hid/drivers/wiimote (what we don't want)
std::error_code err;
const auto sys_driver = fs::canonical(sys_device / "driver", err);
if (err)
{
WARN_LOG_FMT(WIIMOTE, "Could not determine current driver of {}. error: {}", device_path,
err.message());
return;
}
const auto driver_name = sys_driver.filename();
if (driver_name == "wiimote")
{
// If Linux's 'wiimote' driver is attached, we will be forever fighting with it.
// It's not going to work very well.
// One could write fs::canonical(sys_device).filename() to (sys_driver / "unbind")
// And then also write it to "/sys/bus/hid/drivers/hid-generic/bind"
// This should create a new hidraw device with the 'hid-generic' driver.
// But that requires elevated permissions.. Linux is annoying.
ERROR_LOG_FMT(WIIMOTE, "Wii remote at {} has the Linux 'wiimote' driver attached.",
device_path);
}
else
{
DEBUG_LOG_FMT(WIIMOTE, "Wii remote at {} has driver: {}", device_path, driver_name.string());
}
}
#endif
static bool IsDeviceUsable(const std::string& device_path)
{
hid_device* handle = hid_open_path(device_path.c_str());
@ -73,6 +119,10 @@ void WiimoteScannerHidapi::FindWiimotes(std::vector<Wiimote*>& wiimotes, Wiimote
if (!is_wiimote || !IsNewWiimote(device->path) || !IsDeviceUsable(device->path))
continue;
#if defined(__linux__)
LogLinuxDriverName(device->path);
#endif
auto* wiimote = new WiimoteHidapi(device->path);
const bool is_balance_board = IsBalanceBoardName(name) || wiimote->IsBalanceBoard();
if (is_balance_board)