Skip to content

feat: Move audio driver setting from config.h to keyboard.json Form Editor#923

Merged
yoichiro merged 1 commit intomainfrom
feature/workbench-audio-driver-keyboard-json
Apr 12, 2026
Merged

feat: Move audio driver setting from config.h to keyboard.json Form Editor#923
yoichiro merged 1 commit intomainfrom
feature/workbench-audio-driver-keyboard-json

Conversation

@yoichiro
Copy link
Copy Markdown
Collaborator

@yoichiro yoichiro commented Apr 12, 2026

Summary

  • Remove AUDIO_DRIVER from config.h Form Editor (QMK ignores it in config.h)
  • Add Audio tab to keyboard.json Form Editor with audio.driver selection
    • FreeSolo autocomplete: select from pwm_hardware, pwm_software, dac_basic, dac_additive or type a custom value
  • Tab order: General → USB/MCU → Hardware → Features → Lighting → Audio → Encoder

Test plan

  • Open config.h → Form Editor → Audio tab → verify AUDIO_DRIVER field is gone
  • Open keyboard.json → Form Editor → verify Audio tab exists between Lighting and Encoder
  • Select "pwm_hardware" from dropdown → switch to Code Editor → verify "audio": { "driver": "pwm_hardware" } in JSON
  • Type custom driver value → verify it appears in JSON
  • Clear driver value → verify audio object is removed from JSON
  • Verify Encoder tab still works at index 6

🤖 Generated with Claude Code

…ditor

QMK Firmware ignores AUDIO_DRIVER in config.h and requires audio.driver
in keyboard.json instead. Remove AUDIO_DRIVER from config.h Form Editor
and add Audio tab to keyboard.json Form Editor with driver selection
(freeSolo autocomplete supporting pwm_hardware, pwm_software, dac_basic,
dac_additive and custom values).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@yoichiro yoichiro merged commit 63e5d3f into main Apr 12, 2026
1 check passed
@yoichiro yoichiro deleted the feature/workbench-audio-driver-keyboard-json branch April 12, 2026 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant