[PR #145] [MERGED] Fix slot issues #223

New issue

Closed

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

roman commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/safijari/openpuck/pull/145
Author: @Leseratte10
Created: 7/6/2026
Status: ✅ Merged
Merged: 7/6/2026
Merged by: @safijari

Base: main ← Head: fix-slot

📝 Commits (1)

8ff861b Fix slot issues

📊 Changes

2 files changed (+20 additions, -3 deletions)

View changed files

📝 Makefile (+4 -3)
📝 ReversePuck/scpair.py (+16 -0)

📄 Description

This seems to fix #126.

The issue is that by default, the OpenPuck Wake mouse is enabled in Steam mode. This mouse uses a HID slot, but the Makefile was configured to only allow 4 HID slots. One was used by the mouse, leaving only three for Puck pairing slots, causing writes to one of the slots to fail.

Steam knew the 1st slot was a mouse, not a puck slot, and didn't display it / refuse to write to it.

scpair.py didn't know that, so it tried to write to the mouse's record, which obviously failed.
Increasing the allowed slot count to 6 fixes the issue, Steam now displays all four slots correctly.

scpair.py, however, did now start displaying 5 slots - one for the mouse, four for the puck slots. So I added some code to skip interfaces that aren't puck bonding slots.

_{🔄 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/145 **Author:** [@Leseratte10](https://github.com/Leseratte10) **Created:** 7/6/2026 **Status:** ✅ Merged **Merged:** 7/6/2026 **Merged by:** [@safijari](https://github.com/safijari) **Base:** `main` ← **Head:** `fix-slot` --- ### 📝 Commits (1) - [`8ff861b`](https://github.com/safijari/openpuck/commit/8ff861b1cec8bd28873095e0aeaf408aa1da78ad) Fix slot issues ### 📊 Changes **2 files changed** (+20 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Makefile` (+4 -3) 📝 `ReversePuck/scpair.py` (+16 -0) </details> ### 📄 Description This seems to fix #126. The issue is that by default, the OpenPuck Wake mouse is enabled in Steam mode. This mouse uses a HID slot, but the Makefile was configured to only allow 4 HID slots. One was used by the mouse, leaving only three for Puck pairing slots, causing writes to one of the slots to fail. Steam knew the 1st slot was a mouse, not a puck slot, and didn't display it / refuse to write to it. scpair.py didn't know that, so it tried to write to the mouse's record, which obviously failed. Increasing the allowed slot count to 6 fixes the issue, Steam now displays all four slots correctly. scpair.py, however, did now start displaying 5 slots - one for the mouse, four for the puck slots. So I added some code to skip interfaces that aren't puck bonding slots. --- <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:50 +00:00