From: Takashi Iwai <tiwai@suse.de>
To: "Geoffrey D. Bennett" <g@b4.vu>
Cc: Takashi Iwai <tiwai@suse.de>, Takashi Iwai <tiwai@suse.com>,
linux-sound@vger.kernel.org
Subject: Re: [PATCH] ALSA: scarlett2: Add Vocaster speaker/headphone mute controls
Date: Fri, 05 Sep 2025 10:36:28 +0200 [thread overview]
Message-ID: <87y0qtqos3.wl-tiwai@suse.de> (raw)
In-Reply-To: <aLm2Dy6+O/eLTGVn@m.b4.vu>
On Thu, 04 Sep 2025 17:53:51 +0200,
Geoffrey D. Bennett wrote:
>
> Add support for the speaker and headphone mute controls on Focusrite
> Vocaster interfaces. Unlike other Focusrite interfaces, these mute
> controls are per-output, not per-channel.
>
> Signed-off-by: Geoffrey D. Bennett <g@b4.vu>
> ---
> sound/usb/mixer_scarlett2.c | 97 ++++++++++++++++++++++++++++++++++++-
> 1 file changed, 96 insertions(+), 1 deletion(-)
>
> diff --git a/sound/usb/mixer_scarlett2.c b/sound/usb/mixer_scarlett2.c
> index e762d6110b1a..f2446bf3982c 100644
> --- a/sound/usb/mixer_scarlett2.c
> +++ b/sound/usb/mixer_scarlett2.c
> @@ -10,8 +10,9 @@
> * - Solo/2i2/4i4 Gen 4
> * - Clarett 2Pre/4Pre/8Pre USB
> * - Clarett+ 2Pre/4Pre/8Pre
> + * - Vocaster One/Two
> *
> - * Copyright (c) 2018-2024 by Geoffrey D. Bennett <g at b4.vu>
> + * Copyright (c) 2018-2025 by Geoffrey D. Bennett <g at b4.vu>
> * Copyright (c) 2020-2021 by Vladimir Sadovnikov <sadko4u@gmail.com>
> * Copyright (c) 2022 by Christian Colglazier <christian@cacolglazier.com>
> *
> @@ -75,6 +76,9 @@
> * to many LinuxMusicians people and to Focusrite for hardware
> * donations).
> *
> + * Support for Vocaster One and Two added in Mar 2024 (thanks to many
> + * LinuxMusicians people and to Focusrite for hardware donations).
> + *
> * This ALSA mixer gives access to (model-dependent):
> * - input, output, mixer-matrix muxes
> * - mixer-matrix gain stages
> @@ -364,6 +368,21 @@ static const char *const scarlett2_dim_mute_names[SCARLETT2_DIM_MUTE_COUNT] = {
> "Mute Playback Switch", "Dim Playback Switch"
> };
>
> +/* Vocaster One speaker/headphone mute names */
> +static const char *const vocaster_one_sp_hp_mute_names[] = {
> + "Speaker Mute Playback Switch",
> + "Headphones Mute Playback Switch",
Usually "XXX Playback Switch" indicates that it's a switch for
mute/unmute, hence we don't put "Mute" here.
Actually, having a "mute" switch makes difficult to judge how it
behaves: when the mixer switch is "on", is it muted, i.e. the sound
goes off? It's a contradiction to other mixer controls.
thanks,
Takashi
next prev parent reply other threads:[~2025-09-05 8:36 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-04 15:53 [PATCH] ALSA: scarlett2: Add Vocaster speaker/headphone mute controls Geoffrey D. Bennett
2025-09-05 8:36 ` Takashi Iwai [this message]
2025-09-05 11:51 ` Geoffrey D. Bennett
2025-09-05 12:05 ` Takashi Iwai
2025-09-06 18:03 ` Takashi Iwai
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=87y0qtqos3.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=g@b4.vu \
--cc=linux-sound@vger.kernel.org \
--cc=tiwai@suse.com \
/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.