On Sun, Apr 07, 2013 at 09:52:17PM -0500, Timur Tabi wrote:
Markus Pargmann wrote:
Notes: Changes in v2: - Simplify the driver, by combining audmux port configurations. The audmux driver actually knows on which platform he is running and will return the appropriate error code if we use functions for another platform. So we don't need to have the knowledge about it in phycore-ac97 and can try both functions. This removes the need of different compatibilities and renames it to imx27-ac97. - Use a phandle for the cpu_dai link. - Add devicetree binding documentation. - Rename binding to phytec,phycore-ac97 and fsl,ssi to phytec,ssi
I think some of this information belongs in the changelog.
Okay
+#ifdef CONFIG_MACH_IMX27_DT
.platform_name = "imx-fiq-pcm-audio",+#else .cpu_dai_name = "imx-ssi.0", .platform_name = "imx-fiq-pcm-audio.0", +#endif
What's the difference between "imx-fiq-pcm-audio" and "imx-fiq-pcm-audio.0"?
The previous version of phycore-ac97 was using devices created by the board init code. They passed a proper pdev id which was used in all drivers they created. Since fsl-ssi is using DT, that id is -1 and is ignored for the device name. So we have to remove the .0 for DT here. This driver is not used multiple times in one system, so it shouldn't be a problem.
Regards,
Markus
-- Timur Tabi