Files
rootiest df671b656b fix(q5_max): fix RAW_EPSIZE and via_command_kb hook conflict
Two build errors:

1. RAW_EPSIZE undeclared — usb_descriptor.h is not in scope when
   keymap.c is compiled through Keychron's build path. Replace all
   uses with HID_PACKET_SIZE (= 32), now defined in hid_protocol.h.

2. via_command_kb duplicate symbol — keychron_raw_hid.c already defines
   via_command_kb (non-weak) so we cannot redefine it in keymap.c.

Fix by adding a weak kc_raw_hid_rx_kb() extension hook to
keychron_raw_hid.c (following the same pattern as kc_rgb_matrix_rx).
kc_raw_hid_rx() now calls this hook from its default case instead of
returning false directly. The keymap overrides kc_raw_hid_rx_kb() to
handle our custom HID command range (0x40-0x7E).
2026-04-10 15:10:14 -04:00

80 lines
2.5 KiB
C

/* Copyright 2024 @ Keychron (https://www.keychron.com)
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
#pragma once
#define PROTOCOL_VERSION 0x02
#define MISC_PROTOCOL_VERSION 0x0002
#define QMK_COMMAND_SET 2
enum {
KC_GET_PROTOCOL_VERSION = 0xA0,
KC_GET_FIRMWARE_VERSION = 0xA1,
KC_GET_SUPPORT_FEATURE = 0xA2,
KC_GET_DEFAULT_LAYER = 0xA3,
};
enum {
FEATURE_DEFAULT_LAYER = 0x01U << 0,
FEATURE_BLUETOOTH = 0x01U << 1,
FEATURE_P24G = 0x01U << 2,
FEATURE_ANALOG_MATRIX = 0x01U << 3,
FEATURE_INFO_CHAGNED_NOTIFY = 0x01U << 4,
FEATURE_DYNAMIC_DEBOUNCE = 0x01U << 5,
FEATURE_SNAP_CLICK = 0x01U << 6,
FEATURE_KEYCHRON_RGB = 0x01U << 7,
};
enum {
MISC_DFU_INFO = 0x01 << 0,
MISC_LANGUAGE = 0x01 << 1,
MISC_DEBOUNCE = 0x01 << 2,
MISC_SNAP_CLICK = 0x01 << 3,
MISC_WIRELESS_LPM = 0x01 << 4,
MISC_REPORT_REATE = 0x01 << 5,
};
enum {
MISC_GET_PROTOCOL_VER = 0x01,
DFU_INFO_GET,
LANGUAGE_GET,
LANGUAGE_SET,
DEBOUNCE_GET, // 5
DEBOUNCE_SET,
SNAP_CLICK_GET_INFO,
SNAP_CLICK_GET,
SNAP_CLICK_SET,
SNAP_CLICK_SAVE, // A
WIRELESS_LPM_GET,
WIRELESS_LPM_SET,
REPORT_RATE_GET,
REPORT_RATE_SET,
};
/**
* Keyboard-level Raw HID extension hook.
*
* Called by kc_raw_hid_rx() for any command ID not handled by Keychron's own
* protocol (0xA0-0xAB range). The default weak implementation returns false
* so that unrecognised commands fall through to VIA.
*
* Override this in your keymap to handle custom command IDs without
* conflicting with Keychron's via_command_kb() definition.
* Return true if the packet was fully handled (including calling
* raw_hid_send() if a reply is needed), false to let VIA process it.
*/
bool kc_raw_hid_rx_kb(uint8_t *data, uint8_t length);