Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WiiUtils: Add helper functions to get emulated/real Bluetooth device #9638

Merged
merged 2 commits into from Apr 13, 2021

Conversation

leoetlino
Copy link
Member

This adds a function to get the emulated or real Bluetooth device for
an active emulation instance. This lets us deduplicate all the
ios->GetDeviceByName("/dev/usb/oh1/57e/305") calls that are currently
scattered in the codebase and ensures Bluetooth passthrough is being
handled correctly.

This also fixes the broken check in WiimoteCommon::UpdateSource.
There was a confusion between "emulated Bluetooth" (as opposed to
"real Bluetooth" aka Bluetooth passthrough) and "emulated Wiimote".

@leoetlino leoetlino force-pushed the btemu-is-not-wiimoteemu branch 2 times, most recently from 16f1aaf to e894a5a Compare April 12, 2021 14:10
Source/Core/Core/Movie.cpp Outdated Show resolved Hide resolved
Source/Core/Core/HW/Wiimote.cpp Outdated Show resolved Hide resolved
This adds a function to get the emulated or real Bluetooth device for
an active emulation instance. This lets us deduplicate all the
`ios->GetDeviceByName("/dev/usb/oh1/57e/305")` calls that are currently
scattered in the codebase and ensures Bluetooth passthrough is being
handled correctly.

This also fixes the broken check in WiimoteCommon::UpdateSource.
There was a confusion between "emulated Bluetooth" (as opposed to
"real Bluetooth" aka Bluetooth passthrough) and "emulated Wiimote".
Copy link
Contributor

@JMC47 JMC47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified to fix the issue.

@JosJuice JosJuice merged commit e8ac63d into dolphin-emu:master Apr 13, 2021
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants