Re: [PATCH] ASoC: simple-card: add support for aux devices

[Kuninori Morimoto]

Hi Nikita

> Example usage:
> 
> 	codec: tlv320dac3100@18 {
> 		compatible = "ti,tlv320dac3100";
> 		...
> 	}
> 
> 	amp: tpa6130a2@60 {
> 		compatible = "ti,tpa6130a2";
> 		...
> 	}
> 
> 	sound {
> 		compatible = "simple-audio-card";
> 		...
> 		simple-audio-card,widgets =
> 			"Headphone", "Headphone Jack";
> 		simple-audio-card,routing =
> 			"Headphone Jack", "HPLEFT",
> 			"Headphone Jack", "HPRIGHT",
> 			"LEFTIN", "HPL",
> 			"RIGHTIN", "HPR";
> 		simple-audio-card,aux-devs = <&amp>;
> 		simple-audio-card,cpu {
> 			sound-dai = <&ssi2>;
> 		};
> 		simple-audio-card,codec {
> 			sound-dai = <&codec1>;
> 			clocks = ...
> 		};

This case, I think you want ...

	simple-audio-card,codec {
-		sound-dai = <&codec1>;
+		sound-dai = <&codec>;


> diff --git a/Documentation/devicetree/bindings/sound/simple-card.txt b/Documentation/devicetree/bindings/sound/simple-card.txt
> index 59d8628..5579f40 100644
> --- a/Documentation/devicetree/bindings/sound/simple-card.txt
> +++ b/Documentation/devicetree/bindings/sound/simple-card.txt
> @@ -22,6 +22,8 @@ Optional properties:
>  					  headphones are attached.
>  - simple-audio-card,mic-det-gpio	: Reference to GPIO that signals when
>  					  a microphone is attached.
> +- simple-audio-card,aux-devs		: List of phandles pointing to auxiliary devices, such
> +					  as amplifiers, to be added to the sound card.
>  
>  Optional subnodes:

I think it is very helpful if this document has above sample

Best regards
---
Kuninori Morimoto