Skip to content

Reviewed and updated MA12070P driver. Added I2S 32-bit word mode#220

Merged
CarlosDerSeher merged 1 commit intoCarlosDerSeher:developfrom
anabolyc:features/ma12070p-driver-support
May 6, 2026
Merged

Reviewed and updated MA12070P driver. Added I2S 32-bit word mode#220
CarlosDerSeher merged 1 commit intoCarlosDerSeher:developfrom
anabolyc:features/ma12070p-driver-support

Conversation

@anabolyc
Copy link
Copy Markdown
Contributor

Hey folks.

Some time ago, I've created a board with Infineon MA12070P DAC: Loud-ESP32-Plus
I finally found some time to try to spin it up with the driver included with snapclient. As it turned out, code was really raw, not compiling at all, and contained multiple DAC specific errros. I did a full review and rewrite, testing different DAC modes and settings. These are the ones I found optimal.

One thing that took me a long time to figure out - this DAC required 32-bit word on I2S in standard mode. I had to update player.c to change to 32-bit, when this DAC is selected.

Other implementations I took as a reference:

  1. Linux kernel version: https://github.com/raspberrypi/linux/blob/rpi-6.12.y/sound/soc/codecs/ma120x0p.c
  2. muVox project: https://github.com/muvox-io/euphonium/blob/develop/src/core/fs/pkgs/platform_esp32/src/dacs/ma12070p_driver.be

…uired for this DAC).

Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown
Owner

@CarlosDerSeher CarlosDerSeher left a comment

Choose a reason for hiding this comment

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

I'll have to trust you on the actual driver as I don't know the specifics of this DAC. There is only one thing, please see my comment.

Comment thread components/audio_board/Kconfig.projbuild
@CarlosDerSeher CarlosDerSeher merged commit 3e3efe3 into CarlosDerSeher:develop May 6, 2026
2 checks passed
@anabolyc anabolyc deleted the features/ma12070p-driver-support branch May 7, 2026 13:00
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