Meet the team
See all 1 on GitHub →
Commit activity
Last 12 weeks · 1 commit
Community health
2 of 6 standards met
42
✓README✓License○Contributing○Code of Conduct○Issue Template○PR Template
Recent PRs & issues
Quiet · 4 in progress · Last activity 2mo ago
See all on GitHub →
On line 23 of , there is a small typo, or at least an incomplete sentence: "You don't need a LED for any of this, but it's nice to have for visual debugging, so just be aware that". I don't know what this line should actually say, but my best guess is that is missing 'of' and a dot. This issue feels a bit unnecessary, since this was probably already acknowledged, but hope this helps anyway!
debounce_program_init() for state machine 3 was passed GPIO_SW3 instead of GPIO_SW4. State machine 2 already handles SW3, so SW4 was left without a working state machine and never fired.
as5600_init() was reading and writing AS5600_STATUS (0x0B, read-only) instead of AS5600_CONF (0x07), so hysteresis and filter settings were never applied. The fourth debounce state machine was initialized against GPIO_SW3 instead of GPIO_SW4, leaving SW4 unmonitored.
Recent fixes
init_sw_pins() passes GPIO_SW3 to both sm=2 and sm=3, leaving sm=3 watching the wrong pin. Since read_switch(3) drains sm=3's RX FIFO which never fills, SW4 never fires no matter how it's wired. One-line fix: pass GPIO_SW4 to debounce_program_init(pio, 3, ...). // Line 50
Structured data for AI agents
Repository: torvalds/ScrollWheel. Description: Minimalist RP2350 magnetic sensor scroll wheel toy project Stars: 364, Forks: 12. Primary language: C. Languages: C (85.3%), CMake (12%), Makefile (2.7%). License: GPL-2.0. Open PRs: 4, open issues: 2. Last activity: 2mo ago. Community health: 42%. Top contributors: torvalds.