Skip to content

Conversation

dhalbert
Copy link
Collaborator

@dhalbert dhalbert commented Aug 21, 2025

Use CIRCUITPY_AUDIOIO ?= 1 as the default, then turn it off for chips that don't have it with CIRCUITPY_AUDIOIO = 0. That way, it can also be turned off selectively as needed.

Also add some prominent comments to fence the choices for each chip type, to make it easier to find a particular chip's settings.

Also conditionalize CIRCUITPY_ALARM_TOUCH as being dependent on CIRCUITPY_ALARM on boards that support it.

Tested on Metro ESP32-S2 that import audioio works.

@dhalbert dhalbert requested review from tannewt and FoamyGuy August 22, 2025 01:09
Copy link
Collaborator

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, and looks good to me.

@dhalbert dhalbert merged commit 56bff43 into adafruit:main Aug 22, 2025
247 checks passed
@dhalbert dhalbert deleted the restore-audioio-on-espressif-boards branch August 22, 2025 03:22
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.

Restore audioio in Espressif for boards that can use it
2 participants