[PATCH 4/5] arch/powerpc/boot/dts pcm030 add mpc5200-soc-audio node

Mark Brown broonie at opensource.wolfsonmicro.com
Wed Sep 12 13:17:20 EST 2012


On Tue, Sep 11, 2012 at 10:14:48PM -0400, Eric Millbrandt wrote:

> +		analog at 0 {
> +			stream-name = "AC97 Analog";
> +			codec-name = "wm9712-codec.0";

This name is fairly clearly an internal implementation detail of how
Linux does audio drivers, we shouldn't be using it in device tree
bindings.  We should instead be doing something like referencing a
device tree node for the CODEC.  Look at how drivers like the nVidia
Tegra ones handle this, though for AC'97 we should really be able to
figure out a standard hookup like this automatically at runtime, it's
all enumerable.


More information about the Linuxppc-dev mailing list