[PATCH] MXS: Add DENX M28 dts file

Shawn Guo shawn.guo at linaro.org
Wed Jun 6 01:54:00 EDT 2012


On Mon, May 28, 2012 at 12:27:29PM -0300, Fabio Estevam wrote:
> Hi Marek,
> 
> On Sat, May 26, 2012 at 11:12 PM, Marek Vasut <marex at denx.de> wrote:
> 
> > +       sound {
> > +               compatible = "fsl,imx28-evk-sgtl5000",
> > +                            "fsl,mxs-audio-sgtl5000";
> > +               model = "imx28-evk-sgtl5000";
> > +               saif-controllers = <&saif0 &saif1>;
> > +               audio-codec = <&sgtl5000>;
> > +       };
> > +};
> 
> Shouldn't this be:
> 
> sound {
> 	compatible = "fsl,imx28-m28-sgtl5000",
> 		     "fsl,mxs-audio-sgtl5000";
> 	model = "imx28-m28-sgtl5000";
> 	saif-controllers = <&saif0 &saif1>;
> 	audio-codec = <&sgtl5000>;
> };

Yes, this one looks sane to me.

-- 
Regards,
Shawn



More information about the linux-arm-kernel mailing list