* Re: [pulseaudio-discuss] [PATCH v0 07/20] Consider unknown availability in module-switch-on-port-available
[not found] ` <5034A2B6.5080905@canonical.com>
@ 2012-08-22 12:00 ` Raymond Yau
2012-08-22 12:08 ` David Henningsson
0 siblings, 1 reply; 2+ messages in thread
From: Raymond Yau @ 2012-08-22 12:00 UTC (permalink / raw)
To: General PulseAudio Discussion, alsa-devel
2012-8-22 下午5:13 於 "David Henningsson" <david.henningsson@canonical.com> 寫道:
>
> On 08/22/2012 10:08 AM, Mikel Astiz wrote:
>>
>> From: Mikel Astiz <mikel.astiz@bmw-carit.de>
>>
>> A port with the availability state flag set to unknown should not be
>> considered an active port.
>> ---
>> src/modules/module-switch-on-port-available.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/modules/module-switch-on-port-available.c
b/src/modules/module-switch-on-port-available.c
>> index 1da612d..8c16821 100644
>> --- a/src/modules/module-switch-on-port-available.c
>> +++ b/src/modules/module-switch-on-port-available.c
>> @@ -85,7 +85,7 @@ static pa_bool_t try_to_switch_profile(pa_card *card,
pa_device_port *port) {
>> PA_IDXSET_FOREACH(sink, card->sinks, state2) {
>> if (!sink->active_port)
>> continue;
>> - if (sink->active_port->available !=
PA_PORT_AVAILABLE_NO)
>> + if (sink->active_port->available ==
PA_PORT_AVAILABLE_YES)
>> found_active_port = TRUE;
>> }
>>
>>
>
> I think this will break the following use case (on a normal laptop) :
>
> Assume we plug in HDMI. With this patch we will get inconsistent
behaviour: if headphones are plugged in (available = yes), we won't switch
to HDMI, but if not (speakers are available = unknown), we will switch to
HDMI.
>
>
if you look at those bug reports of no sound for those on-board ac97 codecs
with ubuntu 12.04
https://bugs.launchpad.net/ubuntu/+source/alsa-driver/+bug/1033996
how does pulseaudio determine which controls are effective in the headphone
path and speaker path of those ac97 codecs ?
the analog-output-stereo profile for the line out jack is missing
( 0.056| 0.000) D: [pulseaudio] alsa-mixer.c: Removing path 'analog-output'
as it is a subset of 'analog-output-headphone
is there any solution for those snd-intel8x0 and those onboard ac97 codecs
with headphone playback volume control ( no ctljack control and no phantom
port)?
http://cgit.freedesktop.org/pulseaudio/pulseaudio/commit/src/modules/alsa/alsa-mixer.c?id=91b25005978d343a989c9485e226dc9a18a2fb28
http://git.freedesktop.org/pulseaudio/pulseaudio/commit/src/modules/alsa/alsa-mixer.c?id=5c832a5b8a2bc2ce7447eab10acaeed30cd3ed78
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [pulseaudio-discuss] [PATCH v0 07/20] Consider unknown availability in module-switch-on-port-available
2012-08-22 12:00 ` [pulseaudio-discuss] [PATCH v0 07/20] Consider unknown availability in module-switch-on-port-available Raymond Yau
@ 2012-08-22 12:08 ` David Henningsson
0 siblings, 0 replies; 2+ messages in thread
From: David Henningsson @ 2012-08-22 12:08 UTC (permalink / raw)
To: General PulseAudio Discussion; +Cc: Raymond Yau, alsa-devel
On 08/22/2012 02:00 PM, Raymond Yau wrote:
>
> 2012-8-22 下午5:13 於 "David Henningsson"
> <david.henningsson@canonical.com
> <mailto:david.henningsson@canonical.com>> 寫道:
> > Assume we plug in HDMI. With this patch we will get inconsistent
> behaviour: if headphones are plugged in (available = yes), we won't
> switch to HDMI, but if not (speakers are available = unknown), we will
> switch to HDMI.
> >
> >
>
> if you look at those bug reports of no sound for those on-board ac97
> codecs with ubuntu 12.04
>
> https://bugs.launchpad.net/ubuntu/+source/alsa-driver/+bug/1033996
>
> how does pulseaudio determine which controls are effective in the
> headphone path and speaker path of those ac97 codecs ?
Please do not hijack threads to discuss something different. If you have
an unrelated question such as this, start a new thread instead.
Thanks.
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-08-22 12:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1345622913-16470-1-git-send-email-mikel.astiz.oss@gmail.com>
[not found] ` <1345622913-16470-8-git-send-email-mikel.astiz.oss@gmail.com>
[not found] ` <5034A2B6.5080905@canonical.com>
2012-08-22 12:00 ` [pulseaudio-discuss] [PATCH v0 07/20] Consider unknown availability in module-switch-on-port-available Raymond Yau
2012-08-22 12:08 ` David Henningsson
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.