Skip to content

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

Open
leosat wants to merge 2 commits intoemberian:mainfrom
leosat:main
Open

Inertia and Damper are also condition-ed effects.#169
leosat wants to merge 2 commits intoemberian:mainfrom
leosat:main

Conversation

@leosat
Copy link

@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 those 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
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.

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.

2 participants