summaryrefslogtreecommitdiff
path: root/tmk_core
AgeCommit message (Collapse)Author
2023-09-25[Core] Bump mouse endpoint packet size to 16 bytes (#21711)Stefan Kerkmann
2023-09-03Remove old `IS_LED_ON/OFF()` macros (#21878)Ryan
2023-08-27Revert changes to ChibiOS Suspend Code (#21830)Drashna Jaelre
* Partially revert #19780 * Finish * Get teensy 3.5/3.6 board files too * fix lint issue * Revert "[Bug] Restore usb suspend wakeup delay (#21676)" This reverts commit e8e989fd7ad7c10e725e50ae8b0a4426e09f7f30. * Apply suggestions from code review Co-authored-by: Joel Challis <git@zvecr.com> --------- Co-authored-by: Joel Challis <git@zvecr.com>
2023-08-02[Bug] Restore usb suspend wakeup delay (#21676)Stefan Kerkmann
* Respect USB_SUSPEND_WAKEUP_DELAY on wakeup This delay wasn't honored after removing `restart_usb_driver` from the suspend and wakeup handling. It is now re-introduced in the appropriate spot, namely after issuing a remote wakeup to a sleeping host. * Remove old, unused and commented testing code
2023-08-02Fix mouse-key spamming empty reports (#21663)Stefan Kerkmann
Problem: `mousekey_task` spams empty hid reports with when a mouse key is pressed, causing resource exhaustion in the USB mouse endpoint. Cause: The check whether or not to send a new mouse report would always evaluate to true if a mouse key is pressed: 1. `mouse_report` has non-zero fields and `tmpmr` is a copy of this fields. 2. `mouse_report` is set to zero, `tmpmr` has now non-zero fields. 3. `has_mouse_report_changed` compares the two and evaluates to true 4. a mouse report is sent. Fix: The check condition of `has_mouse_report_changed` will evaluate any empty record as unchanged, as mouse report data is relative and doesn't need to return to zero. An empty report will still be send by `register_mouse` on release of all mouse buttons.
2023-07-28Update keyboard report descriptor for USB compliance (#21626)Stefan Kerkmann
Running the "HID Tests" suite of the USB 3 Command Verifier (USB3CV) tool resulted in the following error: (HID: 3.2.61) The report descriptor returned in response to a GetDescriptor(Report) must be compliant with the HID specification. Byte Number: 37h ( 55d) Data Field: 91 02 Mnemonic: Output Value: (Variable) Errors: Error: LOGICAL MAX MUST be bounded by Report Size The error stems from the fact that logical minimum and maximum are global items, which means that the next item in a report descriptor inherits the value from the previously set value. In this case the status leds item inherited the logical minimum (=0) and maximum (=255) from the keycodes item. As the status leds set a report size of 1 bit, wich can only hold a boolean, it becomes clear that this range would never fit. The fix is straightforward, we just define a appropriate logical maximum (=1), the mismatch is solved and our keyboard now passes the compliance tests. Defining the logical minimum is redundant in this case but is kept to form a logical block.
2023-07-16quantum: remove direct `quantum.h` includes (#21507)Ryan
2023-07-15Eliminate `TMK_COMMON_*` in makefiles (#21517)Ryan
2023-07-07tmk_core: remove direct `quantum.h` includes (#21465)Ryan
2023-06-27Get rid of `USB_LED_KANA` and `USB_LED_COMPOSE` (#21366)Ryan
2023-06-26[Bug] Fix non-functional S3 wakeup / resume from suspense (#19780)Stefan Kerkmann
* Update ChibiOS-Contrib for USB suspend fixes * Remove S3 wakup workaround ChibiOS OTGv1 driver has a remote wakeup bug that prevents the device to resume it's operation. 02516cbc24647f522eee975e69cc0c8a925470eb introduced a hotfix that forcefully restarted the usb driver as a workaround. This workaround broke multiple boards which do not use this driver / peripheral. With the update of ChibiOS this hotfix is now obsolete. * Remove restart_usb_driver overrides they are no longer necessary as the workaround is not needed anymore for stm32f4 * Remove unused RP_USB_USE_SOF_INTR defines The SOF interrupt is enabled dynamically by the RP2040 usb driver
2023-06-26Chibios USB: Take into account if host wants remote wakeup or not (#21287)Purdea Andrei
According to the USB 2.0 spec, remote wakeup should be disabled by default, and should only be enabled if the host explicitly requests it. The chibios driver code already takes care of storing this information, and returning it on GET_STATUS requests. However our application code has been ignoring it so far. This is a USB compliance issue, but also a bug that causes trouble in some cases: On RP2040 targets this has been causing problems if a key is held down while the keyboard is plugged in. The keyboard would fail to enumerate until all keys are released. With this change that behavior is fixed. Note that for LUFA targets this is already done correctly.
2023-06-22Move protocol makefiles into their respective folders (#21332)Ryan
* Move protocol makefiles into their respective folders * Fix USB-USB converter
2023-04-05Disable specific warnings to mitigate compilation problems with ↵Nick Brassel
`KEEP_INTERMEDIATES=yes`. (#20339)
2023-02-19Move `KC_MISSION_CONTROL`/`KC_LAUNCHPAD` keycodes to core (#19884)Joel Challis
2023-02-06Remove `IS_HOST_LED_ON` and migrate usages (#19753)Ryan
2023-01-31Fix midi after recent refactoring (#19723)Joel Challis
2023-01-30Move MIDI code out of tmk_core (#19704)Ryan
2023-01-20Fix functions with empty params (#19647)Ryan
* Fix functions with empty params * Found a bunch more
2023-01-16ChibiOS: Consolidate report sending code (#19607)Ryan
2023-01-15Fix joystick build for ChibiOS (#19602)Sergey Vlasov
`joystick_report_t` was renamed to `report_joystick_t`, but apparently one place in the code was missed.
2023-01-15usb_main.c: remove `CH_KERNEL_MAJOR` check (#19597)Ryan
2022-12-09[CI] Format code according to conventions (#19265)QMK Bot
2022-12-09Return USB HID GET_REPORT requests (#14814)Jack Humbert
Co-authored-by: Sergey Vlasov <sigprof@gmail.com> Co-authored-by: Nick Brassel <nick@tzarc.org>
2022-12-09Detect host OS based on USB fingerprint (#18463)Ruslan Sayfutdinov
Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Nick Brassel <nick@tzarc.org>
2022-12-09Fixed NKRO issue caused by HID_SET_PROTOCOL on Chibios platform (#17588)lokher
2022-12-03Revert WB32 ISO workaround (#19224)Joel Challis
2022-11-27Joystick feature improvements (#19052)Ryan
2022-11-12Digitizer feature improvements (#19034)Ryan
2022-10-22ChibiOS USB: Add a dummy IN callback to work around LLD bugs (#18811)Sergey Vlasov
In #18631 some IN notification callbacks that were doing nothing were removed, which should be a valid thing to do (ChibiOS HAL checks the `in_cb` and `out_cb` pointers for being non-NULL before invoking those optional callbacks). However, it turned out that some less popular USB LLDs (KINETIS and MIMXRT1062) have their own checks for those pointers, and (incorrectly) skip the ChibiOS callback handling when those pointers are NULL, which breaks the code for the `USB_USE_WAIT` configuration option (the waiting thread never gets resumed if the corresponding callback pointer is NULL). Add those dummy callbacks again (but use a single function for all of them instead of individual ones for each endpoint); this restores the KINETIS and MIMXRT1062 boards to the working state while the LLDs are getting fixed.
2022-10-19Widen the ARM Cortex-M family support. Allow USB peripheral change. (#18767)Nick Brassel
2022-10-15LUFA: Consolidate report sending code (#18629)Ryan
2022-10-13Fix joystick functionality for ChibiOS and OTG (Blackpill) (#18631)Ryan
Co-authored-by: Sergey Vlasov <sigprof@gmail.com>
2022-10-09Make MIDI output endpoint use the out direction (#18654)nezumee
2022-10-06Refactor `send_extra` (#18615)Ryan
2022-10-05Refactor more host code (programmable button & digitizer) (#18565)Ryan
2022-10-02[Core] ChibiOS: Fix USB bus disconnect handling (#18566)Stefan Kerkmann
2022-09-29Start Bluetooth API (#18366)Ryan
2022-09-27Further refactoring of joystick feature (#18437)Ryan
2022-09-17Chromeos keycodes (#18212)Joshua Diamond
2022-09-12Merge remote-tracking branch 'origin/master' into developJoshua Diamond
2022-09-10GMMK2: Fix 'ISO' within product name (#18322)Joel Challis
2022-09-07Move Bluetooth-related function calls up to host/keyboard level (#18274)Ryan
* Move Bluetooth-related function calls up to host/keyboard level * Remove pointless set_output() call * Move bluetooth (rn42) init to end of keyboard_init() * Enable SPI/UART for ChibiOS targets * Some more slight tweaks
2022-09-02Adjust `EXTRAKEY_ENABLE` ifdefs for `send_extra()` (#18249)Ryan
2022-09-02Simplify extrakeys sending at the host driver level (#18230)Ryan
* Simplify extrakeys sending at the host driver level * There are two arguments here * Wrong syntax * Adjust keyboards which use a custom host driver
2022-08-30Use a macro to compute the size of arrays at compile time (#18044)Jeff Epler
* Add ARRAY_SIZE and CEILING utility macros * Apply a coccinelle patch to use ARRAY_SIZE * fix up some straggling items * Fix 'make test:secure' * Enhance ARRAY_SIZE macro to reject acting on pointers The previous definition would not produce a diagnostic for ``` int *p; size_t num_elem = ARRAY_SIZE(p) ``` but the new one will. * explicitly get definition of ARRAY_SIZE * Convert to ARRAY_SIZE when const is involved The following spatch finds additional instances where the array is const and the division is by the size of the type, not the size of the first element: ``` @ rule5a using "empty.iso" @ type T; const T[] E; @@ - (sizeof(E)/sizeof(T)) + ARRAY_SIZE(E) @ rule6a using "empty.iso" @ type T; const T[] E; @@ - sizeof(E)/sizeof(T) + ARRAY_SIZE(E) ``` * New instances of ARRAY_SIZE added since initial spatch run * Use `ARRAY_SIZE` in docs (found by grep) * Manually use ARRAY_SIZE hs_set is expected to be the same size as uint16_t, though it's made of two 8-bit integers * Just like char, sizeof(uint8_t) is guaranteed to be 1 This is at least true on any plausible system where qmk is actually used. Per my understanding it's universally true, assuming that uint8_t exists: https://stackoverflow.com/questions/48655310/can-i-assume-that-sizeofuint8-t-1 * Run qmk-format on core C files touched in this branch Co-authored-by: Stefan Kerkmann <karlk90@pm.me>
2022-08-29Merge remote-tracking branch 'origin/master' into developQMK Bot
2022-08-29Redefinition of MIN is avoided in midi.c (#18203)3araht
2022-08-28Switch over MANUFACTURER and PRODUCT to string literals (#18183)Ryan
2022-08-08Remove tmk_core 'serial' code (#17866)Joel Challis