Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] SPLIT_HAPTIC_ENABLE doesn't work #57

Closed
2 tasks
Onefabis opened this issue Dec 24, 2022 · 3 comments
Closed
2 tasks

[Bug] SPLIT_HAPTIC_ENABLE doesn't work #57

Onefabis opened this issue Dec 24, 2022 · 3 comments

Comments

@Onefabis
Copy link

Onefabis commented Dec 24, 2022

Describe the Bug

I did a test with your PR.
#define SPLIT_HAPTIC_ENABLE still don't want to work on the slave side, only on the master side. If I switch the sides connection, the problem remains the same: master vibrating, slave - not.

Keyboard Used

onefabis

Link to product page (if applicable)

[firmware]https://github.com/Onefabis/qmk_firmware/tree/Haptic_split
[keyboard]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: master
Ψ Repo version: 0.19.5
⚠ Git has unstashed/uncommitted changes.
Ψ - Latest master: 2022-12-23 18:14:23 +0000 (b56c0fa) -- Bump actions/stale from 6 to 7 (qmk#19392)
Ψ - Latest upstream/master: 2022-12-23 18:14:23 +0000 (b56c0fa) -- Bump actions/stale from 6 to 7 (qmk#19392)
Ψ - Latest upstream/develop: 2022-12-23 11:18:57 -0700 (003cee0) -- Validate keyboard name before accepting further input (qmk#19394)
Ψ - Common ancestor with upstream/master: 2022-12-23 18:14:23 +0000 (b56c0fa) -- Bump actions/stale from 6 to 7 (qmk#19392)
Ψ - Common ancestor with upstream/develop: 2022-12-23 18:14:23 +0000 (b56c0fa) -- Bump actions/stale from 6 to 7 (qmk#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
[Alex_Laptop@Alex_Laptop ~]$

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

@drashna
Copy link
Owner

drashna commented May 5, 2023

Sorry, I've finally been able to test this. Unfortunately, it seems like this works fine for me, no matter which side is plugged in, or if only one side has the haptics connected.

@freznel10
Copy link

Can confirm as well that this works for me as well

@drashna
Copy link
Owner

drashna commented May 7, 2023

for reference, used 2x blackpills with drv2605l controllers, and tested them out:

20230504_214435

@drashna drashna closed this as completed Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants