[PR #44] [CLOSED] fix: reduce USB HID poll interval to 1ms across all non-Xbox modes #170

New issue

Closed

opened 2026-10-22 05:07:02 +00:00 by roman · 0 comments

roman commented

2026-10-22 05:07:02 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/safijari/openpuck/pull/44
Author: @Copilot
Created: 6/16/2026
Status: ❌ Closed

Base: main ← Head: copilot/43

📝 Commits (1)

66ce6a1 fix: reduce USB poll interval to 1ms on all non-Xbox modes (#43)

📊 Changes

4 files changed (+8 additions, -8 deletions)

View changed files

📝 OpenPuck/mode_hidgyro.cpp (+2 -2)
📝 OpenPuck/mode_ps5.cpp (+2 -2)
📝 OpenPuck/mode_switch_hori.cpp (+2 -2)
📝 OpenPuck/mode_switch_pro.cpp (+2 -2)

📄 Description

Non-Xbox emulation modes had unnecessarily high USB interrupt endpoint bInterval values, adding direct host-polling latency on top of the RF round-trip. Xbox already declared bInterval=1ms ("so the RF rate is the only limit"); the other modes did not.

Mode	bInterval before	Latency before	bInterval after
PS5 / Switch Pro / DS4	4 ms	~9 ms	1 ms
Switch Hori	8 ms	~16 ms	1 ms

Changes

mode_ps5.cpp — setPollInterval(4 → 1)
mode_switch_hori.cpp — setPollInterval(8 → 1)
mode_switch_pro.cpp — setPollInterval(4 → 1)
mode_hidgyro.cpp — setPollInterval(4 → 1)
bcdDevice bumped on each affected mode so Windows invalidates its cached config descriptor

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/safijari/openpuck/pull/44 **Author:** [@Copilot](https://github.com/apps/copilot-swe-agent) **Created:** 6/16/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `copilot/43` --- ### 📝 Commits (1) - [`66ce6a1`](https://github.com/safijari/openpuck/commit/66ce6a1e237dc257730bf5bab6ef6047658b91c4) fix: reduce USB poll interval to 1ms on all non-Xbox modes (#43) ### 📊 Changes **4 files changed** (+8 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `OpenPuck/mode_hidgyro.cpp` (+2 -2) 📝 `OpenPuck/mode_ps5.cpp` (+2 -2) 📝 `OpenPuck/mode_switch_hori.cpp` (+2 -2) 📝 `OpenPuck/mode_switch_pro.cpp` (+2 -2) </details> ### 📄 Description Non-Xbox emulation modes had unnecessarily high USB interrupt endpoint `bInterval` values, adding direct host-polling latency on top of the RF round-trip. Xbox already declared `bInterval=1ms` ("so the RF rate is the only limit"); the other modes did not. | Mode | bInterval before | Latency before | bInterval after | |---|---|---|---| | PS5 / Switch Pro / DS4 | 4 ms | ~9 ms | 1 ms | | Switch Hori | 8 ms | ~16 ms | 1 ms | ## Changes - **`mode_ps5.cpp`** — `setPollInterval(4 → 1)` - **`mode_switch_hori.cpp`** — `setPollInterval(8 → 1)` - **`mode_switch_pro.cpp`** — `setPollInterval(4 → 1)` - **`mode_hidgyro.cpp`** — `setPollInterval(4 → 1)` - `bcdDevice` bumped on each affected mode so Windows invalidates its cached config descriptor --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

roman

2026-10-22 05:07:02 +00:00