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] Right Hand Side Split Keyboard Suspend Issue. #23133

Open
2 tasks
chrisistansell opened this issue Feb 23, 2024 · 1 comment
Open
2 tasks

[Bug] Right Hand Side Split Keyboard Suspend Issue. #23133

chrisistansell opened this issue Feb 23, 2024 · 1 comment

Comments

@chrisistansell
Copy link

Describe the Bug

Minor modifications to the code implemented on the keyboard puts the right hand into an always on state when my computer is shut down.

In the main q11 file q11.c - modified line 84 end of the command from (255, 255, 255) to (255, 0, 0)

// RGB_MATRIX_INDICATOR_SET_COLOR(index, red, green, blue);

if defined(CAPS_LOCK_LED_INDEX)

if (host_keyboard_led_state().caps_lock) {
    RGB_MATRIX_INDICATOR_SET_COLOR(CAPS_LOCK_LED_INDEX, 255, 0, 0);

Keyboard right side keyboard will not shut down with the computer.

Tested modifying the main folder info.json file instead. I added all the RGB animations to the list and compiled with that being the only changes.

         "none": true,
        "solid_color ": true,
        "alphas_mods": true,
        "gradient_up_down": true,
        "gradient_left_right": true,
        "breathing": true,
        "band_sat": true,
        "band_val": true,
        "band_pinwheel_sat": true,
        "band_pinwheel_val": true,
        "band_spiral_sat": true,
        "band_spiral_val": true,
        "cycle_all": true,
        "cycle_left_right": true,
        "cycle_up_down": true,
        "cycle_out_in": true,
        "cycle_out_in_dual": true,
        "rainbow_moving_chevron": true,
        "cycle_pinwheel": true,
        "cycle_spiral": true,
        "dual_beacon": true,
        "rainbow_beacon": true,
        "rainbow_pinwheels": true,
        "flower_blooming": true,
        "raindrops": true,
        "jellybean_raindrops": true,
        "hue_breathing": true,
        "hue_pendulum": true,
        "hue_wave": true,
        "pixel_fractal": true,
        "pixel_flow": true,
        "pixel_rain": true,
        "typing_heatmap": true,
        "digital_rain": true,
        "solid_reactive_simple": true,
        "solid_reactive": true,
        "solid_reactive_wide": true,
        "solid_reactive_multiwide": true,
        "solid_reactive_cross": true,
        "solid_reactive_multicross": true,
        "solid_reactive_nexus": true,
        "solid_reactive_multinexus": true,
        "splash": true,
        "multisplash": true,
        "solid_splash": true,
        "solid_multisplash": true,
        "starlight": true,
        "starlight_dual_hue": true,
        "starlight_dual_sat": true,
        "riverflow": true,
        "effect_max": true

This produced the same result, the suspend issue on the right keyboard happens with this change as well.

I gain both functions but at the cost of the keyboard staying on.

Keyboard Used

qmk compile -kb keychron/q11/ansi_encoder -km keychron

Link to product page (if applicable)

https://www.keychron.com/products/keychron-q11-qmk-custom-mechanical-keyboard

Operating System

Windows 11 - Edited with Visual Studio

qmk doctor Output

Welcome to QMK MSYS!
  * Documentation:    https://docs.qmk.fm
  * Support:          https://discord.gg/Uq7gcHh

If you have not already done so,
  run qmk setup to get started.
  run qmk compile -kb <keyboard> -km default to start building.
  run qmk config user.hide_welcome=True to hide this message.
[chris@CHRISEKWB ~]$ qmk doctor
Ψ QMK Doctor is checking your environment.
Ψ CLI version: 1.1.2
Ψ QMK home: C:/Users/chris/qmk_firmware
Ψ Detected Windows 10 (10.0.22631).
Ψ QMK MSYS version: 1.8.0
Ψ Userspace enabled: False
Ψ Git branch: master
Ψ Repo version: 0.20.6
⚠ Git has unstashed/uncommitted changes.
Ψ - Latest master: 2024-02-16 18:35:10 -0500 (496d093fc3) -- [Keyboard] Add chlx ppr merro (#23014)
Ψ - Latest upstream/master: 2024-02-21 14:35:39 -0800 (4e04da397e) -- [Keyboard] Add the Compensator (#23103)
Ψ - Latest upstream/develop: None
Ψ - Common ancestor with upstream/master: 2024-02-16 18:35:10 -0500 (496d093fc3) -- [Keyboard] Add chlx ppr merro (#23014)
Ψ - Common ancestor with upstream/develop: None
Ψ All dependencies are installed.
Ψ Found arm-none-eabi-gcc version 12.2.0
Ψ Found avr-gcc version 12.2.0
⚠ We do not recommend avr-gcc newer than 8. Downgrading to 8.x is recommended.
Ψ Found avrdude version 7.0
Ψ Found dfu-programmer version 1.1.0
Ψ Found dfu-util version 0.11
Ψ Submodules are up to date.
Ψ Submodule status:
Ψ - lib/chibios: 2023-04-15 13:48:04 +0000 --  (11edb1610)
Ψ - lib/chibios-contrib: 2023-07-17 11:39:05 +0200 --  (da78eb37)
Ψ - lib/googletest: 2021-06-11 06:37:43 -0700 --  (e2239ee6)
Ψ - lib/lufa: 2022-08-26 12:09:55 +1000 --  (549b97320)
Ψ - lib/vusb: 2022-06-13 09:18:17 +1000 --  (819dbc1)
Ψ - lib/printf: 2022-06-29 23:59:58 +0300 --  (c2e3b4e)
Ψ - lib/pico-sdk: 2023-02-12 20:19:37 +0100 --  (a3398d8)
Ψ - lib/lvgl: 2022-04-11 04:44:53 -0600 --  (e19410f8)
Ψ QMK is ready to go, but minor problems were found
[chris@CHRISEKWB ~]$

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

I've saved each one of these as I made the changes in both the firmware files and the .build created from compiling. Can share if needed.

@chrisistansell
Copy link
Author

I tried moving this keyboard around after modifications to the firmware. Suspend issue didn't show up on an Aorus motherboard but would on an Asus.

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

1 participant