Allow the user to select a single tone for the additive DAC #21591
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
zvecr
approved these changes
Jul 23, 2023
fauxpark
approved these changes
Jul 23, 2023
antempus
added a commit
to antempus/qmk_firmware
that referenced
this pull request
Sep 6, 2023
* [Keyboard] Add minidivide (qmk#19870) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Drashna Jaelre <drashna@live.com> * Fix jaykeeb indicator state and pins (qmk#21466) * Bump JamesIves/github-pages-deploy-action from 4.4.2 to 4.4.3 (qmk#21518) Bumps [JamesIves/github-pages-deploy-action](https://github.com/jamesives/github-pages-deploy-action) from 4.4.2 to 4.4.3. - [Release notes](https://github.com/jamesives/github-pages-deploy-action/releases) - [Commits](JamesIves/github-pages-deploy-action@v4.4.2...v4.4.3) --- updated-dependencies: - dependency-name: JamesIves/github-pages-deploy-action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Prepare ymdk/ymd75 for rev4 (qmk#21484) * [Keyboard] add mothwing (qmk#19342) Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: jack <0x6a73@protonmail.com> * Eliminate `TMK_COMMON_*` in makefiles (qmk#21517) * quantum: remove direct `quantum.h` includes (qmk#21507) * [Core] Update ChibiOS-Contrib (qmk#21553) * AliceH66 Pianoforte Hotswap Touch-Up (qmk#21542) * [refactor] rename LAYOUT_tkl_ansi to LAYOUT_tkl_f13_ansi * [enhancement] enable Community Layout support * [chore] correct layout data Correct the position of the spacebar. * GMMK2 65% ANSI Touch-Up (qmk#21544) * [chore] correct layout data * [chore] rename LAYOUT to LAYOUT_65_ansi_blocker Also adds a layout alias so as to not have to edit the user keymaps. * [enhancement] enable Community Layout support * Keychron V4 Touch-Up (qmk#21545) * Keychron V4 ANSI: rename LAYOUT_ansi_61 to LAYOUT_60_ansi * Keychron V4 ANSI: enable Community Layout support * Keychron V4 ISO: correct layout data Move position [3, 1] up by 0.25u. * Keychron V4 ISO: rename LAYOUT_iso_62 to LAYOUT_60_iso * Keychron V4 ISO: enable Community Layout support * Fix compilation of lily58 rgb_state_reader (qmk#21502) * HolySwitch Lightweight65 Community Layout Support (qmk#21550) * [chore] rename LAYOUT to LAYOUT_65_ansi_blocker * [enhancement] enable Community Layout support * Automata02 Alisaie Touch-Up (qmk#21552) * [style] friendly-format info.json * [chore] remove key labels from info.json * [chore] correct layout data Fix a misleading Configurator implementation, and incorrect key sizes and positioning on the bottom row. * [chore] re-add key labels to info.json * Move files to rev1 folder, add rev2 (qmk#21563) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * Amend layout and matrix positions for `dactyl_cc` (qmk#21523) * Spruce up `dactyl_lightcycle` and `dactyl_maximus` layouts (qmk#21519) * Add Bastion75 PCB to QMK (qmk#21529) * Add Bastion75 to QMK * Remove blank layers * Apply suggestions from code review Co-authored-by: jack <0x6a73@protonmail.com> * Update keymaps --------- Co-authored-by: jack <0x6a73@protonmail.com> * Document the right location for indicator functions (qmk#21512) * Rename `dactyl_manuform` variant `3x5_3` (qmk#21015) * haptic: naming cleanups (qmk#21551) * backlight: split AVR PWM and timer drivers (qmk#21540) * Fix backlight support for some boards (qmk#21554) * [Keyboard] Add boardsource/unicorne (qmk#21510) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Cole Smith <smithcole@protonmail.com> * [Keyboard] Add EC Theca (qmk#21233) Co-authored-by: jack <0x6a73@protonmail.com> * [Bug] Fix compilation error when Split Watchdog enabled (qmk#21543) * Revert "[Bug] Fix compilation error when Split Watchdog enabled" (qmk#21572) * kinesis: remove stacked split layouts (qmk#21569) * Fix VS code live share extension link (qmk#21585) * Allow the user to select one tone for the additive DAC (qmk#21591) Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error: ``` platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror] 91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0}; | ^ ``` This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz. * [Keyboard] Update ERA65 PCB (qmk#21592) * Raw HID: documentation improvements (qmk#21596) * Add Nyquistpad (qmk#21578) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> * Fix paragon keyboard encoder step (qmk#21582) * [Keyboard] Add linworks fave60a (qmk#20795) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: jack <0x6a73@protonmail.com> * [Keyboard] Delete jscotto directory (qmk#21157) * Add Caps Lock indicator for Kyechron S1 white backlight version (qmk#21559) * [Keyboard] Add nico (qmk#20686) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Joel Challis <git@zvecr.com> * Fix ik75 scroll lock and windows lock status LEDs (qmk#20380) * [keyboard] tominabox1/le_chiffre refactor pt 1 (qmk#21567) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * Move model h controller under ibm/model_m (qmk#21526) * Add Bastion60 PCB to QMK (qmk#21527) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Danny <nooges@users.noreply.github.com> * Add BastionTKL PCB to QMK (qmk#21530) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Danny <nooges@users.noreply.github.com> * Add Bastion65 PCB to QMK (qmk#21528) Co-authored-by: jack <0x6a73@protonmail.com> * [Keyboard] boardsource/equals (qmk#21230) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: jack <0x6A73@pm.me> * [Keyboard] Add m1 keyboard (qmk#19799) Co-authored-by: adophoxia <100170946+adophoxia@users.noreply.github.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: wb-joy <chang.li@westberrytech.com> Co-authored-by: Joy Lee <joylee.lc@foxmail.com> * [Keyboard] Phoenix Project No 1 support (qmk#21068) Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Drashna Jaelre <drashna@live.com> * Revert "Equals keyboard" (qmk#21622) * Fix stuck note with square wave in additive DAC (qmk#21589) * [Keyboard] Add splitkb.com's Aurora Sofle v2 (qmk#21499) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Thomas Baart <mail@thomasbaart.nl> * On-each-release tap dance function (qmk#20255) Co-authored-by: Sergey Vlasov <sigprof@gmail.com> * Reduce needless precision in audio note frequency calculation (qmk#21496) * Remove needless precision in additive DAC sample generation (qmk#21498) * Amend documented `RGBLIGHT_HUE_STEP` default value (qmk#21617) * helix/rev3_4rows, helix/rev3_5rows: Fix build with converters (qmk#21608) The code for Helix rev3 boards could not be built with, e.g., `CONVERT_TO=rp2040_ce` due to invalid `#include <stdio.h>` in the keyboard level `config.h` (apparently the AVR version of that file contains guards against usage in assembly code, which was hiding the bug). Move `#include <stdio.h>` to the C sources which need it. * Unicode: move keycode aliases to a separate header (qmk#21613) * Unify MIDI note calculation with the audio feature's algorithm (qmk#21588) * [Keyboard] Add STM32f3 Discovery onekey (qmk#21625) * [Keyboard] Add Divine keyboard (qmk#21593) * Update keyboard report descriptor for USB compliance (qmk#21626) 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. * [Keyboard] add itstleo40 PCB (qmk#21536) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Drashna Jaelre <drashna@live.com> * [Keymap] update colemad crkbd (corne) keyboard (qmk#21515) Co-authored-by: Álvaro Cortés Devesa <alvarocortesdevesa@Alvaros-MacBook-Pro.local> * [Keyboard] Add ANAVI Macro Pad 12 (qmk#21547) * Updated info.json of c2 pro (qmk#21581) * quantum.h: clean up process_keycode includes (qmk#21579) * Bluetooth: driver naming cleanups (qmk#21624) * LED Matrix: driver naming cleanups (qmk#21580) * Correct "less than" to "up to" in squeezing_avr?id=layers (qmk#21639) * CLI: Improve error messages when layout key matrix row/col is OOB (qmk#21640) * Fix layout offsets for a handful of boards (qmk#21636) * keycode aliases: work around ChibiOS ch.h include guard (qmk#21497) * Remove old OLED API code (qmk#21651) * AMJKeyboard AMJ84 Develop Touch-Up (qmk#21645) * [style] refactor default keymap, phase 1 - grid-align keycodes - use QMK-native keycode aliases - update box drawings * [refactor] refactor default keymap, phase 2 - move position [5, 8] to the top row - update keycode grid alignment * [refactor] refactor default keymap, phase 3 - remove matrix position [5, 9] * Remove layout aliases from keyboard_aliases.hjson (qmk#21658) * Keychron Q11 ISO: Fix left Shift breakage due to a wrong matrix mask (qmk#21655) Keychron Q11 uses `matrix_mask` to mask away the matrix location used for `SPLIT_HAND_MATRIX_GRID` (otherwise the keyboard will constantly generate a wakeup request whenever the host enters suspend). However, the value of `matrix_mask` is actually chosen to mask away all unused locations, but it was apparently generated for the ANSI board and therefore masks away the left Shift key on the ISO board, breaking the functionality of that key. Fix `matrix_mask` to make the masking work as intended: - Set the `[4, 1]` bit to 1; this is the matrix location for the ISO left Shift. - Set the `[2, 5]` bit to 0; this is actually the matrix location which corresponds to `#define SPLIT_HAND_MATRIX_GRID A2, A15` on the left side, which was intended to be masked; apparently the wakeup problem does not happen due to `SPLIT_HAND_MATRIX_GRID_LOW_IS_RIGHT`, so the left half actually has that matrix location open. On the right side `A2, A15` is `[9, 6]`, which was masked off correctly. * Fix Dynamic Macro Compilation for avr-gcc 5.4.0 + Linux (qmk#21653) Co-authored-by: Joel Challis <git@zvecr.com> * Automata02 Alisaie Develop Touch-Up (qmk#21630) * Fix mouse-key spamming empty reports (qmk#21663) 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. * audio: Don't play the first note of zero-note melodies (qmk#21661) * [Keyboard] Add Bento Box macropad (qmk#20521) * [Bug] Restore usb suspend wakeup delay (qmk#21676) * 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 * moondrop/dash75: fix keymap (qmk#21680) * [Keyboard] Add PW88 keyboard (qmk#21482) Co-authored-by: Drashna Jaelre <drashna@live.com> * Clarify custom keycode enum requirement for PRs (qmk#21648) * Add VIA layout for Dactyl Manuform 5x6 (qmk#21649) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * haptic: further naming cleanups (qmk#21682) * Contributing docs: add note to make sure repo is up to date (qmk#21694) * Fix `binepad/bn009` and `durgod/dgk6x/hades_ansi` keymaps (qmk#21701) * doio/kb38: fix layout (qmk#21704) * Remove more legacy config.h options (qmk#21709) * Tidy up encoder in matrix references (qmk#21718) * [Keyboard] Update usb.* of `dactyl_lightcycle` (qmk#21462) * [Keyboard] add laneware `raindrop` keyboard (qmk#21277) Co-authored-by: Sergey Vlasov <sigprof@gmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> * [Keyboard] Add Projectd/65/Projectd_65_ansi (qmk#21520) Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: gksygithub <106651989+gksygithub@users.noreply.github.com> * [Keymap] Add Via keymap for CXT Studio (qmk#21675) * [Keyboard] fix json for nemui65 (qmk#21629) * [Keyboard] Update USB settings for dactyl_maximus (qmk#21666) * [Keyboard] Add mk47 keyboard (qmk#19797) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Joy <chang.li@westberrytech.com> Co-authored-by: Joy Lee <joylee.lc@foxmail.com> * correct keymap file name for tkl_f13_jis layout (qmk#21631) * [Keyboard] Refactor Caticorn PCB (qmk#21644) * Kintwin controller for kinesis keyboard, split layout (qmk#21614) Co-authored-by: Ryan <fauxpark@gmail.com> * Fix `keyboard` field for some JSON keymaps (qmk#21752) * [Keyboard] Add Shostudio Arc (qmk#21733) * [Keymap] Update rpbaptist crkbd (qmk#21674) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Richard Baptist <richard@remote.com> * [Keymap] Update andrebrait keymap for GMMK Pro (qmk#21668) * [Keymap] submitting janat-t planck keymap (qmk#21619) Co-authored-by: Janat Taerakul <jtaerakul@indeed.com> * [keyboard] Graystudio Think6.5 V3 Support Revised (qmk#21560) Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> * Remove invalid config option from kikoslab/kl90 (qmk#21708) * [Keyboard] Add Chouchou keyboard (qmk#21699) * OLED: driver naming cleanups (qmk#21710) * Update usb information for dactyl_cc (qmk#21612) * rgblight: driver selection cleanups (qmk#21558) * [Keyboard] add willoucom/keypad (qmk#21714) * Add ScottoErgo Keyboard (qmk#20944) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> * Add Scotto34 Keyboard (qmk#20943) Co-authored-by: Albert Y <76888457+filterpaper@users.noreply.github.com> Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> * Add ScottoInvader Keyboard (qmk#20941) Co-authored-by: Ryan <fauxpark@gmail.com> * coarse/cordillera: fix backlight config (qmk#21761) * The PoweredByPorridge layout for the Keyboardio Atreus (qmk#21737) Co-authored-by: jack <0x6a73@protonmail.com> * [Keymap] add squigglybob splitkb kyria rev2 keymap (qmk#21751) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> Co-authored-by: jack <0x6a73@protonmail.com> * [Bug] Fix compilation error for APA on ChibiOS (qmk#21773) * Add Galatea rev3 keyboard (qmk#21758) * Adding support for TAN67 PCB (qmk#21003) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * [Keyboard] Add Jelly Evolv (qmk#21407) Co-authored-by: jack <0x6a73@protonmail.com> * [Keyboard] Add linworks favepada (qmk#20794) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> Co-authored-by: jack <0x6a73@protonmail.com> * Initial support for ALT v2 (qmk#21754) * Initial support for CTRL v2 (qmk#21755) * Initial support for SHIFT v2 (qmk#21756) * Align SENSE75 with recent Drop additions (qmk#21757) * Fix drop/shift/v2 compilation (qmk#21800) * [Keyboard] Add city42 (qmk#21727) Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * fix: restore indicators to jellybean_raindrops (qmk#21792) After qmk#21169, rgb_matrix_indicators() was limited to running on the final render iteration. Since the jellybean_raindrops animation immediately returns false after updating a single LED, the iteration count no longer ends up high enough to render non-advanced indicators. This change also brings jellybean_raindrops more in line with raindrops. * Simplfy RGB/LED matrix effect logic (qmk#21703) * RGB Matrix: driver naming cleanups (qmk#21594) * Fix keyboards with old RGB driver names (qmk#21815) * [Doc] Improve magic keycode section (qmk#21787) * Fix keyboards with old RGB driver names (qmk#21817) * keychron/c2_pro/ansi/white: Fix column 19 in the custom matrix (qmk#21805) Although `keychron/c2_pro/ansi/rgb` and `keychron/c2_pro/ansi/white` use the same custom matrix code, the matrix layouts are slightly different; in particular, only the `keychron/c2_pro/ansi/white` board actually uses column 19. However, the handling of column 19 in the custom matrix code was broken, therefore that column did not work. Looks like the custom matrix code assumes that `SHIFT_COL_END` refers to the last column connected to the shift register, and not to the column past that; so the value of `SHIFT_COL_END` needs to be changed from 19 to 18 (columns 11...18 are connected to the shift register, and column 19 is connected to the C14 pin). Also the code which was determining `SIZE_T` and `UNSELECT_ALL_COL` had an off-by-one bug when counting the required number of bits (again due to the confusion on the `SHIFT_COL_END` meaning); this had been fixed too (the actual behavior of that part of the code did not change, because both the old and the new version select the 8 bit variant). * Fix infinite recursion in lxxt encoder logic (qmk#21819) * Bump tj-actions/changed-files from 37 to 38 (qmk#21822) Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 37 to 38. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](tj-actions/changed-files@v37...v38) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Generalize AVR -Werror=array-bounds workaround (qmk#21798) GCC 13 needs the same workaround as GCC 12. To avoid having to maintain an ever-growing list of broken versions apply the workaround wherever it can be used. If at some point a fixed version of GCC is released the workaround can be disabled for those fixed versions. See qmk#17064 * Move RGBLight animations to data driven (qmk#21635) * Move RGBLight animations to data driven, 0-9 * Move RGBLight animations to data driven, A * Move RGBLight animations to data driven, B * Move RGBLight animations to data driven, C * Move RGBLight animations to data driven, D * Move RGBLight animations to data driven, E * Move RGBLight animations to data driven, F * Move RGBLight animations to data driven, G * Move RGBLight animations to data driven, H * Move RGBLight animations to data driven, handwired * Move RGBLight animations to data driven, I * Move RGBLight animations to data driven, J * Move RGBLight animations to data driven, K * Move RGBLight animations to data driven, L * Move RGBLight animations to data driven, M * Move RGBLight animations to data driven, N * Move RGBLight animations to data driven, O * Move RGBLight animations to data driven, P * Move RGBLight animations to data driven, Q * Move RGBLight animations to data driven, R * Move RGBLight animations to data driven, S * Move RGBLight animations to data driven, T * Move RGBLight animations to data driven, U * Move RGBLight animations to data driven, V * Move RGBLight animations to data driven, W * Move RGBLight animations to data driven, X * Move RGBLight animations to data driven, Y * Move RGBLight animations to data driven, Z * Fix incorrect placement * Fix build failures and mismatches * converter/ibm_terminal: Fix F0_BREAK state transition (qmk#21824) * Remove `led_matrix.hue_steps` and `led_matrix.sat_steps` from schema (qmk#21827) * Add "apm32-dfu" in keyboard.jsonschema (qmk#21842) * Corrected duration example, should be number not string (qmk#21839) * Rework info.json reference (qmk#21324) * Rework info.json reference * Apply suggestions from code review Co-authored-by: Sergey Vlasov <sigprof@gmail.com> * Update backlight.pins description * Add rgblight.driver and lowercase other driver names * Remove `split.main` and `split.matrix_grid` for now as they are not working yet --------- Co-authored-by: Sergey Vlasov <sigprof@gmail.com> * Revert changes to ChibiOS Suspend Code (qmk#21830) * Partially revert qmk#19780 * Finish * Get teensy 3.5/3.6 board files too * fix lint issue * Revert "[Bug] Restore usb suspend wakeup delay (qmk#21676)" This reverts commit e8e989f. * Apply suggestions from code review Co-authored-by: Joel Challis <git@zvecr.com> --------- Co-authored-by: Joel Challis <git@zvecr.com> * Add *_MATRIX_LED_COUNT generation (qmk#21110) * Unicode, Unicodemap and UCIS refactor (qmk#21659) * Clean up Unicode API usage in user keymaps (qmk#21849) * Breaking changes changelog, 2023q3 version (qmk#21848) Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: honorless <86894501+lesshonor@users.noreply.github.com> * Remove develop notice. * Fix compilation of dynamic_keymap.c without VIA (qmk#21858) * fix 60_hhkb ASCII diagram (qmk#21813) * chore(ferris/test): visually format keymap (qmk#21851) ... and set all keys to printable characters (as described in the keymap's readme) * fix 60_jis keymap.c (qmk#21820) * Fix invalid encoder_map declaration (qmk#21868) * Update keymap.c to add encoder (qmk#21469) * Update keymap.c to add encoder * Fix build --------- Co-authored-by: zvecr <git@zvecr.com> * doio/kb16: Added timeout option for OLED boot logo. (qmk#21823) * Revert "fix 60_hhkb ASCII diagram (qmk#21813)" (qmk#21869) This reverts commit af2d6ea. * CannonKeys Meetup Pad 2023 (qmk#21806) Co-authored-by: Ryan <fauxpark@gmail.com> * [keyboard] add sweeq, a ferris/sweep variant with diodes (qmk#21597) * Update macOS install script (qmk#21854) * Abko AK84BT Layout Touch-Up (qmk#21876) * [refactor] rename LAYOUT_75_ansi to LAYOUT * [enhancement] touch-up layout * AH Haven65 Community Layout support (qmk#21880) * [refactor] rename LAYOUT to LAYOUT_65_ansi_blocker * [enhancement] enable Community Layout support * nullbitsco/snap: fix broken keymaps (qmk#21877) * chore: ignore virtual environment directory (qmk#20905) * Add 1upkeyboards Super16v3 (qmk#21665) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> * [keyboard] add redragon/k667 (qmk#21417) Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: syc <squall_sha@163.com> * Add M6-C Keyboard Firmware (qmk#21852) Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Wolf <contact@vwolf.be> * Added kiserdesigns keyboard folder and Madeline keyboard (qmk#21857) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> * redragon/k667: Remove 2 layer VIA restriction (qmk#21889) * AH Haven80 Community Layout support (qmk#21885) * [enhancement] correct layout data * [refactor] rename LAYOUT to LAYOUT_tkl_f13_ansi_tsangan * [enhancement] enable Community Layout support * Haven80 Solder: add LAYOUT_tkl_f13_ansi_tsangan_split_bs_rshift [enhancement] * Haven80 Solder: add LAYOUT_tkl_f13_iso_tsangan_split_bs_rshift [enhancement] * Haven80 Solder: add LAYOUT_tkl_f13_iso_tsangan [enhancement] * Haven80 Solder: extend Community Layout support [enhancement] * ANAVI Macro Pad 12: rename LAYOUT to LAYOUT_ortho_4x3 (qmk#21886) [refactor] * Chosfox CF81: correct layout data (qmk#21902) * Auto-label data-driven efforts (qmk#21900) * CannonKeys Bastion75 Layout Additions (qmk#21899) * [docs] add layout/matrix diagram * [refactor] rename LAYOUT to LAYOUT_all * [enhancement] add LAYOUT_ansi_blocker_split_bs * [enhancement] add LAYOUT_ansi_blocker * [enhancement] add LAYOUT_ansi_blocker_tsangan_split_bs * [enhancement] add LAYOUT_ansi_blocker_tsangan * [enhancement] add LAYOUT_iso_blocker_split_bs * [enhancement] add LAYOUT_iso_blocker * [enhancement] add LAYOUT_iso_blocker_tsangan_split_bs * [enhancement] add LAYOUT_iso_blocker_tsangan * CannonKeys Bastion60 Layout Additions (qmk#21887) * [bugfix] correct LAYOUT_60_iso_tsangan key order * [docs] add layout/matrix diagram * [enhancement] add LAYOUT_60_tsangan_hhkb * [enhancement] add LAYOUT_60_ansi_tsangan * [enhancement] add LAYOUT_60_iso * [enhancement] add LAYOUT_60_iso_split_bs_rshift * [enhancement] add LAYOUT_60_iso_tsangan_split_bs_rshift * [enhancement] add LAYOUT_60_hhkb * [enhancement] extend Community Layout support * CannonKeys Bastion65 Layout Additions (qmk#21888) * [docs] add layout/matrix diagram * [refactor] rename LAYOUT to LAYOUT_all * [enhancement] add LAYOUT_65_ansi_blocker_split_bs * [enhancement] add LAYOUT_65_ansi_blocker * [enhancement] add LAYOUT_65_ansi_blocker_tsangan_split_bs * [enhancement] add LAYOUT_65_ansi_blocker_tsangan * [enhancement] add LAYOUT_65_iso_blocker_split_bs * [enhancement] add LAYOUT_65_iso_blocker * [enhancement] add LAYOUT_65_iso_blocker_tsangan_split_bs * [enhancement] add LAYOUT_65_iso_blocker_tsangan * [enhancement] enable Community Layout support * CannonKeys BastionTKL Layout Additions (qmk#21901) * [docs] add layout/matrix diagram * [enhancement] add LAYOUT_tkl_ansi_split_bs_rshift * [enhancement] add LAYOUT_tkl_ansi_tsangan_split_bs_rshift * [enhancement] add LAYOUT_tkl_ansi_tsangan * [enhancement] add LAYOUT_tkl_f13_ansi_tsangan * [enhancement] add LAYOUT_tkl_f13_ansi_split_bs_rshift * [enhancement] add LAYOUT_tkl_f13_ansi * [enhancement] correct LAYOUT_tkl_iso key order * [enhancement] add LAYOUT_tkl_iso_split_bs_rshift * [enhancement] add LAYOUT_tkl_iso_tsangan_split_bs_rshift * [enhancement] add LAYOUT_tkl_iso_tsangan * [enhancement] add LAYOUT_tkl_f13_iso_split_bs_rshift * [enhancement] add LAYOUT_tkl_f13_iso * [enhancement] add LAYOUT_tkl_f13_iso_tsangan_split_bs_rshift * [enhancement] add LAYOUT_tkl_f13_iso_tsangan * [enhancement] extend Community Layout support * Bump actions/checkout from 3 to 4 (qmk#21907) Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * GMMK2 65% ISO: Correct layout data (qmk#21925) Correct key sizes/positioning on the right side of the board. --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: takashicompany <t@kashi.company> Co-authored-by: jack <0x6a73@protonmail.com> Co-authored-by: Ryan <fauxpark@gmail.com> Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: QMK Bot <hello@qmk.fm> Co-authored-by: Alabahuy <rezzaadio@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: tan-t <22813443+tan-t@users.noreply.github.com> Co-authored-by: Stefan Kerkmann <karlk90@pm.me> Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com> Co-authored-by: Alfredo Colas Gullon <57442104+LyricPants66133@users.noreply.github.com> Co-authored-by: Danny <nooges@users.noreply.github.com> Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com> Co-authored-by: Andrew Kannan <andrew.kannan@gmail.com> Co-authored-by: フィルターペーパー <76888457+filterpaper@users.noreply.github.com> Co-authored-by: jack <0x6A73@pm.me> Co-authored-by: Cole Smith <smithcole@protonmail.com> Co-authored-by: Cipulot <40441626+Cipulot@users.noreply.github.com> Co-authored-by: peepeetee <43021794+peepeetee@users.noreply.github.com> Co-authored-by: Nebuleon <2391500+Nebuleon@users.noreply.github.com> Co-authored-by: era <73109780+eerraa@users.noreply.github.com> Co-authored-by: Yak <68242321+yakMM@users.noreply.github.com> Co-authored-by: DUILCHOI <48977039+duilchoi@users.noreply.github.com> Co-authored-by: Joe Scotto <contact@joescotto.com> Co-authored-by: Kumiko as a Service <Dreista@users.noreply.github.com> Co-authored-by: yulei <yuleiz@gmail.com> Co-authored-by: jestercore <33821148+jestercore@users.noreply.github.com> Co-authored-by: Less/Rikki <86894501+lesshonor@users.noreply.github.com> Co-authored-by: Jerome Berclaz <jerome@percipient.ai> Co-authored-by: cole smith <38364556+daysgobye@users.noreply.github.com> Co-authored-by: jonylee@hfd <108282809+jonylee1986@users.noreply.github.com> Co-authored-by: adophoxia <100170946+adophoxia@users.noreply.github.com> Co-authored-by: wb-joy <chang.li@westberrytech.com> Co-authored-by: Joy Lee <joylee.lc@foxmail.com> Co-authored-by: Rico <95101980+mymakercorner@users.noreply.github.com> Co-authored-by: Nick Brassel <nick@tzarc.org> Co-authored-by: harveysch <126267034+harveysch@users.noreply.github.com> Co-authored-by: Thomas Baart <mail@thomasbaart.nl> Co-authored-by: leep-frog <66687468+leep-frog@users.noreply.github.com> Co-authored-by: Sergey Vlasov <sigprof@gmail.com> Co-authored-by: Its TLeo <134472524+ItsTLeo@users.noreply.github.com> Co-authored-by: Álvaro Cortés <alvarocortesdevesa@gmail.com> Co-authored-by: Álvaro Cortés Devesa <alvarocortesdevesa@Alvaros-MacBook-Pro.local> Co-authored-by: Leon Anavi <leon@anavi.org> Co-authored-by: lalalademaxiya1 <66767061+lalalademaxiya1@users.noreply.github.com> Co-authored-by: 3geek14 <nerd.of.pi@gmail.com> Co-authored-by: Synth and Keys <117340582+Synth-and-Keys@users.noreply.github.com> Co-authored-by: SmollChungus <38044391+SmollChungus@users.noreply.github.com> Co-authored-by: Hugh Rawlinson <hughrawlinson@users.noreply.github.com> Co-authored-by: gskygithub <106651989+gskygithub@users.noreply.github.com> Co-authored-by: gksygithub <106651989+gksygithub@users.noreply.github.com> Co-authored-by: Brian McKenna <brian@brianmckenna.org> Co-authored-by: spbgzh <72414103+spbgzh@users.noreply.github.com> Co-authored-by: HorrorTroll <sonicvipduc@gmail.com> Co-authored-by: alvicstep <alvicstep@gmail.com> Co-authored-by: mechlovin <57231893+mechlovin@users.noreply.github.com> Co-authored-by: Richard Baptist <rpbaptist@users.noreply.github.com> Co-authored-by: Richard Baptist <richard@remote.com> Co-authored-by: Andre Brait <andrebrait@gmail.com> Co-authored-by: Janat Taerakul <janat.taerakul@gmail.com> Co-authored-by: Janat Taerakul <jtaerakul@indeed.com> Co-authored-by: Yizhen Liu <62583086+edwardslau@users.noreply.github.com> Co-authored-by: Dane Lipscombe <danelipscombe@gmail.com> Co-authored-by: Wilfried JEANNIARD <willou.com@gmail.com> Co-authored-by: Joe Scotto <8194147+joe-scotto@users.noreply.github.com> Co-authored-by: Powered by Porridge <11315349+PoweredByPorridge@users.noreply.github.com> Co-authored-by: squigglybob <squigglybob@users.noreply.github.com> Co-authored-by: studiokestra <74369928+studiokestra@users.noreply.github.com> Co-authored-by: Pangorin <68218885+Pangorin@users.noreply.github.com> Co-authored-by: owlab2 <64054049+lizhenmingdirk@users.noreply.github.com> Co-authored-by: Controller Works <103382872+controller-works@users.noreply.github.com> Co-authored-by: Thomas Weißschuh <thomas@t-8ch.de> Co-authored-by: patrickxiong <patrick.xiong@gmail.com> Co-authored-by: HiryKun <1951086367@qq.com> Co-authored-by: Andy Smith <andysmithfal@users.noreply.github.com> Co-authored-by: Adam Price <komidore64@gmail.com> Co-authored-by: Laneware <68452738+Laneware@users.noreply.github.com> Co-authored-by: Lasse Luttermann <lasse.luttermann@gmail.com> Co-authored-by: Andrew Kannan <andrew.kannan@klaviyo.com> Co-authored-by: Idan Kamara <i@idank.me> Co-authored-by: Jay Greco <jayv.greco@gmail.com> Co-authored-by: Frank Ebel <git@ebel.page> Co-authored-by: ziptyze <mwcordier@gmail.com> Co-authored-by: temp4gh <temp4bbs@163.com> Co-authored-by: syc <squall_sha@163.com> Co-authored-by: Wolf Van Herreweghe <wolfvanh@gmail.com> Co-authored-by: Wolf <contact@vwolf.be> Co-authored-by: NCKiser <37519411+NCKiser@users.noreply.github.com>
jesperhellberg
pushed a commit
to jesperhellberg/qmk_firmware
that referenced
this pull request
Sep 9, 2023
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
thismarvin
pushed a commit
to thismarvin/qmk_firmware
that referenced
this pull request
Sep 27, 2023
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
akeep
pushed a commit
to akeep/qmk_firmware
that referenced
this pull request
Oct 2, 2023
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
csolje
pushed a commit
to csolje/qmk_firmware
that referenced
this pull request
Oct 21, 2023
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
autoferrit
pushed a commit
to SpaceRockMedia/bastardkb-qmk
that referenced
this pull request
Dec 8, 2023
Previously, if the user tried to use `#define AUDIO_MAX_SIMULTANEOUS_TONES 1` to conserve CPU time, their firmware would fail to build with this error:
```
platforms/chibios/drivers/audio_dac_additive.c:91:73: error: excess elements in array initializer [-Werror]
91 | static float active_tones_snapshot[AUDIO_MAX_SIMULTANEOUS_TONES] = {0, 0};
| ^
```
This also affected the preset `#define AUDIO_DAC_QUALITY_VERY_HIGH`, which called up only one simultaneous tone at 88200 Hz.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Previously, if the user tried to build a firmware with audio enabled and
#define AUDIO_MAX_SIMULTANEOUS_TONES 1to conserve CPU time, their firmware would fail to build with this error:This also affected the preset
#define AUDIO_DAC_QUALITY_VERY_HIGH, which called up only one simultaneous tone at 88200 Hz.This pull request changes the array initialization to allow for only one element if chosen.
Types of Changes
Checklist