Linux Sound subsystem development
 help / color / mirror / Atom feed
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

  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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox