Re: [PATCH 1/1 - try2] ASoC: add support for multiple cards/codecs in debugfs

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

 



On Thu, Oct 01, 2009 at 10:32:47AM +0300, Peter Ujfalusi wrote:
> In order to support multiple codecs on the same system in the debugfs
> the directory hierarchy need to be changed by adding directory per codec
> under the asoc direcorty:

> debugfs/asoc/{dev_name(socdev->dev)}-{codec->name}/codec_reg

I'd rather use dev_name() for the CODEC itself if possible, that is more
likely to be stable going forward and one of the immediate aims with the
API refactoring is to remove socdev entirely at runtime.

That can be done in a followup, though.

> @@ -126,7 +126,7 @@ static int soc_pcm_apply_symmetry(struct snd_pcm_substream *substream)
> 
>  	if (codec_dai->symmetric_rates || cpu_dai->symmetric_rates ||
>  	    machine->symmetric_rates) {
> -		dev_dbg(card->dev, "Symmetry forces %dHz rate\n",
> +		dev_dbg(card->dev, "Symmetry forces %dHz rate\n",
>  			machine->rate);
> 
>  		ret = snd_pcm_hw_constraint_minmax(substream->runtime,

but this doesn't seem to accomplish that goal?  I can't actually spot
the difference either, I'm assuming it's just whitespace?  If so I'll
apply as-is.
_______________________________________________
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