qmk/qmk_firmware

[Bug] PMW3360 Trackball sensor unequal CPI on the master/slave halves

Open

#19,433 opened on Dec 28, 2022

View on GitHub
 (4 comments) (0 reactions) (0 assignees)C (43,867 forks)batch import
bughelp wanted

Repository metrics

Stars
 (20,368 stars)
PR merge metrics
 (Avg merge 20d 9h) (27 merged PRs in 30d)

Description

Describe the Bug

When I set #define PMW33XX_CPI 6000 both senshors should be the same in the movement sensitivity. But for some reason the slave sensor moved slowly. I.e. slave cpi = master cpi / 2. Can't find the reason why this behaviour remains if I switch the master/slave halves. I'm using f401 with full duplex and SPLIT_HAND_PIN.

Keyboard Used

fabis

Link to product page (if applicable)

https://github.com/Onefabis/qmk_firmware/tree/Haptic_split/keyboards/fabis

Operating System

Windows

qmk doctor Output

Ψ QMK Doctor is checking your environment.                                                                                              
Ψ CLI version: 1.1.1                                                                                                                    
Ψ QMK home: C:/Users/Alex_Laptop/qmk_firmware                                                                                           
Ψ Detected Windows 10 (10.0.22000).                                                                                                     
Ψ Git branch: Haptic_split                                                                                                              
Ψ Repo version: 0.19.5                                                                                                                  
⚠ Git has unstashed/uncommitted changes.                                                                                                
Ψ - Latest Haptic_split: 2022-12-24 23:40:28 +0300 (036ad6bc8b) -- Update readme.md                                                     
Ψ - Latest upstream/master: 2022-12-28 05:29:19 +0900 (4c32e060e8) -- [Keyboard] Add XR63GL keyboard (#19380)                           
Ψ - Latest upstream/develop: 2022-12-27 20:29:52 +0000 (40fddcb1e5) -- Merge remote-tracking branch 'origin/master' into develop        
Ψ - Common ancestor with upstream/master: 2022-12-23 18:14:23 +0000 (b56c0fa674) -- Bump actions/stale from 6 to 7 (#19392)             
Ψ - Common ancestor with upstream/develop: 2022-12-23 18:14:23 +0000 (b56c0fa674) -- Bump actions/stale from 6 to 7 (#19392)            
Ψ All dependencies are installed.                                                                                                       
Ψ Found arm-none-eabi-gcc version 10.1.0                                                                                                
Ψ Found avr-gcc version 8.5.0                                                                                                           
Ψ Found avrdude version 7.0                                                                                                             
Ψ Found dfu-util version 0.11                                                                                                           
Ψ Found dfu-programmer version 0.7.2                                                                                                    
Ψ Submodules are up to date.                                                                                                            
Ψ Submodule status:                                                                                                                     
Ψ - lib/chibios: 2022-09-18 10:01:17 +0000 --  (0e9d558b5)                                                                              
Ψ - lib/chibios-contrib: 2022-10-03 18:09:41 +0200 --  (bb8356fb)                                                                       
Ψ - lib/googletest: 2021-06-11 06:37:43 -0700 --  (e2239ee6)                                                                            
Ψ - lib/lufa: 2022-08-26 12:09:55 +1000 --  (549b97320)                                                                                 
Ψ - lib/pico-sdk: 2022-09-19 18:02:44 +0200 --  (8d56ea3)                                                                               
Ψ - lib/printf: 2022-06-29 23:59:58 +0300 --  (c2e3b4e)                                                                                 
Ψ - lib/vusb: 2022-06-13 09:18:17 +1000 --  (819dbc1)                                                                                   
Ψ QMK is ready to go, but minor problems were found

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Contributor guide

Research direction
Investigate the PMW33XX driver code in qmk firmware to understand how CPI is configured for each half of a split keyboard. Look for any scaling factor or different initialization between master and slave halves. Also check the split communication protocol to ensure CPI values are transmitted correctly.
Tech stack
c
Domain
embedded iot
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
CGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30