From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawn.guo@freescale.com (Shawn Guo) Date: Fri, 4 Jul 2014 22:55:02 +0800 Subject: [PATCH] ARM: dts: imx: correct sdma compatbile for imx6sl and imx6sx In-Reply-To: References: <1404456361-8197-1-git-send-email-shawn.guo@freescale.com> Message-ID: <20140704145500.GJ30239@dragon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Jul 04, 2014 at 10:53:39AM -0300, Fabio Estevam wrote: > On Fri, Jul 4, 2014 at 3:46 AM, Shawn Guo wrote: > > The SDMA on imx6sl and imx6sx is more compatible with imx6q one than > > imx35. Let's use "fsl,imx6q-sdma" instead of "fsl,imx35-sdma", so that > > SDMA ROM script on imx6sl and imx6sx can work for audio driver just like > > the case of imx6q. > > > > Reported-by: Fabio Estevam > > Signed-off-by: Shawn Guo > > --- > > Thanks to Robin for finding this out. > > Very good! Now I can play audio on mx6sl and mx6sx without the need of > loading the external SDMA firmware. > > However, I still have a question: wouldn't it be better to describe > the compatible string as: > compatible = "fsl,imx6sl-sdma", "fsl,imx6q-sdma", "fsl,imx35-sdma"; ? No, it wouldn't be better or necessary. Shawn