<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 19, 2022 at 9:23 PM Fabio Estevam <<a href="mailto:festevam@gmail.com">festevam@gmail.com</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">Hi Shengjiu,<br>
<br>
On Thu, May 19, 2022 at 9:49 AM Shengjiu Wang <<a href="mailto:shengjiu.wang@nxp.com" target="_blank">shengjiu.wang@nxp.com</a>> wrote:<br>
<br>
> diff --git a/sound/soc/fsl/fsl_sai.c b/sound/soc/fsl/fsl_sai.c<br>
> index fa950dde5310..dae16a14f177 100644<br>
> --- a/sound/soc/fsl/fsl_sai.c<br>
> +++ b/sound/soc/fsl/fsl_sai.c<br>
> @@ -437,6 +437,12 @@ static int fsl_sai_set_bclk(struct snd_soc_dai *dai, bool tx, u32 freq)<br>
>                                    FSL_SAI_CR2_DIV_MASK | FSL_SAI_CR2_BYP,<br>
>                                    savediv / 2 - 1);<br>
><br>
> +       if (sai->soc_data->max_register >= FSL_SAI_MCTL) {<br>
<br>
Isn't it a bit fragile to take this decision based on the number of<br>
SAI registers in the SoC?<br>
<br>
What about adding a specific field in soc_data for such a purpose?<br></blockquote><div><br></div><div>'max_register' is one field in the soc_data,  until now in our internal</div><div>usage it seems ok for using this condition.</div><div><br></div><div>Thanks.</div><div><br></div><div>Best regards</div><div>Wang shengjiu</div><div> </div></div></div>