On Tue, Nov 08, 2016 at 03:57:48PM +0800, Chen-Yu Tsai wrote:
On Tue, Nov 8, 2016 at 3:44 PM, Maxime Ripard maxime.ripard@free-electrons.com wrote:
On Mon, Nov 07, 2016 at 10:11:45PM +0800, Chen-Yu Tsai wrote:
On Mon, Nov 7, 2016 at 9:08 PM, Maxime Ripard maxime.ripard@free-electrons.com wrote:
The GR8-EVB comes with a wm8978 codec connected to the i2s bus.
Add a card in order to have it working
Signed-off-by: Maxime Ripard maxime.ripard@free-electrons.com
arch/arm/boot/dts/ntc-gr8-evb.dts | 14 ++++++++++++++ 1 file changed, 14 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/ntc-gr8-evb.dts b/arch/arm/boot/dts/ntc-gr8-evb.dts index 12b4317a4383..5291e425caf9 100644 --- a/arch/arm/boot/dts/ntc-gr8-evb.dts +++ b/arch/arm/boot/dts/ntc-gr8-evb.dts @@ -76,6 +76,20 @@ default-brightness-level = <8>; };
i2s {
"sound" might be a better node name? The I2S controllers are also named "i2s".
I know, but we also had the codec and SPDIF on this board, so sound was too generic to be meaningful I guess. I don't really care about the name though, if you have any suggestion...
Well people seem to use "sound" for the sound card nodes...
What about "sound-analog" for this one, and "sound-spdif" for the SPDIF simple card? Or "analog-sound" and "spdif-sound" if that looks better.
sound-analog works for me. I'll either fix it in the v2, or while applying.
Thanks! Maxime