Re: Alternative binding proposal for tda998x audio (Was: Re: [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/01/16 21:26, Jean-Francois Moine wrote:
On Tue, 1 Mar 2016 20:29:17 +0200
Jyri Sarha <jsarha@xxxxxx> wrote:

I understand the short comings of simple-card and it's binding. However,
the binding is documented and it is feasible to extract the audio
connections from a simple-card binding too. In fact it models the I2S
connections better than straight out of tehe box graph binding. Actually
a graph is not the best way describe an i2s-bus with multiple DAIs
(codec or CPU) connected to it.

I still don't understand your problem. You want something like:


The problem is adding redundant unused details into binding without any plan of ever using them.

Fundamentally my problem is finding some consensus on the tda998x ASoC implementation. I've been reusing your binding for couple of review rounds and there has been some well justified critique towards it. I feel stupid in pushing forward something that I do not completely agree myself, so I decided to try something else.

	audio-ports = <	TDA998x_SPDIF	0x04
			TDA998x_I2S	0x03>;

and the graph definition would be:

	port@03 {
		reg = <0x03>;
		port-type = "audio-i2s";
		...
	};

	port@04 {
		reg = <0x04>;
		port-type = "audio-spdif";
		...
	};

Apart the syntax, I don't really see the difference.


Yes, the necessary information is contained in both bindings. I can live with either one of them, but I would prefer my version. Essentially I would just like to move forward.

Best regards,
Jyri
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux