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

add sdkconfig for PCM5102A sdkconfig_custom_PCM5102A #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LeoSum8
Copy link

@LeoSum8 LeoSum8 commented Aug 8, 2024

Working config for an ESP32 (tested on ESP-WROOM-32 module using ESP32-D0WD-V3 chip revision v3.0) with a PCM5102A DAC, using the following PINS:

CONFIG_MASTER_I2S_BCK_PIN=26
CONFIG_MASTER_I2S_LRCK_PIN=25
CONFIG_MASTER_I2S_DATAOUT_PIN=22
CONFIG_PCM5102A_MUTE_PIN=33

enables DSP and soft volume control that is required for this dac.
enables sample stuffing.

Working config for an ESP32 (tested on ESP-WROOM-32 module using ESP32-D0WD-V3 chip revision v3.0) with a PCM5102A DAC, using the following PINS:

CONFIG_MASTER_I2S_BCK_PIN=26
CONFIG_MASTER_I2S_LRCK_PIN=25
CONFIG_MASTER_I2S_DATAOUT_PIN=22
CONFIG_PCM5102A_MUTE_PIN=33
@CarlosDerSeher
Copy link
Owner

Are those the default pins from menuconfig or did you change them?

@LeoSum8
Copy link
Author

LeoSum8 commented Aug 8, 2024

I changed them. I think originally I was inspired by this one here when I tried to get the DAC to work so I used the same Pins.

But what do you mean by default pins from menuconfig?
To me, after selecting that DAC in menuconfig, the default pins didn't make any sense. E.g. i2s slave still needed to be manually set to -1.

@CarlosDerSeher
Copy link
Owner

CarlosDerSeher commented Aug 8, 2024

Ok, I'll see to that, probably i2s slave should be removed as it is unused anyhow. It's just some relic from the beginning of the project.
Maybe I'll just change the defaults to the ones you provide for i2s master.

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