Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers/audio: Add support for the ES8311 codec #9189

Merged
merged 5 commits into from May 4, 2023

Conversation

lucasssvaz
Copy link
Contributor

@lucasssvaz lucasssvaz commented May 4, 2023

Summary

This commit aims to add support for the ES8311 codec on the ESP32-S2-Kaluga-1 board.
It also has small QoL improvements like separating common ESXXXX settings and fixing compilation warnings on ESP32-S2.

Impact

This might break custom applications using i2s_mclkfrequency() as it was renamed to i2s_setmclkfrequency().

Testing

Tested using the provided defconfigs on the ESP32-S2-Kaluga-1.

@jerpelea
Copy link
Contributor

jerpelea commented May 4, 2023

please add commit messages describing the change

include/nuttx/audio/i2s.h Outdated Show resolved Hide resolved
drivers/audio/esxxxx_common.h Outdated Show resolved Hide resolved
Rename i2s_mclkfrequency to i2s_setmclkfrequency and add i2s_getmclkfrequency for getting the current MCLK frequency from the I2S interface
To avoid code duplication, the common settings for Everest Semiconductor's codecs were separated in an "esxxxx_common" header
This commit aims to fix compilation warnings that happens when enabling SPIRAM for ESP32-S2
Adds support for the ES8311 audio codec by Everest Semiconductor on NuttX. Both output and input are supported.
Adds support for the ES8311 codec on the ESP32-S2-Kaluga-1 board. This commit also adds basic configurations for testing and updates the board documentation.
@acassis acassis merged commit b738379 into apache:master May 4, 2023
26 checks passed
@lucasssvaz lucasssvaz deleted the feature/es8311 branch May 5, 2023 12:12
@jerpelea jerpelea added this to To-Add in Release Notes - 12.2.0 Jun 13, 2023
@jerpelea jerpelea moved this from To-Add to In Progress in Release Notes - 12.2.0 Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants