linux-kernel - Re: mm snapshot broken-out-2007-03-05-02-22.tar.gz uploaded

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 08 Mar 2007 14:28:07 -0500
From:	Randy Cushman <rcushman_linux@...thlink.net>
To:	Takashi Iwai <tiwai@...e.de>
CC:	Michal Piotrowski <michal.k.k.piotrowski@...il.com>,
	Jaroslav Kysela <perex@...e.cz>, alsa-devel@...a-project.org,
	linux-kernel@...r.kernel.org, Adrian Bunk <bunk@...sta.de>
Subject: Re: mm snapshot broken-out-2007-03-05-02-22.tar.gz uploaded

Takashi Iwai wrote:
> At Thu, 08 Mar 2007 12:05:49 -0500,
> Randy Cushman wrote:
>   
>> Takashi Iwai wrote:
>>     
>>> At Thu, 8 Mar 2007 17:52:43 +0100,
>>> Michal Piotrowski wrote:
>>>   
>>>       
>>>> On 08/03/07, Takashi Iwai <tiwai@...e.de> wrote:
>>>>     
>>>>         
>>>>> At Thu, 08 Mar 2007 16:42:26 +0100,
>>>>> Michal Piotrowski wrote:
>>>>>       
>>>>>           
>>>>>>> Also, it'd be helpful if you compare
>>>>>>> /proc/asound/card0/codec97#0/ac97#0-0+regs file before and after the
>>>>>>> patch.  This reveals which register bits differ actually.
>>>>>>>           
>>>>>>>               
>>>>>> a2.txt is form 2.6.21-rc3 witchout 831466f4ad2b5fe23dff77edbe6a7c244435e973
>>>>>>
>>>>>> diff -uprN a2.txt /proc/asound/card0/codec97#0/ac97#0-0+regs
>>>>>> --- a2.txt      2007-03-08 16:36:47.000000000 +0100
>>>>>> +++ /proc/asound/card0/codec97#0/ac97#0-0+regs  2007-03-08 16:36:49.447083597 +0100
>>>>>> @@ -56,8 +56,8 @@
>>>>>>  0:6e = 0000
>>>>>>  0:70 = 0000
>>>>>>  0:72 = 0004
>>>>>> -0:74 = 1001
>>>>>> -0:76 = 64a4
>>>>>> +0:74 = 1201
>>>>>> +0:76 = 7ca4
>>>>>>  0:78 = 000c
>>>>>>  0:7a = 0000
>>>>>>  0:7c = 4144
>>>>>>         
>>>>>>             
>>>>> OK, some wrong bits there.  Could you try the patch below?
>>>>>       
>>>>>           
>>>> Problem fixed. Thanks!
>>>>
>>>> (speaker-test -c6 -Dplug:surround51 is still broken)
>>>>     
>>>>         
>>> Hm, how about turning off "Spread Front To..." control and run
>>> speaker-test?
>>>
>>>
>>> Takashi
>>>
>>>
>>>   
>>>       
>> Michal and Takashi,
>>
>> I've been researching your issue this morning.  I just noticed that your 
>> exchange had ensued.
>>
>> I configured my AD1986-based system as closely as I could to match your 
>> AD1985 mixer settings.  In doing so, I experienced results similar to 
>> what I'm assuming you described:  Only front channel sound came out of 
>> the surround and c/lfe jacks.
>>
>> Here is what I had to change to get the appropriate multichannel sound:
>>
>> 1) The command "speaker-test -c6" does not work on my system.  I have to 
>> use "speaker-test -Dsurround51 -c6"  (Perhaps this is related to running 
>> an old version of speaker-test.)
>>
>> 2) I disabled "Spread Front to Surround and Center/LFE"
>>
>> The explanation for 2) is this:  There can be only one source for each 
>> of the Surround and Center/LFE jacks active at one time.  When "Spread 
>> Front to Surround and Center/LFE" is disabled, the surround channels go 
>> to the the jacks; when this control is enabled, ONLY the front channels 
>> go to the surround and c/lfe jacks.
>>
>> Considering the register values Michal provided and Takashi's patch:
>>
>> The logic of the code is as follows:  (Slightly simplifying,) functions 
>> is_shared_linein() and is_shared_micin() return TRUE if the mixer is 
>> configured for shared jacks AND NOT for 6 channel mode.  In this case it 
>> is appropriate to
>> set these bits (LODIS and CLDIS).
>>     
>
> Yep.  So, can you confirm that my fix is OK?
> Then I'll queue it up to ALSA tree.
>
>
> thanks,
>
> Takashi
>
>
>   
Takashi,

I need to think this through before I can confirm.  Something doesn't 
seem right.

Let me figure out why I wrote ad1986_update_jacks() to be so different 
from ad1888_update_jacks().

I'll keep you posted.

Randy Cushman


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/