All of lore.kernel.org
 help / color / mirror / Atom feed
* [ALSA - driver 0000925]: no sound with CMI9761A, Cus fix verified OK
@ 2005-02-22 20:26 thomas schorpp
  2005-02-23 14:01 ` Takashi Iwai
  0 siblings, 1 reply; 3+ messages in thread
From: thomas schorpp @ 2005-02-22 20:26 UTC (permalink / raw)
  To: alsa-devel

post it here, cant get on bugtracker:

this is ok with:

-pcm playback
-pcm capture
-looped through lines

thx, cus.

  Additional Information  	 After some testing i found that the 
following patch fixes the problem:

--- ac97_patch.c.orig 2005-02-18 00:08:01.000000000 +0100
+++ ac97_patch.c 2005-02-19 18:08:23.000000000 +0100
@@ -2010,7 +2010,7 @@
          ac97->ext_id |= AC97_EI_SPDIF;
         /* to be sure: we overwrite the ext status bits */
         snd_ac97_write_cache(ac97, AC97_EXTENDED_STATUS, 0x05c0);
- snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0209);
+ snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0009);
         ac97->rates[AC97_RATES_SPDIF] = SNDRV_PCM_RATE_48000; /* 48k 
only */

         /* set-up multi channel */


I don't know why it works, and what that 0x0200 bit means in that 
register, someone may explain.

Cus

outstanding : Maybe better to use 0x0001 instead? I guess 0x0008 
controls the "IEC958 Capture Monitor", and got some reports that this 
disturbs the output. tashaki.

-no, now it disturbs no more.

all fine, thx.

add.:

with this you get master vol slider back:


	/* CM9761 has no Master and PCM volume although the register reacts */
	ac97->flags |= /*AC97_HAS_NO_MASTER_VOL |*/ AC97_HAS_NO_PCM_VOL;

@takashi: maybe this was tested only with pcm not with looped through 
lines, before. i assume false negatives from previous tests.


tom



-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [ALSA - driver 0000925]: no sound with CMI9761A, Cus fix verified OK
  2005-02-22 20:26 [ALSA - driver 0000925]: no sound with CMI9761A, Cus fix verified OK thomas schorpp
@ 2005-02-23 14:01 ` Takashi Iwai
  2005-02-23 14:19   ` thomas schorpp
  0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2005-02-23 14:01 UTC (permalink / raw)
  To: t.schorpp; +Cc: alsa-devel

At Tue, 22 Feb 2005 21:26:14 +0100,
thomas schorpp wrote:
> 
> post it here, cant get on bugtracker:

Still not possible?

> this is ok with:
> 
> -pcm playback
> -pcm capture
> -looped through lines
> 
> thx, cus.
> 
>   Additional Information  	 After some testing i found that the 
> following patch fixes the problem:
> 
> --- ac97_patch.c.orig 2005-02-18 00:08:01.000000000 +0100
> +++ ac97_patch.c 2005-02-19 18:08:23.000000000 +0100
> @@ -2010,7 +2010,7 @@
>           ac97->ext_id |= AC97_EI_SPDIF;
>          /* to be sure: we overwrite the ext status bits */
>          snd_ac97_write_cache(ac97, AC97_EXTENDED_STATUS, 0x05c0);
> - snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0209);
> + snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0009);
>          ac97->rates[AC97_RATES_SPDIF] = SNDRV_PCM_RATE_48000; /* 48k 
> only */
> 
>          /* set-up multi channel */
> 
> 
> I don't know why it works, and what that 0x0200 bit means in that 
> register, someone may explain.

> 
> Cus
> 
> outstanding : Maybe better to use 0x0001 instead? I guess 0x0008 
> controls the "IEC958 Capture Monitor", and got some reports that this 
> disturbs the output. tashaki.
> 
> -no, now it disturbs no more.

Then let's take 0x09.


> all fine, thx.
> 
> add.:
> 
> with this you get master vol slider back:
> 
> 
> 	/* CM9761 has no Master and PCM volume although the register reacts */
> 	ac97->flags |= /*AC97_HAS_NO_MASTER_VOL |*/ AC97_HAS_NO_PCM_VOL;
> 
> @takashi: maybe this was tested only with pcm not with looped through 
> lines, before. i assume false negatives from previous tests.

OK, I'll fix the CVS now.


thanks,

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [ALSA - driver 0000925]: no sound with CMI9761A, Cus fix verified OK
  2005-02-23 14:01 ` Takashi Iwai
@ 2005-02-23 14:19   ` thomas schorpp
  0 siblings, 0 replies; 3+ messages in thread
From: thomas schorpp @ 2005-02-23 14:19 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: alsa-devel

Takashi Iwai wrote:
> At Tue, 22 Feb 2005 21:26:14 +0100,
> thomas schorpp wrote:
> 
>>post it here, cant get on bugtracker:
> 
> 
> Still not possible?

no. do you respond to my notice to the webmaster?

> 
> 
>>this is ok with:
>>
>>-pcm playback
>>-pcm capture
>>-looped through lines
>>
>>thx, cus.
>>
>>  Additional Information  	 After some testing i found that the 
>>following patch fixes the problem:
>>
>>--- ac97_patch.c.orig 2005-02-18 00:08:01.000000000 +0100
>>+++ ac97_patch.c 2005-02-19 18:08:23.000000000 +0100
>>@@ -2010,7 +2010,7 @@
>>          ac97->ext_id |= AC97_EI_SPDIF;
>>         /* to be sure: we overwrite the ext status bits */
>>         snd_ac97_write_cache(ac97, AC97_EXTENDED_STATUS, 0x05c0);
>>- snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0209);
>>+ snd_ac97_write_cache(ac97, AC97_CM9761_SPDIF_CTRL, 0x0009);
>>         ac97->rates[AC97_RATES_SPDIF] = SNDRV_PCM_RATE_48000; /* 48k 
>>only */
>>
>>         /* set-up multi channel */
>>
>>
>>I don't know why it works, and what that 0x0200 bit means in that 
>>register, someone may explain.
> 
> 
>>Cus
>>
>>outstanding : Maybe better to use 0x0001 instead? I guess 0x0008 
>>controls the "IEC958 Capture Monitor", and got some reports that this 
>>disturbs the output. tashaki.
>>
>>-no, now it disturbs no more.
> 
> 
> Then let's take 0x09.

0x0009 ? (LSB/MSB)

> 
> 
> 
>>all fine, thx.
>>
>>add.:
>>
>>with this you get master vol slider back:
>>
>>
>>	/* CM9761 has no Master and PCM volume although the register reacts */
>>	ac97->flags |= /*AC97_HAS_NO_MASTER_VOL |*/ AC97_HAS_NO_PCM_VOL;
>>
>>@takashi: maybe this was tested only with pcm not with looped through 
>>lines, before. i assume false negatives from previous tests.
> 
> 
> OK, I'll fix the CVS now.
> 
> 
> thanks,

at your sevice ;)

> 
> Takashi

tom




-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-02-23 14:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-02-22 20:26 [ALSA - driver 0000925]: no sound with CMI9761A, Cus fix verified OK thomas schorpp
2005-02-23 14:01 ` Takashi Iwai
2005-02-23 14:19   ` thomas schorpp

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.