Request for help: How to upgrade a sound driver from kernel 4.14 to 6.6
I have an audio driver/module that builds for kernel 4.14, but not for 6.6 due to changes in the kernel API.I find that API types (structs) and functions have been removed or changed, but IDK how to modify my code to use the updated API. Further, some of the changes seem less than clean. For example, in a couple places struct fields have been removed, but comments that used to precede the fields are still there such the comments are clearly no longer relevant and are therefore confusing. I have to wonder how many comments are out-of-date.
My ask is for help fixing my code. I don't expect you to fix directly, but I need to know how to go about figuring out how to fix it. What changed in the API? Is there an upgrade guide/info?
I am going to include more than a little info. I realize that it might be overwhelming. But, I fear that I try to reduce the info too much it will not convey the relevant context.
I'll start with my analysis of my compiler errors that I get. Seems that the struct snd_soc_codec and struct snd_soc_codec_driver were removed. Seems that snd_soc_register_codec() and snd_soc_unregister_codec() were removed. What replaces these things? What do I need to change to accommodate the removal of these?
FYI, here are my compiler errors:
<build-errors> $ make -C ../../.. M=$PWD make: Entering directory '/home/steve/code/compton-kernel' CC [M] /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:110:38: warning: 'struct snd_soc_codec' declared inside parameter list will not be visible outside of this definition or declaration 110 | static int codec_driver_probe(struct snd_soc_codec *codec) | ^~~~~~~~~~~~~ In file included from ./include/linux/device.h:15, from ./include/linux/platform_device.h:13, from ./include/linux/of_device.h:5, from /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:7: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'codec_driver_probe': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:19: error: invalid use of undefined type 'struct snd_soc_codec' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~ ./include/linux/dev_printk.h:110:25: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:5: note: in expansion of macro 'dev_info' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:21: error: variable 'codec_driver' has initializer but incomplete type 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:6: error: 'const struct snd_soc_codec_driver' has no member named 'probe' 117 | .probe = codec_driver_probe, | ^~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:14: warning: excess elements in struct initializer 117 | .probe = codec_driver_probe, | ^~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:14: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:6: error: 'const struct snd_soc_codec_driver' has no member named 'component_driver' 118 | .component_driver = { | ^~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: error: extra brace group at end of initializer 118 | .component_driver = { | ^ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: warning: excess elements in struct initializer /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_probe_': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:153:12: error: implicit declaration of function 'snd_soc_register_codec'; did you mean 'snd_soc_register_dai'? [-Werror=implicit-function-declaration] 153 | stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); | ^~~~~~~~~~~~~~~~~~~~~~ | snd_soc_register_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_remove': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:166:5: error: implicit declaration of function 'snd_soc_unregister_codec'; did you mean 'snd_soc_unregister_dai'? [-Werror=implicit-function-declaration] 166 | snd_soc_unregister_codec(&pdev->dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ | snd_soc_unregister_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:42: error: storage size of 'codec_driver' isn't known 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~ cc1: some warnings being treated as errors make[2]: *** [scripts/Makefile.build:243: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o] Error 1 make[1]: *** [/home/steve/code/compton-kernel/Makefile:1921: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s] Error 2 make: *** [Makefile:234: __sub-make] Error 2 make: Leaving directory '/home/steve/code/compton-kernel' </build-errors>
My code (for 4.14 kernel) --------------------------------- Here's my driver code. The macro INCLUDE_PROBLEMATIC_CODE is used around each chuck that fails to compile. If not defined, the code does compile, but of course is missing important parts.
<my-code> #define INCLUDE_PROBLEMATIC_CODE /* * Audindate Dante Embedded Platform (DEP) I2S Codec (ALSA SoC) driver; Linux module * Supports sample rates, bit depths and TDM modes for Dante devices. */ #include <linux/module.h> #include <linux/of_device.h> #include <linux/clk.h> #include <sound/soc.h> #include <sound/pcm.h> #include <linux/printk.h>
#define DEP_I2S_PCM_RATES (SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 | \ SNDRV_PCM_RATE_88200 | SNDRV_PCM_RATE_96000 | \ SNDRV_PCM_RATE_176400 | SNDRV_PCM_RATE_192000) #define DEP_I2S_PCM_FORMATS ( SNDRV_PCM_FMTBIT_S16_LE | \ SNDRV_PCM_FMTBIT_S24_LE | \ SNDRV_PCM_FMTBIT_S32_LE )
// for future use struct instance_state { unsigned int dummy_holder; };
static const struct snd_soc_dapm_widget widgets[] = { SND_SOC_DAPM_DAC("DAC", "Playback", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_ADC("ADC", "Capture", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_OUTPUT("LINEVOUT"), SND_SOC_DAPM_INPUT("LINEVIN"), };
static const struct snd_soc_dapm_route routes[] = { { "LINEVOUT", NULL, "DAC" }, { "ADC", NULL, "LINEVIN" }, };
static int set_fmt(struct snd_soc_dai *dai, unsigned int fmt) { switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { case SND_SOC_DAIFMT_I2S: // supported break; case SND_SOC_DAIFMT_LEFT_J: // supported break; default: // all other formats unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports left justified and I2S, got 0x%04X", fmt); return -EINVAL; }
switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { // codec as bit master and frame master (or external clock generator) // note that jumpers on DEP-EVK-IMX8 must be set correctly case SND_SOC_DAIFMT_CBM_CFM: // supported break; // codec as bit slave and frame slave (ie clocks driven from SOC) case SND_SOC_DAIFMT_CBS_CFS: // supported break; default: // all others unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports CODEC as bit and frame clock master, got 0x%04X", fmt); return -EINVAL; }
switch (fmt & SND_SOC_DAIFMT_INV_MASK) { default: // support bit and frame inversion break; }
dev_info(dai->dev, "Set fmt to %u", fmt); return 0; }
static struct snd_soc_dai_ops dep_i2s_dai_ops = { .set_fmt = set_fmt, // add mute here };
struct snd_soc_dai_driver dai_driver = { .name = "dep-i2s", .playback = { .stream_name = "Playback", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .capture = { .stream_name = "Capture", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .ops = &dep_i2s_dai_ops, .symmetric_rate = 1, };
#ifdef INCLUDE_PROBLEMATIC_CODE
// Dummy probe static int codec_driver_probe(struct snd_soc_codec *codec) { dev_info(codec->dev, "Entered %s", __func__); return 0; }
static const struct snd_soc_codec_driver codec_driver = { .probe = codec_driver_probe, .component_driver = { .dapm_widgets = widgets, .num_dapm_widgets = ARRAY_SIZE(widgets), .dapm_routes = routes, .num_dapm_routes = ARRAY_SIZE(routes), }, };
#endif
#if defined(CONFIG_OF) static const struct of_device_id match_table[] = { {.compatible = "audinate,dep_i2s"}, {} }; MODULE_DEVICE_TABLE(of, match_table); #endif
static int platform_driver_probe_(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__);
int stat = 0;
struct instance_state *instance_state = devm_kzalloc(&pdev->dev, sizeof(struct instance_state), GFP_KERNEL); if (instance_state == NULL) { return -ENOMEM; }
platform_set_drvdata(pdev, instance_state);
dev_info(&pdev->dev, "Codec probe pt 2");
#ifdef INCLUDE_PROBLEMATIC_CODE // 1 == number of DAIs, might need to be 2 for dual direction, or for 2* sai? stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); if (stat < 0) { dev_err(&pdev->dev, "Failed to register codec: %d", stat); } #endif
return stat; }
static int platform_driver_remove(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__); #ifdef INCLUDE_PROBLEMATIC_CODE snd_soc_unregister_codec(&pdev->dev); #endif return 0; }
static struct platform_driver platform_driver = { .probe = platform_driver_probe_, .remove = platform_driver_remove, .driver = { .name = "dep-i2s-codec", .of_match_table = of_match_ptr(match_table), }, };
module_platform_driver(platform_driver);
MODULE_DESCRIPTION("Audinate DEP generic I2S ASoC codec"); MODULE_AUTHOR("James Stuart opensource@audinate.com"); MODULE_ALIAS("platform:dep-i2s-codec"); MODULE_LICENSE("GPL"); </my-code>
Steve Broshar Palmer Wireless Medtech
On 5/16/25 12:50, Steve Broshar wrote:
I have an audio driver/module that builds for kernel 4.14, but not for 6.6 due to changes in the kernel API.I find that API types (structs) and functions have been removed or changed, but IDK how to modify my code to use the updated API. Further, some of the changes seem less than clean. For example, in a couple places struct fields have been removed, but comments that used to precede the fields are still there such the comments are clearly no longer relevant and are therefore confusing. I have to wonder how many comments are out-of-date.
My ask is for help fixing my code. I don't expect you to fix directly, but I need to know how to go about figuring out how to fix it. What changed in the API? Is there an upgrade guide/info?
I am going to include more than a little info. I realize that it might be overwhelming. But, I fear that I try to reduce the info too much it will not convey the relevant context.
I'll start with my analysis of my compiler errors that I get. Seems that the struct snd_soc_codec and struct snd_soc_codec_driver were removed. Seems that snd_soc_register_codec() and snd_soc_unregister_codec() were removed. What replaces these things? What do I need to change to accommodate the removal of these?
Hi,
snd_soc_codec was replaced with snd_soc_component, most of the API translate 1to1. Here try this, see if it works https://chatgpt.com/share/6828e403-d564-8008-b2a4-42c76475a3a4
FYI, here are my compiler errors:
<build-errors> $ make -C ../../.. M=$PWD make: Entering directory '/home/steve/code/compton-kernel' CC [M] /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:110:38: warning: 'struct snd_soc_codec' declared inside parameter list will not be visible outside of this definition or declaration 110 | static int codec_driver_probe(struct snd_soc_codec *codec) | ^~~~~~~~~~~~~ In file included from ./include/linux/device.h:15, from ./include/linux/platform_device.h:13, from ./include/linux/of_device.h:5, from /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:7: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'codec_driver_probe': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:19: error: invalid use of undefined type 'struct snd_soc_codec' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~ ./include/linux/dev_printk.h:110:25: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:5: note: in expansion of macro 'dev_info' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:21: error: variable 'codec_driver' has initializer but incomplete type 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:6: error: 'const struct snd_soc_codec_driver' has no member named 'probe' 117 | .probe = codec_driver_probe, | ^~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:14: warning: excess elements in struct initializer 117 | .probe = codec_driver_probe, | ^~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:14: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:6: error: 'const struct snd_soc_codec_driver' has no member named 'component_driver' 118 | .component_driver = { | ^~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: error: extra brace group at end of initializer 118 | .component_driver = { | ^ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: warning: excess elements in struct initializer /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_probe_': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:153:12: error: implicit declaration of function 'snd_soc_register_codec'; did you mean 'snd_soc_register_dai'? [-Werror=implicit-function-declaration] 153 | stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); | ^~~~~~~~~~~~~~~~~~~~~~ | snd_soc_register_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_remove': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:166:5: error: implicit declaration of function 'snd_soc_unregister_codec'; did you mean 'snd_soc_unregister_dai'? [-Werror=implicit-function-declaration] 166 | snd_soc_unregister_codec(&pdev->dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ | snd_soc_unregister_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:42: error: storage size of 'codec_driver' isn't known 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~ cc1: some warnings being treated as errors make[2]: *** [scripts/Makefile.build:243: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o] Error 1 make[1]: *** [/home/steve/code/compton-kernel/Makefile:1921: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s] Error 2 make: *** [Makefile:234: __sub-make] Error 2 make: Leaving directory '/home/steve/code/compton-kernel' </build-errors>
My code (for 4.14 kernel)
Here's my driver code. The macro INCLUDE_PROBLEMATIC_CODE is used around each chuck that fails to compile. If not defined, the code does compile, but of course is missing important parts.
<my-code> #define INCLUDE_PROBLEMATIC_CODE /* * Audindate Dante Embedded Platform (DEP) I2S Codec (ALSA SoC) driver; Linux module * Supports sample rates, bit depths and TDM modes for Dante devices. */ #include <linux/module.h> #include <linux/of_device.h> #include <linux/clk.h> #include <sound/soc.h> #include <sound/pcm.h> #include <linux/printk.h>
#define DEP_I2S_PCM_RATES (SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 | \ SNDRV_PCM_RATE_88200 | SNDRV_PCM_RATE_96000 | \ SNDRV_PCM_RATE_176400 | SNDRV_PCM_RATE_192000) #define DEP_I2S_PCM_FORMATS ( SNDRV_PCM_FMTBIT_S16_LE | \ SNDRV_PCM_FMTBIT_S24_LE | \ SNDRV_PCM_FMTBIT_S32_LE )
// for future use struct instance_state { unsigned int dummy_holder; };
static const struct snd_soc_dapm_widget widgets[] = { SND_SOC_DAPM_DAC("DAC", "Playback", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_ADC("ADC", "Capture", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_OUTPUT("LINEVOUT"), SND_SOC_DAPM_INPUT("LINEVIN"), };
static const struct snd_soc_dapm_route routes[] = { { "LINEVOUT", NULL, "DAC" }, { "ADC", NULL, "LINEVIN" }, };
static int set_fmt(struct snd_soc_dai *dai, unsigned int fmt) { switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { case SND_SOC_DAIFMT_I2S: // supported break; case SND_SOC_DAIFMT_LEFT_J: // supported break; default: // all other formats unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports left justified and I2S, got 0x%04X", fmt); return -EINVAL; }
switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { // codec as bit master and frame master (or external clock generator) // note that jumpers on DEP-EVK-IMX8 must be set correctly case SND_SOC_DAIFMT_CBM_CFM: // supported break; // codec as bit slave and frame slave (ie clocks driven from SOC) case SND_SOC_DAIFMT_CBS_CFS: // supported break; default: // all others unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports CODEC as bit and frame clock master, got 0x%04X", fmt); return -EINVAL; } switch (fmt & SND_SOC_DAIFMT_INV_MASK) { default: // support bit and frame inversion break; } dev_info(dai->dev, "Set fmt to %u", fmt); return 0;}
static struct snd_soc_dai_ops dep_i2s_dai_ops = { .set_fmt = set_fmt, // add mute here };
struct snd_soc_dai_driver dai_driver = { .name = "dep-i2s", .playback = { .stream_name = "Playback", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .capture = { .stream_name = "Capture", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .ops = &dep_i2s_dai_ops, .symmetric_rate = 1, };
#ifdef INCLUDE_PROBLEMATIC_CODE
// Dummy probe static int codec_driver_probe(struct snd_soc_codec *codec) { dev_info(codec->dev, "Entered %s", __func__); return 0; }
static const struct snd_soc_codec_driver codec_driver = { .probe = codec_driver_probe, .component_driver = { .dapm_widgets = widgets, .num_dapm_widgets = ARRAY_SIZE(widgets), .dapm_routes = routes, .num_dapm_routes = ARRAY_SIZE(routes), }, };
#endif
#if defined(CONFIG_OF) static const struct of_device_id match_table[] = { {.compatible = "audinate,dep_i2s"}, {} }; MODULE_DEVICE_TABLE(of, match_table); #endif
static int platform_driver_probe_(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__);
int stat = 0; struct instance_state *instance_state = devm_kzalloc(&pdev->dev, sizeof(struct instance_state), GFP_KERNEL); if (instance_state == NULL) { return -ENOMEM; } platform_set_drvdata(pdev, instance_state); dev_info(&pdev->dev, "Codec probe pt 2");#ifdef INCLUDE_PROBLEMATIC_CODE // 1 == number of DAIs, might need to be 2 for dual direction, or for 2* sai? stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); if (stat < 0) { dev_err(&pdev->dev, "Failed to register codec: %d", stat); } #endif
return stat;}
static int platform_driver_remove(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__); #ifdef INCLUDE_PROBLEMATIC_CODE snd_soc_unregister_codec(&pdev->dev); #endif return 0; }
static struct platform_driver platform_driver = { .probe = platform_driver_probe_, .remove = platform_driver_remove, .driver = { .name = "dep-i2s-codec", .of_match_table = of_match_ptr(match_table), }, };
module_platform_driver(platform_driver);
MODULE_DESCRIPTION("Audinate DEP generic I2S ASoC codec"); MODULE_AUTHOR("James Stuart opensource@audinate.com"); MODULE_ALIAS("platform:dep-i2s-codec"); MODULE_LICENSE("GPL");
</my-code>
Steve Broshar Palmer Wireless Medtech
I learned that AI can be useful. How about that. Thanks.
-----Original Message----- From: Lars-Peter Clausen lars@metafoo.de Sent: Saturday, May 17, 2025 2:33 PM To: Steve Broshar steve@palmerwirelessmedtech.com; alsa-devel@alsa-project.org Subject: Re: Request for help: How to upgrade a sound driver from kernel 4.14 to 6.6
On 5/16/25 12:50, Steve Broshar wrote:
I have an audio driver/module that builds for kernel 4.14, but not for 6.6 due to changes in the kernel API.I find that API types (structs) and functions have been removed or changed, but IDK how to modify my code to use the updated API. Further, some of the changes seem less than clean. For example, in a couple places struct fields have been removed, but comments that used to precede the fields are still there such the comments are clearly no longer relevant and are therefore confusing. I have to wonder how many comments are out-of-date.
My ask is for help fixing my code. I don't expect you to fix directly, but I need to know how to go about figuring out how to fix it. What changed in the API? Is there an upgrade guide/info?
I am going to include more than a little info. I realize that it might be overwhelming. But, I fear that I try to reduce the info too much it will not convey the relevant context.
I'll start with my analysis of my compiler errors that I get. Seems that the struct snd_soc_codec and struct snd_soc_codec_driver were removed. Seems that snd_soc_register_codec() and snd_soc_unregister_codec() were removed. What replaces these things? What do I need to change to accommodate the removal of these?
Hi,
snd_soc_codec was replaced with snd_soc_component, most of the API translate 1to1. Here try this, see if it works https://chatgpt.com/share/6828e403-d564-8008-b2a4-42c76475a3a4
FYI, here are my compiler errors:
<build-errors> $ make -C ../../.. M=$PWD make: Entering directory '/home/steve/code/compton-kernel' CC [M] /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:110:38: warning: 'struct snd_soc_codec' declared inside parameter list will not be visible outside of this definition or declaration 110 | static int codec_driver_probe(struct snd_soc_codec *codec) | ^~~~~~~~~~~~~ In file included from ./include/linux/device.h:15, from ./include/linux/platform_device.h:13, from ./include/linux/of_device.h:5, from /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:7: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'codec_driver_probe': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:19: error: invalid use of undefined type 'struct snd_soc_codec' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~ ./include/linux/dev_printk.h:110:25: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:112:5: note: in expansion of macro 'dev_info' 112 | dev_info(codec->dev, "Entered %s", __func__); | ^~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:21: error: variable 'codec_driver' has initializer but incomplete type 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:6: error: 'const struct snd_soc_codec_driver' has no member named 'probe' 117 | .probe = codec_driver_probe, | ^~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:117:14: warning: excess elements in struct initializer 117 | .probe = codec_driver_probe, | ^~~~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c :117:14: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:6: error: 'const struct snd_soc_codec_driver' has no member named 'component_driver' 118 | .component_driver = { | ^~~~~~~~~~~~~~~~ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:118:25: error: extra brace group at end of initializer 118 | .component_driver = { | ^ /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c :118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c :118:25: warning: excess elements in struct initializer /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c :118:25: note: (near initialization for 'codec_driver') /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_probe_': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:153:12: error: implicit declaration of function 'snd_soc_register_codec'; did you mean 'snd_soc_register_dai'? [-Werror=implicit-function-declaration] 153 | stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); | ^~~~~~~~~~~~~~~~~~~~~~ | snd_soc_register_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: In function 'platform_driver_remove': /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:166:5: error: implicit declaration of function 'snd_soc_unregister_codec'; did you mean 'snd_soc_unregister_dai'? [-Werror=implicit-function-declaration] 166 | snd_soc_unregister_codec(&pdev->dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ | snd_soc_unregister_dai /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c: At top level: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.c:116:42: error: storage size of 'codec_driver' isn't known 116 | static const struct snd_soc_codec_driver codec_driver = { | ^~~~~~~~~~~~ cc1: some warnings being treated as errors make[2]: *** [scripts/Makefile.build:243: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s/dep_i2s.o ] Error 1 make[1]: *** [/home/steve/code/compton-kernel/Makefile:1921: /home/steve/code/compton-kernel/drivers/compton/dep-imx8-i2s] Error 2 make: *** [Makefile:234: __sub-make] Error 2 make: Leaving directory '/home/steve/code/compton-kernel' </build-errors>
My code (for 4.14 kernel)
Here's my driver code. The macro INCLUDE_PROBLEMATIC_CODE is used around each chuck that fails to compile. If not defined, the code does compile, but of course is missing important parts.
<my-code> #define INCLUDE_PROBLEMATIC_CODE /* * Audindate Dante Embedded Platform (DEP) I2S Codec (ALSA SoC) driver; Linux module * Supports sample rates, bit depths and TDM modes for Dante devices. */ #include <linux/module.h> #include <linux/of_device.h> #include <linux/clk.h> #include <sound/soc.h> #include <sound/pcm.h> #include <linux/printk.h>
#define DEP_I2S_PCM_RATES (SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 | \ SNDRV_PCM_RATE_88200 | SNDRV_PCM_RATE_96000 | \ SNDRV_PCM_RATE_176400 | SNDRV_PCM_RATE_192000) #define DEP_I2S_PCM_FORMATS ( SNDRV_PCM_FMTBIT_S16_LE | \ SNDRV_PCM_FMTBIT_S24_LE | \ SNDRV_PCM_FMTBIT_S32_LE )
// for future use struct instance_state { unsigned int dummy_holder; };
static const struct snd_soc_dapm_widget widgets[] = { SND_SOC_DAPM_DAC("DAC", "Playback", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_ADC("ADC", "Capture", SND_SOC_NOPM, 0, 0), SND_SOC_DAPM_OUTPUT("LINEVOUT"), SND_SOC_DAPM_INPUT("LINEVIN"), };
static const struct snd_soc_dapm_route routes[] = { { "LINEVOUT", NULL, "DAC" }, { "ADC", NULL, "LINEVIN" }, };
static int set_fmt(struct snd_soc_dai *dai, unsigned int fmt) { switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { case SND_SOC_DAIFMT_I2S: // supported break; case SND_SOC_DAIFMT_LEFT_J: // supported break; default: // all other formats unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports left justified and I2S, got 0x%04X", fmt); return -EINVAL; }
switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { // codec as bit master and frame master (or external clock generator) // note that jumpers on DEP-EVK-IMX8 must be set correctly case SND_SOC_DAIFMT_CBM_CFM: // supported break; // codec as bit slave and frame slave (ie clocks driven from SOC) case SND_SOC_DAIFMT_CBS_CFS: // supported break; default: // all others unsupported dev_err(dai->dev, "Invalid DAI format: DEP I2S only supports CODEC as bit and frame clock master, got 0x%04X", fmt); return -EINVAL; } switch (fmt & SND_SOC_DAIFMT_INV_MASK) { default: // support bit and frame inversion break; } dev_info(dai->dev, "Set fmt to %u", fmt); return 0;}
static struct snd_soc_dai_ops dep_i2s_dai_ops = { .set_fmt = set_fmt, // add mute here };
struct snd_soc_dai_driver dai_driver = { .name = "dep-i2s", .playback = { .stream_name = "Playback", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .capture = { .stream_name = "Capture", .channels_min = 1, .channels_max = 16, .rate_min = 44100, .rate_max = 192000, .rates = DEP_I2S_PCM_RATES, .formats = DEP_I2S_PCM_FORMATS, }, .ops = &dep_i2s_dai_ops, .symmetric_rate = 1, };
#ifdef INCLUDE_PROBLEMATIC_CODE
// Dummy probe static int codec_driver_probe(struct snd_soc_codec *codec) { dev_info(codec->dev, "Entered %s", __func__); return 0; }
static const struct snd_soc_codec_driver codec_driver = { .probe = codec_driver_probe, .component_driver = { .dapm_widgets = widgets, .num_dapm_widgets = ARRAY_SIZE(widgets), .dapm_routes = routes, .num_dapm_routes = ARRAY_SIZE(routes), }, };
#endif
#if defined(CONFIG_OF) static const struct of_device_id match_table[] = { {.compatible = "audinate,dep_i2s"}, {} }; MODULE_DEVICE_TABLE(of, match_table); #endif
static int platform_driver_probe_(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__);
int stat = 0; struct instance_state *instance_state = devm_kzalloc(&pdev->dev, sizeof(struct instance_state), GFP_KERNEL); if (instance_state == NULL) { return -ENOMEM; } platform_set_drvdata(pdev, instance_state); dev_info(&pdev->dev, "Codec probe pt 2");#ifdef INCLUDE_PROBLEMATIC_CODE // 1 == number of DAIs, might need to be 2 for dual direction, or for 2* sai? stat = snd_soc_register_codec(&pdev->dev, &codec_driver, &dai_driver, 1); if (stat < 0) { dev_err(&pdev->dev, "Failed to register codec: %d", stat); } #endif
return stat;}
static int platform_driver_remove(struct platform_device *pdev) { dev_info(&pdev->dev, "Entered %s", __func__); #ifdef INCLUDE_PROBLEMATIC_CODE snd_soc_unregister_codec(&pdev->dev); #endif return 0; }
static struct platform_driver platform_driver = { .probe = platform_driver_probe_, .remove = platform_driver_remove, .driver = { .name = "dep-i2s-codec", .of_match_table = of_match_ptr(match_table), }, };
module_platform_driver(platform_driver);
MODULE_DESCRIPTION("Audinate DEP generic I2S ASoC codec"); MODULE_AUTHOR("James Stuart opensource@audinate.com"); MODULE_ALIAS("platform:dep-i2s-codec"); MODULE_LICENSE("GPL");
</my-code>
Steve Broshar Palmer Wireless Medtech
Thanks for the help. I learned that to output I2S to a slave device, I shouldn't need a device-specific driver; that built-in drivers are all I need. I thought that was going to be easy to setup, but I have no luck with it. The boot log contains: platform sound-auditon: deferred probe pending
I have been working with chatgpt extensively as it seems you do too. It has a deep understanding of the linux kernel codebase. But, so far I cannot get the device to initialize (probe). Do you agree that I should be able to avoid the use of a device-specific driver? Do you see any issues in the device tree below?
Here's the device tree setup:
&sai2 { #sound-dai-cells = <0>; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_sai2>; assigned-clocks = <&clk IMX8MN_CLK_SAI2>; assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>; assigned-clock-rates = <24576000>; fsl,sai-mclk-direction-output; status = "okay"; };
pinctrl_sai2: sai2grp { fsl,pins = < MX8MN_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 MX8MN_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 MX8MN_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 //MX8MN_IOMUXC_SAI2_RXD0_SAI2_RX_DATA0 0xd6 MX8MN_IOMUXC_SAI2_MCLK_SAI2_MCLK 0xd6 >; };
// Compton audio output does not require a device-specific codec; the built-in, dummy driver is sufficient codec_auditon: codec-snd-doc-dummy { compatible = "linux,snd-soc-dummy"; #sound-dai-cells = <0>; };
// audio output for Compton can be accomplished via the built-in simple-audio-card driver // since only need to stream I2S; device (ADAU1467) does not require setup/control sound-auditon { // specify the built-in simple sound card driver compatible = "simple-audio-card"; // description that shows in userland; i.e. via 'aplay -l' // auditon: a quantum of sound ;) simple-audio-card,name = "auditon"; // specify stream format // supported options: i2s, right_j, left_j, dsp_a, dsp_b, ac97, pdm, msb, lsb //simple-audio-card,format = "i2s"; // with i2s, boot log shows: deferred probe pending and no device is setup simple-audio-card,format = "dsp_a"; // assign CPU as master for bit-clock simple-audio-card,bitclock-master = <&cpu_link_auditon>; // assign CPU as master for frame-clock simple-audio-card,frame-master = <&cpu_link_auditon>;
status = "okay";
cpu_link_auditon: simple-audio-card,cpu { sound-dai = <&sai2>; dai-tdm-slot-num = <2>; // 2=>stereo dai-tdm-slot-width = <32>; // bits per slot };
simple-audio-card,codec { sound-dai = <&codec_auditon>; }; };
Thanks in advance.
Steve Broshar Palmer Wireless Medtech
Hi,
This seems to be one of the cases where chatgpt doesn't get it quite right and just makes stuff up.
Generally you still need a simple driver for you codec that at least specifies which formats and rates are supported. But this can be a really simple driver, doesn't have to do anything other than registering the codec.
On 5/30/25 14:47, Steve Broshar wrote:
Thanks for the help. I learned that to output I2S to a slave device, I shouldn't need a device-specific driver; that built-in drivers are all I need. I thought that was going to be easy to setup, but I have no luck with it. The boot log contains: platform sound-auditon: deferred probe pending
I have been working with chatgpt extensively as it seems you do too. It has a deep understanding of the linux kernel codebase. But, so far I cannot get the device to initialize (probe). Do you agree that I should be able to avoid the use of a device-specific driver? Do you see any issues in the device tree below?
Here's the device tree setup:
&sai2 { #sound-dai-cells = <0>; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_sai2>; assigned-clocks = <&clk IMX8MN_CLK_SAI2>; assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>; assigned-clock-rates = <24576000>; fsl,sai-mclk-direction-output; status = "okay"; };
pinctrl_sai2: sai2grp { fsl,pins = < MX8MN_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 MX8MN_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 MX8MN_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 //MX8MN_IOMUXC_SAI2_RXD0_SAI2_RX_DATA0 0xd6 MX8MN_IOMUXC_SAI2_MCLK_SAI2_MCLK 0xd6 >; };
// Compton audio output does not require a device-specific codec; the built-in, dummy driver is sufficient codec_auditon: codec-snd-doc-dummy { compatible = "linux,snd-soc-dummy"; #sound-dai-cells = <0>; };
// audio output for Compton can be accomplished via the built-in simple-audio-card driver // since only need to stream I2S; device (ADAU1467) does not require setup/control sound-auditon { // specify the built-in simple sound card driver compatible = "simple-audio-card"; // description that shows in userland; i.e. via 'aplay -l' // auditon: a quantum of sound ;) simple-audio-card,name = "auditon"; // specify stream format // supported options: i2s, right_j, left_j, dsp_a, dsp_b, ac97, pdm, msb, lsb //simple-audio-card,format = "i2s"; // with i2s, boot log shows: deferred probe pending and no device is setup simple-audio-card,format = "dsp_a"; // assign CPU as master for bit-clock simple-audio-card,bitclock-master = <&cpu_link_auditon>; // assign CPU as master for frame-clock simple-audio-card,frame-master = <&cpu_link_auditon>;
status = "okay"; cpu_link_auditon: simple-audio-card,cpu { sound-dai = <&sai2>; dai-tdm-slot-num = <2>; // 2=>stereo dai-tdm-slot-width = <32>; // bits per slot }; simple-audio-card,codec { sound-dai = <&codec_auditon>; };};
Thanks in advance.
Steve Broshar Palmer Wireless Medtech
Thanks for the direction.
Can you suggest a resource for writing such a driver? I cannot find anything that describes what a driver would do exactly. Is there a template or example that you can suggest that I use as a starting point?
-steve
-----Original Message----- From: Lars-Peter Clausen lars@metafoo.de Sent: Saturday, May 31, 2025 11:33 PM To: Steve Broshar steve@palmerwirelessmedtech.com; alsa-devel@alsa-project.org Subject: Re: Request for help: How to upgrade a sound driver from kernel 4.14 to 6.6
Hi,
This seems to be one of the cases where chatgpt doesn't get it quite right and just makes stuff up.
Generally you still need a simple driver for you codec that at least specifies which formats and rates are supported. But this can be a really simple driver, doesn't have to do anything other than registering the codec.
On 5/30/25 14:47, Steve Broshar wrote:
Thanks for the help. I learned that to output I2S to a slave device, I shouldn't need a device-specific driver; that built-in drivers are all I need. I thought that was going to be easy to setup, but I have no luck with it. The boot log contains: platform sound-auditon: deferred probe pending
I have been working with chatgpt extensively as it seems you do too. It has a deep understanding of the linux kernel codebase. But, so far I cannot get the device to initialize (probe). Do you agree that I should be able to avoid the use of a device-specific driver? Do you see any issues in the device tree below?
Here's the device tree setup:
&sai2 { #sound-dai-cells = <0>; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_sai2>; assigned-clocks = <&clk IMX8MN_CLK_SAI2>; assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>; assigned-clock-rates = <24576000>; fsl,sai-mclk-direction-output; status = "okay"; };
pinctrl_sai2: sai2grp { fsl,pins = < MX8MN_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 MX8MN_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 MX8MN_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 //MX8MN_IOMUXC_SAI2_RXD0_SAI2_RX_DATA0 0xd6 MX8MN_IOMUXC_SAI2_MCLK_SAI2_MCLK 0xd6 >; };
// Compton audio output does not require a device-specific codec; the built-in, dummy driver is sufficient codec_auditon: codec-snd-doc-dummy { compatible = "linux,snd-soc-dummy"; #sound-dai-cells = <0>; };
// audio output for Compton can be accomplished via the built-in simple-audio-card driver // since only need to stream I2S; device (ADAU1467) does not require setup/control sound-auditon { // specify the built-in simple sound card driver compatible = "simple-audio-card"; // description that shows in userland; i.e. via 'aplay -l' // auditon: a quantum of sound ;) simple-audio-card,name = "auditon"; // specify stream format // supported options: i2s, right_j, left_j, dsp_a, dsp_b, ac97, pdm, msb, lsb //simple-audio-card,format = "i2s"; // with i2s, boot log shows: deferred probe pending and no device is setup simple-audio-card,format = "dsp_a"; // assign CPU as master for bit-clock simple-audio-card,bitclock-master = <&cpu_link_auditon>; // assign CPU as master for frame-clock simple-audio-card,frame-master = <&cpu_link_auditon>;
status = "okay"; cpu_link_auditon: simple-audio-card,cpu { sound-dai = <&sai2>; dai-tdm-slot-num = <2>; // 2=>stereo dai-tdm-slot-width = <32>; // bits per slot }; simple-audio-card,codec { sound-dai = <&codec_auditon>; };};
Thanks in advance.
Steve Broshar Palmer Wireless Medtech
On 10 June 2025 15:59:57 BST, Steve Broshar steve@palmerwirelessmedtech.com wrote:
Thanks for the direction.
Can you suggest a resource for writing such a driver? I cannot find anything that describes what a driver would do exactly. Is there a template or example that you can suggest that I use as a starting point?
-steve
-----Original Message----- From: Lars-Peter Clausen lars@metafoo.de Sent: Saturday, May 31, 2025 11:33 PM To: Steve Broshar steve@palmerwirelessmedtech.com; alsa-devel@alsa-project.org Subject: Re: Request for help: How to upgrade a sound driver from kernel 4.14 to 6.6
Hi,
This seems to be one of the cases where chatgpt doesn't get it quite right and just makes stuff up.
Generally you still need a simple driver for you codec that at least specifies which formats and rates are supported. But this can be a really simple driver, doesn't have to do anything other than registering the codec.
On 5/30/25 14:47, Steve Broshar wrote:
Thanks for the help. I learned that to output I2S to a slave device, I shouldn't need a device-specific driver; that built-in drivers are all I need. I thought that was going to be easy to setup, but I have no luck with it. The boot log contains: platform sound-auditon: deferred probe pending
I have been working with chatgpt extensively as it seems you do too. It has a deep understanding of the linux kernel codebase. But, so far I cannot get the device to initialize (probe). Do you agree that I should be able to avoid the use of a device-specific driver? Do you see any issues in the device tree below?
Here's the device tree setup:
&sai2 { #sound-dai-cells = <0>; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_sai2>; assigned-clocks = <&clk IMX8MN_CLK_SAI2>; assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>; assigned-clock-rates = <24576000>; fsl,sai-mclk-direction-output; status = "okay"; };
pinctrl_sai2: sai2grp { fsl,pins = < MX8MN_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 MX8MN_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 MX8MN_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 //MX8MN_IOMUXC_SAI2_RXD0_SAI2_RX_DATA0 0xd6 MX8MN_IOMUXC_SAI2_MCLK_SAI2_MCLK 0xd6 >; };
// Compton audio output does not require a device-specific codec; the built-in, dummy driver is sufficient codec_auditon: codec-snd-doc-dummy { compatible = "linux,snd-soc-dummy"; #sound-dai-cells = <0>; };
// audio output for Compton can be accomplished via the built-in simple-audio-card driver // since only need to stream I2S; device (ADAU1467) does not require setup/control sound-auditon { // specify the built-in simple sound card driver compatible = "simple-audio-card"; // description that shows in userland; i.e. via 'aplay -l' // auditon: a quantum of sound ;) simple-audio-card,name = "auditon"; // specify stream format // supported options: i2s, right_j, left_j, dsp_a, dsp_b, ac97, pdm, msb, lsb //simple-audio-card,format = "i2s"; // with i2s, boot log shows: deferred probe pending and no device is setup simple-audio-card,format = "dsp_a"; // assign CPU as master for bit-clock simple-audio-card,bitclock-master = <&cpu_link_auditon>; // assign CPU as master for frame-clock simple-audio-card,frame-master = <&cpu_link_auditon>;
status = "okay"; cpu_link_auditon: simple-audio-card,cpu { sound-dai = <&sai2>; dai-tdm-slot-num = <2>; // 2=>stereo dai-tdm-slot-width = <32>; // bits per slot }; simple-audio-card,codec { sound-dai = <&codec_auditon>; };};
Thanks in advance.
Steve Broshar Palmer Wireless Medtech
What SoC does the device have?
Olivia
participants (3)
-
Lars-Peter Clausen -
Olivia Mackintosh -
Steve Broshar