Re: CMI8768 patch

[Takashi Iwai <tiwai@suse.de>]

At Wed, 23 Feb 2005 03:07:02 +0800,
Tien, C.L. wrote:
> 
> I made changes to the CMI8738-MC8.conf so the PCM volume appear after
> play with -D front or -D rear:
> 
> --- alsa/CMI8738-MC8.conf	Fri Feb 18 17:29:38 2005
> +++ CMI8738-MC8.conf	Tue Feb 22 13:28:58 2005
> @@ -7,7 +7,7 @@
>  CMI8738-MC8.pcm.front.0 {
>  	@args [ CARD ]
>  	@args.CARD {
> -		type string
> +		type integer
>  	}
>  	type softvol
>  	slave.pcm {
> @@ -57,7 +57,7 @@
>  CMI8738-MC8.pcm.rear.0 {
>  	@args [ CARD ]
>  	@args.CARD {
> -		type string
> +		type integer
>  	}
>  	type softvol
>  	slave.pcm {
> @@ -76,7 +76,7 @@
>  CMI8738-MC8.pcm.surround40.0 {
>  	@args [ CARD ]
>  	@args.CARD {
> -		type string
> +		type integer
>  	}
>  	type softvol
>  	slave.pcm {

Oh yes, they should be integer.

> But the aplay -D default still shows:
> 
> ALSA lib pcm.c:2068:(snd_pcm_open_noupdate) Unknown PCM 0
> ALSA lib pcm_dmix.c:868:(snd_pcm_dmix_open) unable to open slave
> aplay: main:508: audio open error: No such file or directory

Hmm, are you using CVS version of alsa-lib or 1.0.8?
The config file I sent was for CVS version.

> BTW, the query_chip() should be run with 8738 only, there is no
> chip version for 8338 and the register checked in query_chip()
> is reserved in 8338, so I made following change:

Agreed.  Will be fixed on CVS.

> 
> --- alsa-driver-1.0.8/alsa-kernel/pci/cmipci.c	Mon Jan  3 06:45:31 2005
> +++ alsa-driver-1.0.8-update/alsa-kernel/pci/cmipci.c	Tue Feb 22 13:31:06 2005@@ -2623,7 +2664,8 @@
>  	cm->max_channels = 2;
>  	cm->do_soft_ac3 = soft_ac3[dev];
>  
> -	query_chip(cm);
> +	if (strcmp(card->driver, "CMI8338"))
> +		query_chip(cm);
>  
>  	cm->dig_status = SNDRV_PCM_DEFAULT_CON_SPDIF;
>  	cm->dig_pcm_status = SNDRV_PCM_DEFAULT_CON_SPDIF;
> 
> I also recmmend that the "Four Channel" switch should be turned off for
> surround41 and surround50, as well as surround51 and surround71. The

It is turned off, indeed.  In the configuration:

			{
				name "Four Channel Mode"
				lock true
				preserve true
==>				value false
			}


Takashi


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click