Re: [PATCH v7 7/7] arm64: dts: imx8mm: imx8mp: Add FIO1-Audio DTOs for Data Modul i.MX8M Mini and Plus eDM SBC

Marek Vasut <[email protected]> Fri, 31 Jul 2026 17:56:19 +0200
Newsgroups dev.linux.lists.imx,dev.linux.lists.sashiko-reviews,org.kernel.vger.linux-devicetree
Message-ID <[email protected]>
On 7/31/26 5:23 PM, Frank Li wrote:

Hello Frank,

>>> at imx8mm.dts, 30050000 is SAI5, here you change to to use clock-controller
>>>
>>> Suppose address here is wrong.
>> This is correct and we already discussed this in V4, the SAI here is not
>> used as an audio controller (not using sound/soc/fsl/fsl_sai.c driver), but
>> as a clock controller (yes using drivers/clk/clk-fsl-sai.c driver), the
>> bindings differ, hence this node.
> 
> Thanks, I forget it. Now, may not allow this method. The same hardware
> have to use one compatible string.
This was already discussed at length when the drivers/clk/clk-fsl-sai.c 
was upstreamed, it is clear the two drivers are not ideal. The rationale 
for having two separate drivers, one for SAI as audio device and one for 
SAI as clock controller, with separate compatible strings, is to make 
sound/soc/fsl/fsl_sai.c maintainable. The sound/soc/fsl/fsl_sai.c is 
already complex enough and folding a clock driver into it would make it 
difficult to maintain, hence the separate clock driver.