Skip to content

Enable i2s_codec sample on mcxn5xx_evk #96954

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.

Sign up for GitHub

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
+195 −28
Open

Enable i2s_codec sample on mcxn5xx_evk #96954

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3919bdb
soc: mcxnx4x: Instantiate MICFIL
VitekST Aug 28, 2025
542bfb4
board: mcx_n5xx: Enable MICFIL and da7212 codec on mcx_n5xx
TomasBarakNXP Oct 3, 2025
89451d2
board: mcx_nx4x: format board.c
TomasBarakNXP Oct 3, 2025
2c9d813
samples: i2s_codec: enable i2s_codec sample for mcx_n5xx_evk
TomasBarakNXP Oct 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions samples/drivers/i2s/i2s_codec/boards/mcx_n5xx_evk_mcxn547_cpu0.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Copyright 2025 NXP
#
# SPDX-License-Identifier: Apache-2.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Commit message body is not giving us any extra info. We should try to use more of a natural language description of what the change is about focusing mostly on why the change is needed.

Also mention in the commit message things that are out of the ordinary which people reading the git log history might want to be aware about.

Commit title is nice and clear mostly saying what we need to now.

So, the commit message should look something like this:

samples: i2s_codec: enable i2s_codec sample for mcx_n5xx_evk

In order to test <insert your scenario here> on mcx_n5xx_evk we need to
configure dts nodes for SAI1 interface and MICFIL channels and
also enable some config options for DMA, I2S and DMIC.

Note that we are using newly introduced I2s params (bytes_per_sample, sample_widht, extra_blocks).

Signed-off-by: Tomas Barak [email protected]


CONFIG_DMA_TCD_QUEUE_SIZE=4
CONFIG_AUDIO_CODEC_DA7212=y
CONFIG_SAMPLE_FREQ=16000
CONFIG_I2S_INIT_BUFFERS=1
CONFIG_USE_CODEC_CLOCK=y
CONFIG_USE_DMIC=y
CONFIG_DMIC_CHANNELS=2
CONFIG_EXTRA_BLOCKS=10
CONFIG_SAMPLE_WIDTH=32
CONFIG_BYTES_PER_SAMPLE=4
26 changes: 26 additions & 0 deletions samples/drivers/i2s/i2s_codec/boards/mcx_n5xx_evk_mcxn547_cpu0.overlay
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
/*
* Copyright 2025 NXP
*
* SPDX-License-Identifier: Apache-2.0
*/

/ {
aliases {
i2s-codec-tx = &sai1;
i2s-tx = &sai1;
};
};

&sai1 {
mclk-output;
};

dmic_dev: &micfil {
channel0: micfil-channel@0 {
status = "okay";
};

channel1: micfil-channel@1 {
status = "okay";
};
};
1 change: 1 addition & 0 deletions samples/drivers/i2s/i2s_codec/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ tests:
- mimxrt1060_evk/mimxrt1062/qspi
- mimxrt1180_evk/mimxrt1189/cm33
- mimxrt1180_evk/mimxrt1189/cm7
- mcx_n5xx_evk/mcxn547/cpu0
harness: console
harness_config:
type: one_line
Expand Down
Loading