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.