From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Porter Subject: Re: HDA pin sense and presence detect capability Date: Wed, 25 Jan 2006 10:14:22 -0700 Message-ID: <20060125101422.D29602@cox.net> References: <20060125090825.B29602@cox.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from fed1rmmtao02.cox.net (fed1rmmtao02.cox.net [68.230.241.37]) by alsa.jcu.cz (ALSA's E-mail Delivery System) with ESMTP id AD908199 for ; Wed, 25 Jan 2006 18:14:25 +0100 (MET) Content-Disposition: inline In-Reply-To: ; from apatard@mandriva.com on Wed, Jan 25, 2006 at 05:57:46PM +0100 Sender: alsa-devel-admin@lists.sourceforge.net Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Arnaud Patard Cc: Matt Porter , Takashi Iwai , alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Wed, Jan 25, 2006 at 05:57:46PM +0100, Arnaud Patard wrote: > Matt Porter writes: > > > On Wed, Jan 25, 2006 at 04:50:40PM +0100, Arnaud Patard wrote: > >> About what's done in patch_sigmatel.c : what's the point of disabling > >> the output ? imho, you only need to mute the nid. > > > > It seemed easier to disable the actual pin nid outputs for all line outs > > (thing surround) than walk back to the associated DAC nids (which can > > I see... but afaik, muting the DAC is not the solution because you may > have one DAC with the HP and speakers nids connected to it (for instance > on the ad1981hd). My point was about muting the nid itself. Indeed, that's the other reason for not doing it that way. What do you mean by "muting the nid itself"? The pin widget nid? There's no mute on the sigmatel codec pin widgets since there's no amp capability there. Disabling the output is the way to mute at the pin widget... at least on sigmatel hda codecs. > > vary on sigmatel codecs) and mute those. It accomplishes the same > > thing as muting...except there's no change in mixer state. > > Reflecting a change in the mixer state is more userfriendly but as long > as it's working as the user expects, that's not a real worry. Yeah, I wasn't sure about that. On the sigmatel codecs, the output amps are in the DAC nids so we'd have to mute all of them in an associated group if this is the desired userfriendly view of things. Not a big deal to do, just not as simple as my first pass. > >> Also, if there are several users of similar functions and on others chips than > >> sigmatel, shouldn't it be better to put something in hda_codec.c ? > > > > I think so, but this was somewhat of an experimental first attempt > > at this support. I just haven't had time to do much more with it yet. > > okay. I'm trying to have something working for my codec and then I'll > try to work on this. If you don't agree, scream :) I think it's great. Sounds like Takashi wants to see more shared code as well and his opinion if far more important. ;) -Matt ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642