diff options
author | liamwhite <liamwhite@users.noreply.github.com> | 2024-02-25 20:00:59 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-25 20:00:59 +0100 |
commit | 8416d1c028b3d0fe7ab70dacd28ab8e076014f70 (patch) | |
tree | 3ef0a2763194a01147b9f3e134efeaee8dc41b5b | |
parent | Merge pull request #13160 from liamwhite/time (diff) | |
parent | core: hid: Reintroduce vibration filter (diff) | |
download | yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar.gz yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar.bz2 yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar.lz yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar.xz yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.tar.zst yuzu-8416d1c028b3d0fe7ab70dacd28ab8e076014f70.zip |
-rw-r--r-- | src/hid_core/frontend/emulated_controller.cpp | 17 | ||||
-rw-r--r-- | src/hid_core/frontend/emulated_controller.h | 1 | ||||
-rw-r--r-- | src/hid_core/hid_types.h | 6 | ||||
-rw-r--r-- | src/hid_core/resources/npad/npad.cpp | 1 |
4 files changed, 23 insertions, 2 deletions
diff --git a/src/hid_core/frontend/emulated_controller.cpp b/src/hid_core/frontend/emulated_controller.cpp index 5cd26819c..7664341bd 100644 --- a/src/hid_core/frontend/emulated_controller.cpp +++ b/src/hid_core/frontend/emulated_controller.cpp @@ -2,6 +2,7 @@ // SPDX-License-Identifier: GPL-2.0-or-later #include <algorithm> +#include <chrono> #include <common/scope_exit.h> #include "common/polyfill_ranges.h" @@ -1287,6 +1288,22 @@ bool EmulatedController::SetVibration(DeviceIndex device_index, const VibrationV return false; } + if (!Settings::values.enable_accurate_vibrations.GetValue()) { + using std::chrono::duration_cast; + using std::chrono::milliseconds; + using std::chrono::steady_clock; + + const auto now = steady_clock::now(); + + // Filter out non-zero vibrations that are within 15ms of each other. + if ((vibration.low_amplitude != 0.0f || vibration.high_amplitude != 0.0f) && + duration_cast<milliseconds>(now - last_vibration_timepoint[index]) < milliseconds(15)) { + return false; + } + + last_vibration_timepoint[index] = now; + } + // Exponential amplification is too strong at low amplitudes. Switch to a linear // amplification if strength is set below 0.7f const Common::Input::VibrationAmplificationType type = diff --git a/src/hid_core/frontend/emulated_controller.h b/src/hid_core/frontend/emulated_controller.h index ab3c6fcd3..17ad6069e 100644 --- a/src/hid_core/frontend/emulated_controller.h +++ b/src/hid_core/frontend/emulated_controller.h @@ -583,6 +583,7 @@ private: std::size_t nfc_handles{0}; std::array<VibrationValue, 2> last_vibration_value{DEFAULT_VIBRATION_VALUE, DEFAULT_VIBRATION_VALUE}; + std::array<std::chrono::steady_clock::time_point, 2> last_vibration_timepoint{}; // Temporary values to avoid doing changes while the controller is in configuring mode NpadStyleIndex tmp_npad_type{NpadStyleIndex::None}; diff --git a/src/hid_core/hid_types.h b/src/hid_core/hid_types.h index 1b2fc6295..38888fdd1 100644 --- a/src/hid_core/hid_types.h +++ b/src/hid_core/hid_types.h @@ -638,7 +638,11 @@ struct VibrationValue { if (low_amplitude != b.low_amplitude || high_amplitude != b.high_amplitude) { return false; } - if (low_frequency != b.low_amplitude || high_frequency != b.high_frequency) { + // Changes in frequency without amplitude don't have any effect + if (low_amplitude == 0 && high_amplitude == 0) { + return true; + } + if (low_frequency != b.low_frequency || high_frequency != b.high_frequency) { return false; } return true; diff --git a/src/hid_core/resources/npad/npad.cpp b/src/hid_core/resources/npad/npad.cpp index e10e97e1c..ca1ccd659 100644 --- a/src/hid_core/resources/npad/npad.cpp +++ b/src/hid_core/resources/npad/npad.cpp @@ -3,7 +3,6 @@ #include <algorithm> #include <array> -#include <chrono> #include <cstring> #include "common/assert.h" |