Done.
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/merge_requests/158
On 2019/8/24 上午9:25, Hui Wang wrote:
OK, I will create a merge request.
Thanks,
Hui.
On 2019/8/23 下午3:48, Georg Chini wrote:
On 18.08.19 07:32, Hui Wang wrote:
In the V3, I addressed all Tanu's and Ge
OK, I will create a merge request.
Thanks,
Hui.
On 2019/8/23 下午3:48, Georg Chini wrote:
On 18.08.19 07:32, Hui Wang wrote:
In the V3, I addressed all Tanu's and Georg's comments:
0002-xxx.patch:
remove the pa_subscription_post() in the
pa_sink_input_set_preferred_sink()
0003
database, let all stream entries to bind the sink
users select, this is not correct since the sink is default_sink now.
This is a temp fix for this issue, after gnome-control-center fixes
this problem, this patch should be reverted.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 14
When the default sink changes, the streams from the old default sink
should be moved to the new default sink, unless the preferred_sink
string is set to the old default sink and the active port of the old
default sink is not unavailable
Signed-off-by: Hui Wang
---
src/modules/dbus/iface-core.c
When a sink is unlinked, all streams of this sink are moved to
default_sink, this action is implemented in the core rather than
modules now.
Signed-off-by: Hui Wang
---
src/modules/module-rescue-streams.c | 134 +---
src/modules/module-stream-restore.c | 50
API to simplify the entry_apply
in the module-stream-restore.c.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 10 ++
src/pulsecore/sink-input.c | 13 +
src/pulsecore/sink-input.h | 2 ++
3 files changed, 17 insertions(+), 8 deletions(-)
d
d on Tanu's design:
https://lists.freedesktop.org/archives/pulseaudio-discuss/2018-October/030531.html
And this is only for sink-input and sink, not for source-output and source.
In the V2, I addressed all comments of V1 and split the big patch into small
patches
as suggested by Tanu. It will be easy to revie
The finial objective is to store the preferred sink name in the
sink-input struct, and use module-stream-restore to save and restore
it.
This patch just replaces the save_sink with preferred_sink, and tries
to keep the original logic.
Signed-off-by: Hui Wang
---
src/modules/module-device
When the user moves a stream to the current default sink, the
preferred_sink should be set to NULL and module-stream-restore
should clear the routing for that stream in the stream database. From
that point on the stream will be always routed to the default sink.
Signed-off-by: Hui Wang
---
src
When a new sink appears, all streams that have their preferred_sink
set to the new sink should be moved to the new sink.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 53 -
src/pulsecore/core.c| 28 +++
src/pulsecore
When the active port of a sink becomes unavailable, all streams from
that sink should be moved to the default sink.
When the active port of a sink changes state from unavailable, all
streams that have their preferred_sink set to this sink should be moved
to this sink.
Signed-off-by: Hui Wang
On 2019/6/30 下午8:15, Georg Chini wrote:
On 17.01.19 07:53, Hui Wang wrote:
When a sink is unlinked, all streams of this sink are moved to
default_sink, this action is implemented in the core rather than
modules now.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 50
On 2019/6/30 下午6:47, Georg Chini wrote:
On 17.01.19 07:53, Hui Wang wrote:
When the user moves a stream to the current default sink, the
preferred_sink should be set to NULL and module-stream-restore
should clear the routing for that stream in the stream database. From
that point on the stream
On 2019/6/30 下午6:44, Georg Chini wrote:
Sorry for the late feedback.
On 17.01.19 07:53, Hui Wang wrote:
If the sink here is NULL, that means users want to clear the
preferred_sink and move the sink-input to the default_sink, otherwise
set the preferred_sink to the sink->name and move the s
will not change
the database, so the users' preference is still saved in the database,
after the active_port is not PA_AVAILABLE_NO, the new streams will
still restore to the preferred device.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 12
1 file changed, 8 i
On 2019/3/27 上午1:44, Tanu Kaskinen wrote:
> On Tue, 2019-03-26 at 13:17 +0800, Hui Wang wrote:
>> On 2017/12/28 下午6:09, Tanu Kaskinen wrote:
>>> The Intel HDMI LPE driver works in a peculiar way when the HDMI cable is
>>> not plugged in: any written audio is immedia
On 2017/12/28 下午6:09, Tanu Kaskinen wrote:
> The Intel HDMI LPE driver works in a peculiar way when the HDMI cable is
> not plugged in: any written audio is immediately discarded and underrun
> is reported. That resulted in an infinite loop, because PulseAudio tried
> to keep the buffer filled, whi
On 2019/3/6 下午4:10, Jaroslav Kysela wrote:
> Dne 06. 03. 19 v 1:23 Hui Wang napsal(a):
>> On 2019/3/6 上午4:00, Jaroslav Kysela wrote:
>>> Dne 05. 03. 19 v 15:09 Hui Wang napsal(a):
>>>> Recently I saw two Headphones on some HP machines, the 1st Headphone
>>&g
database, let all stream entries to bind the sink
users select, this is not correct since the sink is default_sink now.
This is a temp fix for this issue, after gnome-control-center fixes
this problem, this patch should be reverted.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 14
When a sink is unlinked, all streams of this sink are moved to
default_sink, this action is implemented in the core rather than
modules now.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 50 -
src/pulsecore/sink.c| 3 ++
2 files
When the user moves a stream to the current default sink, the
preferred_sink should be set to NULL and module-stream-restore
should clear the routing for that stream in the stream database. From
that point on the stream will be always routed to the default sink.
Signed-off-by: Hui Wang
---
src
When a new sink appears, all streams that have their preferred_sink
set to the new sink should be moved to the new sink.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 53 -
src/pulsecore/core.c| 28 +++
src/pulsecore
When the active port of a sink becomes unavailable, all streams from
that sink should be moved to the default sink.
When the active port of a sink changes state from unavailable, all
streams that have their preferred_sink set to this sink should be moved
to this sink.
Signed-off-by: Hui Wang
When the default sink changes, the streams from the old default sink
should be moved to the new default sink, unless the preferred_sink
string is set to the old default sink and the active port of the old
default sink is not unavailable
Signed-off-by: Hui Wang
---
src/modules/dbus/iface-core.c
The finial objective is to store the preferred sink name in the
sink-input struct, and use module-stream-restore to save and restore
it.
This patch just replaces the save_sink with preferred_sink, and tries
to keep the original logic.
Signed-off-by: Hui Wang
---
src/modules/module-device
API to simplify the entry_apply
in the module-stream-restore.c.
Signed-off-by: Hui Wang
---
src/modules/module-stream-restore.c | 10 ++
src/pulsecore/sink-input.c | 14 ++
src/pulsecore/sink-input.h | 2 ++
3 files changed, 18 insertions(+), 8 deleti
ed by Tanu. It will be easy to review in the future.
The 0007 and 0008 are not in the design of the above, I guess the 0007 is
needed but am not sure. The 0008 is a workaround for current
gnome-control-center.
Hui Wang (8):
sink-input: change bool save_sink to char *preferred_sink
sink-input:
nalog-output-speaker.conf
+++ b/src/modules/alsa/mixer/paths/analog-output-speaker.conf
@@ -32,6 +32,11 @@ state.unplugged = unknown
state.plugged = no
state.unplugged = unknown
+[Jack Front Headphone@1]
+index = 1
+state.plugged = no
+state.unplugged = unknown
+
[Jack Front Headphone]
state.pl
to the pa_alsa_jack and pa_alsa_element.
When probing jacks or elements, we add index as a parameter.
Signed-off-by: Hui Wang
---
src/modules/alsa/alsa-mixer.c | 59 ++-
src/modules/alsa/alsa-mixer.h | 2 +
src/modules/alsa/alsa-ucm.c
name to @1, then
in the alsa-mixer.c, the "@1" will be set to the element->name, and
the "" will be set to the element->alsa_name, we generated the new
element based on the element->name, then this issue is fixed.
Signed-off-by: Hui Wang
---
src/mo
On 2019/1/1 上午2:33, Tanu Kaskinen wrote:
On Mon, 2018-11-05 at 09:47 +0800, Hui Wang wrote:
When the active port of a sink becomes unavailable, all streams from
that sink should be moved to the default sink.
When the active port of an existing sink changes state from
unavailable, all streams
On 2019/1/1 上午2:10, Tanu Kaskinen wrote:
On Mon, 2018-12-31 at 20:01 +0200, Tanu Kaskinen wrote:
On Mon, 2018-11-05 at 09:47 +0800, Hui Wang wrote:
diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
index 63a3456e7..a2a390beb 100644
--- a/src/pulsecore/sink.c
+++ b/src/pulsecore/sink.c
On 2018/12/31 下午9:46, Tanu Kaskinen wrote:
On Fri, 2018-12-21 at 11:05 +0800, Hui Wang wrote:
We have 2 HP desktop models which have one front headphone Jack and
one front headset Jack, in the Linux kernel, these two jacks are
named "Front Headphone Front Jack" and "Front Hea
On 2018/12/23 上午2:57, Tanu Kaskinen wrote:
On Sat, 2018-12-15 at 14:48 +0800, Hui Wang wrote:
On 2018/12/13 下午5:16, Tanu Kaskinen wrote:
On Mon, 2018-11-05 at 09:47 +0800, Hui Wang wrote:
When default sink changes, the stream should be moved to new
default_sink too.
Except if the stream
On 2018/12/23 上午3:06, Tanu Kaskinen wrote:
On Sat, 2018-12-15 at 16:47 +0800, Hui Wang wrote:
On 2018/12/12 下午9:39, Tanu Kaskinen wrote:
Thanks for working on this! Sorry for slow review, I hope I'll be much
quicker to comment on subsequent iterations.
On Mon, 2018-11-05 at 09:47 +0800
rs plug a headset or headphone, the
path of headphone will not be activated, and users can't see
headphones from UI like gnome-control-center.
Signed-off-by: Hui Wang
---
For alsa-info.txt of one of machines, please access:
https://pastebin.ubuntu.com/p/CjCGySMpM5/
.../alsa/mixer/paths/anal
On 2018/12/12 下午9:39, Tanu Kaskinen wrote:
Thanks for working on this! Sorry for slow review, I hope I'll be much
quicker to comment on subsequent iterations.
On Mon, 2018-11-05 at 09:47 +0800, Hui Wang wrote:
And don't move the stream in the module-stream-restore anymor
On 2018/12/13 下午5:16, Tanu Kaskinen wrote:
On Mon, 2018-11-05 at 09:47 +0800, Hui Wang wrote:
When default sink changes, the stream should be moved to new
default_sink too.
Except if the stream's preferred sink is the old default sink.
If it is user to call change default function
k.
Signed-off-by: Hui Wang
---
src/pulsecore/device-port.c | 16
src/pulsecore/sink.c| 13 +
src/pulsecore/sink.h| 1 +
3 files changed, 30 insertions(+)
diff --git a/src/pulsecore/device-port.c b/src/pulsecore/device-port.c
index 464c3f8a2..2604c9
it will move the
stream conditionally.
Signed-off-by: Hui Wang
---
src/modules/dbus/iface-core.c | 2 +-
src/modules/module-default-device-restore.c | 2 +-
src/modules/module-switch-on-connect.c | 28 ++-
src/pulsecore/cli-command.c | 2 +-
src
d and the entry->device will be cleared too from database.
Signed-off-by: Hui Wang
---
src/modules/module-device-manager.c| 2 +-
src/modules/module-intended-roles.c| 2 +-
src/modules/module-stream-restore.c| 47 +-
src/modules/module-switch-on-connec
touch
the source and source_output yet. Based on these patches, I tested the switch
between HDMI output and speaker, it worked as expected. If the way of changing
sink/sink_input is basically correct and is worth to apply to the
source/source_output, I will continue to write patches for them too.
H
When a new sink appears, all streams that have their "preferred_sink"
set to the new sink should be moved to the new sink.
Signed-off-by: Hui Wang
---
src/pulsecore/sink.c | 31 +++
src/pulsecore/sink.h | 1 +
2 files changed, 32 insertions(+)
diff -
On 2018年10月02日 18:50, Georg Chini wrote:
On 02.10.2018 11:49, Tanu Kaskinen wrote:
On Mon, 2018-10-01 at 10:18 +0800, Hui Wang wrote:
On 2018年09月30日 18:30, Tanu Kaskinen wrote:
On Sun, 2018-09-30 at 15:03 +0800, Hui Wang wrote:
This issue is also reported to:
https://gitlab.freedesktop.org
On 2018年09月30日 18:30, Tanu Kaskinen wrote:
On Sun, 2018-09-30 at 15:03 +0800, Hui Wang wrote:
This issue is also reported to:
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/579
Recently we found a weird issue on many laptops with the ubuntu 18.04,
it uses the pulseaudio-11.1 (I
switch_from_port(port);
+ check_and_move_sink_inputs_to_default_sink(c, port);
break;
default:
break;
On 2018年09月30日 15:03, Hui Wang wrote:
This issue is also reported to:
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/579
Recently we found a weird is
This issue is also reported to:
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/579
Recently we found a weird issue on many laptops with the ubuntu 18.04,
it uses the pulseaudio-11.1 (I guess the PA of the latest version also
has this problem). The issue is like this:
1. boot the
Signed-off-by: Hui Wang
---
Forgot to add this to build sys with the commit 60c0edd5.
src/Makefile.am | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index e610db7..0348250 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1312,7
://www.dell.com/support/article/sg/en/sgbsdt1/SLN301105
Signed-off-by: Hui Wang
---
.../alsa/mixer/profile-sets/90-pulseaudio.rules| 1 +
.../profile-sets/dell-dock-tb16-usb-audio.conf | 55 ++
2 files changed, 56 insertions(+)
create mode 100644
src/modules/alsa/mixer
pular than traditional microphone,
It is highly possible that users plug in a headset instead of
microphone, it makes sense to make the headset-mic's priority higher
than headphone-mic's.
Signed-off-by: Hui Wang
---
src/modules/alsa/mixer/paths/analog-input-headset-mic.conf | 2 +-
1
keep the headphone-mic's priority to 87, after booting up, the default
input active_port is headset-mic (that is expected), I plug a microphone
and select "headphone-mic" from UI program, the input active_port is
headphone-mic now, then I unplug the headphone-mic, the input
active_port is chang
On 05/23/2017 07:35 PM, Tanu Kaskinen wrote:
On Tue, 2017-05-23 at 17:36 +0800, Hui Wang wrote:
On 05/23/2017 04:20 PM, Tanu Kaskinen wrote:
On Tue, 2017-05-23 at 11:04 +0800, Hui Wang wrote:
On 05/20/2017 10:51 PM, Tanu Kaskinen wrote:
On Fri, 2017-05-19 at 09:29 +0800, Hui Wang wrote
On 05/23/2017 04:20 PM, Tanu Kaskinen wrote:
On Tue, 2017-05-23 at 11:04 +0800, Hui Wang wrote:
On 05/20/2017 10:51 PM, Tanu Kaskinen wrote:
On Fri, 2017-05-19 at 09:29 +0800, Hui Wang wrote:
Hello Tanu,
Could you please help take a look at this patch? This patch really fix
an issue on some
On 05/20/2017 10:51 PM, Tanu Kaskinen wrote:
On Fri, 2017-05-19 at 09:29 +0800, Hui Wang wrote:
Hello Tanu,
Could you please help take a look at this patch? This patch really fix
an issue on some Dell machines (with realtek codec and has no internal
microphone on them), And I think this minor
/2017 11:21 AM, Hui Wang wrote:
On some Dell machines with realtek codec, they have a headset audio
jack which supports headphone, headset and pure microphone, but this
audio jack can only detect something is plugged in but doesn't have
capabilites to distinguish what is plugged in. So we ne
On 05/03/2017 06:51 PM, Tanu Kaskinen wrote:
On Wed, 2017-05-03 at 11:23 +0800, Hui Wang wrote:
Today I tested the latest master branch (already including your two
patches), seems it did not fix the problem. Below is my testing detail:
Install ubuntu 16.04 on my laptop.
clone the latest
this jack can't work with headphone.
A simple fix for this issue is to exchange the position of
headphone-mic and headset-mic, then the headset-mic will be the
active_port of pa-source if there is no internal microphone and no
other available input devices.
Signed-off-by: Hui Wang
---
s
On 05/02/2017 09:27 PM, Tanu Kaskinen wrote:
On Tue, 2017-05-02 at 08:55 +0800, Hui Wang wrote:
On 04/29/2017 09:04 PM, Tanu Kaskinen wrote:
On Fri, 2017-04-28 at 12:21 +0800, Hui Wang wrote:
Hello Tanu,
Could you please take a look at this patch, you are the maintainer and
recently
On 04/29/2017 09:04 PM, Tanu Kaskinen wrote:
On Fri, 2017-04-28 at 12:21 +0800, Hui Wang wrote:
Hello Tanu,
Could you please take a look at this patch, you are the maintainer and
recently contributed couple of commits to
module-switch-on-port-available.c. :-)
Well, I'm trying to concen
Hello Tanu,
Could you please take a look at this patch, you are the maintainer and
recently contributed couple of commits to
module-switch-on-port-available.c. :-)
Thanks in advance.
Hui.
On 04/27/2017 11:20 AM, Hui Wang wrote:
Suppose your machine has two sound cards as below:
Card
nly
has one sound card), there is no issue. It looks like the current
switch-on-port-available can't handle multiple sound cards well.
To fix it, adding a function to check and change default sink or
source after executing switch_to_port() or switch_from_port().
Signed-off-by: Hui Wang
---
s
nly
has one sound card), there is no issue. It looks like the current
switch-on-port-available can't handle multiple sound cards well.
To fix it, adding a function to check and change default sink or
source after executing switch_to_port() or switch_from_port().
Signed-off-by: Hui Wang
---
s
storing from FPU register to memory will introduce
some variation, and this small variation will introduce small
variation to the rewinding result.
So adding the tolerant variation for comparing the rewind result, make
this test case can work on both 64bits OS and 32bits OS.
Signed-off-by: Hui Wang
Signed-off-by: Hui Wang
---
src/tests/resampler-test.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/tests/resampler-test.c b/src/tests/resampler-test.c
index 751d2dc..9832a31 100644
--- a/src/tests/resampler-test.c
+++ b/src/tests/resampler-test.c
@@ -330,8 +330,6 @@ int main(int
storing from FPU register to memory will introduce
some variation, and this small variation will introduce small
variation to the rewinding result.
So adding the tolerant variation for comparing the rewind result, make
this test case can work on both 64bits OS and 32bits OS.
Signed-off-by: Hui Wang
On 03/31/2015 05:54 AM, Glenn Golden wrote:
HW: ThinkPad T-510, x86_64
SW: Arch Linux (synched within a week) kernel = 3.19.2, PA 6.0
On the above setup, the mic mute function (invoked via pavucontrol) works as
expected: Click the mute icon, it mutes the mic input, indicates the muted
state by g
On 09/28/2014 05:50 PM, Tanu Kaskinen wrote:
On Sun, 2014-09-28 at 12:41 +0300, Tanu Kaskinen wrote:
On Mon, 2014-09-22 at 11:50 +0800, Hui Wang wrote:
On a machine without fixed connecting audio devices like internal
microphone or internal speaker, and when there is no external audio
devices
It is possible that the chosen active_port doesn't equal
new_data->active_port, using p->name is more accurate.
Please refer to sink_new_hook_callback()
Signed-off-by: Hui Wang
---
src/modules/module-switch-on-port-available.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
On 09/26/2014 12:13 AM, Raymond Yau wrote:
>>
>> >
>> > On a machine without fixed connecting audio devices like internal
>> > microphone or internal speaker, and when there is no external audio
>> > devices plugging in, the default source/sink is
alsa_input/alsa_output
>> > and there is no i
an argument, the policy is
if a new source/sink is connected and current default source/sink's
active_port is AVAILABLE_NO, we let the new added one switch to
default one.
BugLink: http://bugs.launchpad.net/bugs/1369476
Signed-off-by: Hui Wang
---
In the V3:
if the get_value_boolean
an argument, the policy is
if a new source/sink is connected and current default source/sink's
active_port is AVAILABLE_NO, we let the new added one switch to
default one.
BugLink: http://bugs.launchpad.net/bugs/1369476
Signed-off-by: Hui Wang
---
In the V3:
if the get_value_boolean
an argument, the policy is
if a new source/sink is connected and current default source/sink's
active_port is AVAILABLE_NO, we let the new added one switch to
default one.
BugLink: http://bugs.launchpad.net/bugs/1369476
Signed-off-by: Hui Wang
---
Diff vs v1:
1. change the argument na
an argument, the policy is
if a new source/sink is connected and current default source/sink's
active_port is AVAILABLE_NO, we let the new added one switch to
default one.
BugLink: http://bugs.launchpad.net/bugs/1369476
Signed-off-by: Hui Wang
---
src/modules/module-switch-on-connect.c
if the default sink is not available. (same rules apply to the
source selecting):
construct a new hashmap with all ports (of all relevant sinks) and
then call find_best on the new hashmap to find the best port, finally
find the corresponding sink using the best port.
Signed-off-by: Hui Wang
(same rules apply to the source selecting):
construct a new hashmap with all ports (of all relevant sinks) and
then call find_best on the new hashmap to find the best port, finally
find the corresponding sink using the best port.
Signed-off-by: Hui Wang
---
src/modules/module-rescue-streams.c | 95
(same rules apply to the source selecting):
If the sinks have the port with PA_AVAILABLE_YES, we will choose the
one from them with a highest priority port. If all sinks don't have
the port with PA_AVAILABLE_YES, we will choose the one with a highest
priority port.
Signed-off-by: Hui Wang
---
76 matches
Mail list logo