<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 28, 2022 at 7:00 AM Rob Herring <<a href="mailto:robh@kernel.org">robh@kernel.org</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">On Fri, Jun 17, 2022 at 03:44:36PM +0800, Shengjiu Wang wrote:<br>
> "fsl,dataline" is added to configure the dataline of SAI.<br>
> It has 3 value for each configuration, first one means the type:<br>
> I2S(1) or PDM(2), second one is dataline mask for 'rx', third one is<br>
> dataline mask for 'tx'. for example:<br>
> <br>
> fsl,dataline = <1 0xff 0xff 2 0xff 0x11>,<br>
> <br>
> it means I2S type rx mask is 0xff, tx mask is 0xff, PDM type<br>
> rx mask is 0xff, tx mask is 0x11 (dataline 1 and 4 enabled).<br>
> <br>
> Signed-off-by: Shengjiu Wang <<a href="mailto:shengjiu.wang@nxp.com" target="_blank">shengjiu.wang@nxp.com</a>><br>
> ---<br>
>  Documentation/devicetree/bindings/sound/fsl-sai.txt | 8 ++++++++<br>
>  1 file changed, 8 insertions(+)<br>
> <br>
> diff --git a/Documentation/devicetree/bindings/sound/fsl-sai.txt b/Documentation/devicetree/bindings/sound/fsl-sai.txt<br>
> index c71c5861d787..4c66e6a1a533 100644<br>
> --- a/Documentation/devicetree/bindings/sound/fsl-sai.txt<br>
> +++ b/Documentation/devicetree/bindings/sound/fsl-sai.txt<br>
> @@ -49,6 +49,14 @@ Required properties:<br>
>                         receive data by following their own bit clocks and<br>
>                         frame sync clocks separately.<br>
>  <br>
> +  - fsl,dataline        : configure the dataline. it has 3 value for each configuration<br>
> +                          first one means the type: I2S(1) or PDM(2)<br>
> +                          second one is dataline mask for 'rx'<br>
> +                          third one is dataline mask for 'tx'.<br>
> +                          for example: fsl,dataline = <1 0xff 0xff 2 0xff 0x11>;<br>
> +                          it means I2S type rx mask is 0xff, tx mask is 0xff, PDM type<br>
> +                          rx mask is 0xff, tx mask is 0x11 (dataline 1 and 4 enabled).<br>
<br>
You mean 0 and 4 enabled? Or 1 and 4?<br></blockquote><div><br></div><div>Should be 'dataline 1 and 5 enabled, index start from 1'.  I will fix it. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
How many 3 cell entries can you have?<br></blockquote><div> <br></div><div>There is no limitation for entries,  the code in the driver will query the length first</div><div>by of_property_count_u32_elems() then read the values.</div><div><br></div><div>best regards</div><div>wang shengjiu</div></div></div>