Re: [alsa-devel] [PATCH] ASoC drivers for the Freescale MPC8610 SoC

On 1/3/08, Timur Tabi timur@freescale.com wrote:

Grant Likely wrote:

...

Make the compatible property tell you! :-) If it's connected to an I2S codec, then it could be compatible = "fsl,mpc8610-ssi,i2s". Or for AC7, compatible = "fsl,mpc8610-ssi,ac97"

That won't work. There are too many variations. I think a separate property just makes more sense. Frankly, I don't see what's wrong with it.

Sure it will, that's exactly what I'm doing with the 5200, but I won't argue the point. My *opinion* is that using compatible is a more elegant approach for this type of multifunction device, but using a mode property is neither wrong or bad.

g.