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

Documentation and API adjustments for I2S #309

Merged
merged 6 commits into from
Sep 23, 2023

Conversation

dacut
Copy link
Contributor

@dacut dacut commented Sep 22, 2023

This fixes a number of usability issues identified by https://github.com/kyrias around the I2S driver and fixes a bug in ESP-IDF around PDM mode stereo vs. mono mode on ESP-C3/-C6/-H2 devices.

I've added documentation around frame formats and adjusted the API around DMA buffering to reflect my now better understanding of what those parameters do.

This is #283 but rebased with the latest async fixes.

This changes the slot mask configuration in standard mode for
non ESP32/ESP32-S3 chips to use left-channel only mode. This behavior
differs from the ESP-IDF SDK, which appears to be in error due when
a preprocessor macro was renamed.
@dacut dacut changed the title Closed Documentation and API adjustments for I2S Documentation and API adjustments for I2S Sep 22, 2023
@ivmarkov ivmarkov merged commit 29360a2 into esp-rs:master Sep 23, 2023
8 checks passed
@dacut dacut deleted the i2s-stereo-fix branch September 25, 2023 19:09
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.

2 participants