Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sat, 22 Aug 2020 18:17:41 +0200
Message-ID: <s5h4koubqoq.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     Connor McAdams <conmanx360@gmail.com>
Cc:     Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 05/20] ALSA: hda/ca0132 - Add bass redirection controls.
In-Reply-To: <20200821185239.26133-6-conmanx360@gmail.com>
References: <20200821185239.26133-1-conmanx360@gmail.com>
        <20200821185239.26133-6-conmanx360@gmail.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3
 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, 21 Aug 2020 20:52:22 +0200,
Connor McAdams wrote:
> 
> +static int ca0132_alt_add_bass_redirection_crossover(struct hda_codec *codec)
> +{
> +	const char *namestr = "Output: Bass Redirection Playback Volume";
> +	struct snd_kcontrol_new knew =
> +		HDA_CODEC_VOLUME_MONO(namestr, BASS_REDIRECTION_XOVER, 1, 0,
> +				HDA_OUTPUT);

This is no ideal name string for a control element, too.
Better to omit "Output:" prefix at least.


thanks,

Takashi