From: Arnaud Patard <apatard@mandriva.com>
To: Matt Porter <mporter@embeddedalley.com>
Cc: Takashi Iwai <tiwai@suse.de>, alsa-devel@alsa-project.org
Subject: Re: HDA pin sense and presence detect capability
Date: Wed, 25 Jan 2006 17:57:46 +0100 [thread overview]
Message-ID: <m3bqy0p7n9.fsf@anduin.mandrakesoft.com> (raw)
In-Reply-To: <20060125090825.B29602@cox.net> (Matt Porter's message of "Wed, 25 Jan 2006 09:08:25 -0700")
Matt Porter <mporter@embeddedalley.com> writes:
> On Wed, Jan 25, 2006 at 04:50:40PM +0100, Arnaud Patard wrote:
>> Takashi Iwai <tiwai@suse.de> writes:
>>
>> > At Wed, 25 Jan 2006 16:34:22 +0100,
>> > Arnaud Patard wrote:
>> >> This can be achieve with the presence detect capability (0x709,0xf09): When
>> >> plug/unplugging the headphones, one gets an unsollicited responsed and
>> >> the driver mute the channels.
>> >>
>> >> Is there something to do that in the code or should I go and code it ?
>> >> Any opinion/comments about that ?
>> >
>> > You can find an example of such unsolicited event in
>> > patch_sigmatel.c.
>>
>> okay. Thanks.
>>
>> 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.
> 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.
>
>> 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 :)
Regards,
Arnaud
>
> -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
next prev parent reply other threads:[~2006-01-25 16:55 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-25 15:34 HDA pin sense and presence detect capability Arnaud Patard
2006-01-25 15:43 ` Takashi Iwai
2006-01-25 15:50 ` Arnaud Patard
2006-01-25 16:01 ` Takashi Iwai
2006-01-25 16:08 ` Matt Porter
2006-01-25 16:57 ` Arnaud Patard [this message]
2006-01-25 17:14 ` Matt Porter
2006-01-25 17:32 ` Arnaud Patard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3bqy0p7n9.fsf@anduin.mandrakesoft.com \
--to=apatard@mandriva.com \
--cc=alsa-devel@alsa-project.org \
--cc=mporter@embeddedalley.com \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.