Re: [pulseaudio-discuss] [PATCH] Add support for SteelSeries Arctis 5 and renamed Arctis 7 files appropriately

2018-02-21 Thread Tanu Kaskinen 
On Tue, 2018-02-13 at 20:06 +0100, Bert Hekman wrote:
> ---
>  src/modules/alsa/alsa-mixer.c  |  2 ++
>  .../paths/steelseries-arctis-5-output-chat.conf| 27 
> ++
>  .../paths/steelseries-arctis-5-output-game.conf| 27 
> ++
>  ...-input.conf => steelseries-arctis-7-input.conf} |  0
>  conf => steelseries-arctis-7-output-mono.conf} |  0
>  ...onf => steelseries-arctis-7-output-stereo.conf} |  0
>  .../alsa/mixer/profile-sets/90-pulseaudio.rules|  3 ++-
>  .../steelseries-arctis-5-usb-audio.conf| 22 ++
>  ...io.conf => steelseries-arctis-7-usb-audio.conf} |  6 ++---
>  9 files changed, 83 insertions(+), 4 deletions(-)
>  create mode 100644 
> src/modules/alsa/mixer/paths/steelseries-arctis-5-output-chat.conf
>  create mode 100644 
> src/modules/alsa/mixer/paths/steelseries-arctis-5-output-game.conf
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-input.conf => 
> steelseries-arctis-7-input.conf} (100%)
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-output-mono.conf => 
> steelseries-arctis-7-output-mono.conf} (100%)
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-output-stereo.conf 
> => steelseries-arctis-7-output-stereo.conf} (100%)
>  create mode 100644 
> src/modules/alsa/mixer/profile-sets/steelseries-arctis-5-usb-audio.conf
>  rename 
> src/modules/alsa/mixer/profile-sets/{steelseries-arctis-usb-audio.conf => 
> steelseries-arctis-7-usb-audio.conf} (91%)

Thanks! Applied to the "next" branch (will be in the 13.0 release).

-- 
Tanu

https://liberapay.com/tanuk
https://www.patreon.com/tanuk
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Re: [pulseaudio-discuss] [PATCH] Add support for SteelSeries Arctis 5 and renamed Arctis 7 files appropriately

2018-02-13 Thread Bert Hekman 
So this is a follow up to an earlier patch. This one fixes two
problems the earlier patch had: the separate paths allow both outputs
to be selected from the sound configuration screen and configuring
"Com Speaker" as element in the chat output causes the volume to be
set a lot louder.

2018-02-13 20:06 GMT+01:00 Bert Hekman :
> ---
>  src/modules/alsa/alsa-mixer.c  |  2 ++
>  .../paths/steelseries-arctis-5-output-chat.conf| 27 
> ++
>  .../paths/steelseries-arctis-5-output-game.conf| 27 
> ++
>  ...-input.conf => steelseries-arctis-7-input.conf} |  0
>  conf => steelseries-arctis-7-output-mono.conf} |  0
>  ...onf => steelseries-arctis-7-output-stereo.conf} |  0
>  .../alsa/mixer/profile-sets/90-pulseaudio.rules|  3 ++-
>  .../steelseries-arctis-5-usb-audio.conf| 22 ++
>  ...io.conf => steelseries-arctis-7-usb-audio.conf} |  6 ++---
>  9 files changed, 83 insertions(+), 4 deletions(-)
>  create mode 100644 
> src/modules/alsa/mixer/paths/steelseries-arctis-5-output-chat.conf
>  create mode 100644 
> src/modules/alsa/mixer/paths/steelseries-arctis-5-output-game.conf
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-input.conf => 
> steelseries-arctis-7-input.conf} (100%)
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-output-mono.conf => 
> steelseries-arctis-7-output-mono.conf} (100%)
>  rename src/modules/alsa/mixer/paths/{steelseries-arctis-output-stereo.conf 
> => steelseries-arctis-7-output-stereo.conf} (100%)
>  create mode 100644 
> src/modules/alsa/mixer/profile-sets/steelseries-arctis-5-usb-audio.conf
>  rename 
> src/modules/alsa/mixer/profile-sets/{steelseries-arctis-usb-audio.conf => 
> steelseries-arctis-7-usb-audio.conf} (91%)
>
> diff --git a/src/modules/alsa/alsa-mixer.c b/src/modules/alsa/alsa-mixer.c
> index 7de1c7de..b8c2bc72 100644
> --- a/src/modules/alsa/alsa-mixer.c
> +++ b/src/modules/alsa/alsa-mixer.c
> @@ -2480,6 +2480,8 @@ static int path_verify(pa_alsa_path *p) {
>  { "iec958-passthrough-output",  N_("Digital Passthrough (S/PDIF)") },
>  { "multichannel-input", N_("Multichannel Input") },
>  { "multichannel-output",N_("Multichannel Output") },
> +{ "steelseries-arctis-5-output-game", N_("Game Output") },
> +{ "steelseries-arctis-5-output-chat", N_("Chat Output") },
>  };
>
>  pa_alsa_element *e;
> diff --git 
> a/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-chat.conf 
> b/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-chat.conf
> new file mode 100644
> index ..5842bfe8
> --- /dev/null
> +++ b/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-chat.conf
> @@ -0,0 +1,27 @@
> +# This file is part of PulseAudio.
> +#
> +# PulseAudio is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU Lesser General Public License as
> +# published by the Free Software Foundation; either version 2.1 of the
> +# License, or (at your option) any later version.
> +#
> +# PulseAudio is distributed in the hope that it will be useful, but
> +# WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> +# General Public License for more details.
> +#
> +# You should have received a copy of the GNU Lesser General Public License
> +# along with PulseAudio; if not, see .
> +
> +; Steelseries Arctis 5 USB headset stereo chat path. The headset has two
> +; output devices. The first one is meant for voice audio, and the second
> +; one meant for everything else. The purpose of this unusual design is to
> +; provide separate volume controls for voice and other audio, which can be
> +; useful in gaming.
> +
> +[General]
> +priority = 50
> +
> +[Element Com Speaker]
> +switch = mute
> +volume = merge
> diff --git 
> a/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-game.conf 
> b/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-game.conf
> new file mode 100644
> index ..c675d523
> --- /dev/null
> +++ b/src/modules/alsa/mixer/paths/steelseries-arctis-5-output-game.conf
> @@ -0,0 +1,27 @@
> +# This file is part of PulseAudio.
> +#
> +# PulseAudio is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU Lesser General Public License as
> +# published by the Free Software Foundation; either version 2.1 of the
> +# License, or (at your option) any later version.
> +#
> +# PulseAudio is distributed in the hope that it will be useful, but
> +# WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> +# General Public License for more details.
> +#
> +# You should have received a copy of the GNU Lesser General Public License
> +# along with PulseAudio; if not, see .
> +
> +; Steelseries