<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 1, 2022 at 5:23 PM Marco Felsch <<a href="mailto:m.felsch@pengutronix.de">m.felsch@pengutronix.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If the imx-pcm-dma is required we need to have the module enabled. For<br>
all NXP/FSL sound cards using the ASoC architecture this is the case but<br>
in case of using the simple-audio-card sound card this isn't the case.<br>
<br>
In such case the driver probe fails silently and the card isn't<br>
available. It took a while to find the missing Kconfig. Make this easier<br>
for others by printing a error if this the module isn't available but<br>
required by the HW.<br>
<br>
Signed-off-by: Marco Felsch <<a href="mailto:m.felsch@pengutronix.de" target="_blank">m.felsch@pengutronix.de</a>><br></blockquote><div><br></div><div>Acked-by: Shengjiu Wang <<a href="mailto:shengjiu.wang@gmail.com">shengjiu.wang@gmail.com</a>></div><div><br></div><div>Best regards</div><div>Wang Shengjiu </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 sound/soc/fsl/fsl_sai.c | 5 ++++-<br>
 1 file changed, 4 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/sound/soc/fsl/fsl_sai.c b/sound/soc/fsl/fsl_sai.c<br>
index 3e54f1f71c1e..2371da814b09 100644<br>
--- a/sound/soc/fsl/fsl_sai.c<br>
+++ b/sound/soc/fsl/fsl_sai.c<br>
@@ -1152,8 +1152,11 @@ static int fsl_sai_probe(struct platform_device *pdev)<br>
         */<br>
        if (sai->soc_data->use_imx_pcm) {<br>
                ret = imx_pcm_dma_init(pdev);<br>
-               if (ret)<br>
+               if (ret) {<br>
+                       if (!IS_ENABLED(CONFIG_SND_SOC_IMX_PCM_DMA))<br>
+                               dev_err(dev, "Error: You must enable the imx-pcm-dma support!\n");<br>
                        goto err_pm_get_sync;<br>
+               }<br>
        } else {<br>
                ret = devm_snd_dmaengine_pcm_register(dev, NULL, 0);<br>
                if (ret)<br>
-- <br>
2.30.2<br>
<br>
</blockquote></div></div>