Skip to content

Inertia and Damper are also condition-ed effects.#169

Merged
emberian merged 2 commits intoemberian:mainfrom
leosat:main
Mar 13, 2026
Merged

Inertia and Damper are also condition-ed effects.#169
emberian merged 2 commits intoemberian:mainfrom
leosat:main

Conversation

@leosat
Copy link
Contributor

@leosat leosat commented Feb 6, 2026

The four condition-ed effects are spring, friction, damper, inertia: Ref: Universal Serial Bus (USB) Device Class Definition for Physical Interface Devices (PID) https://www.usb.org/sites/default/files/documents/pid1_01.pdf page 11, Table 3: Effect Types


Hi. Currently condition data is not propagated for inertia and damper effects, however it needs to.

The four condition-ed effects are spring, friction, damper, inertia:

Some refs:

Universal Serial Bus (USB) Device Class Definition for Physical Interface Devices (PID)
https://www.usb.org/sites/default/files/documents/pid1_01.pdf
page 11, Table 3: Effect Types

gdk ref

ffbtools also provide condition settings for those effects.

This patch adds condition data propagation to evdev API for the latter two effects.

The four condition-ed effects are spring, friction, damper, inertia:
Ref: Universal Serial Bus (USB) Device Class Definition for Physical Interface Devices (PID)
https://www.usb.org/sites/default/files/documents/pid1_01.pdf
page 11, Table 3: Effect Types
…s it masks erroneous cases such as forgotten to be handled properly effects.
@laanwj
Copy link

laanwj commented Feb 6, 2026

Looks good to me. I need this as well for https://github.com/laanwj/remote-uinput-rs .

@leosat
Copy link
Contributor Author

leosat commented Feb 7, 2026

Dear maintainers, @emberian , @coolreader18 , please consider the patch as it fixes the 'bug' and shall "unblock" some projects (e.g. mentioned by @laanwj or mine) that need this to be handled properly. Please tell if any changes required. I've tested the patch locally.

@emberian emberian merged commit de39ec0 into emberian:main Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants