Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757034AbcKDBIi (ORCPT ); Thu, 3 Nov 2016 21:08:38 -0400 Received: from smtp.csie.ntu.edu.tw ([140.112.30.61]:39710 "EHLO smtp.csie.ntu.edu.tw" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756725AbcKDBIh (ORCPT ); Thu, 3 Nov 2016 21:08:37 -0400 MIME-Version: 1.0 In-Reply-To: <20161103173602.zchyciwj66zdibc7@lukather> References: <20161103075556.29018-1-wens@csie.org> <20161103075556.29018-7-wens@csie.org> <20161103173602.zchyciwj66zdibc7@lukather> From: Chen-Yu Tsai Date: Fri, 4 Nov 2016 09:08:11 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 06/14] ASoC: sun4i-codec: Add support for A31 playback through headphone output To: Maxime Ripard Cc: Chen-Yu Tsai , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Rob Herring , Mark Rutland , Linux-ALSA , linux-arm-kernel , linux-kernel , devicetree , linux-sunxi Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1049 Lines: 30 On Fri, Nov 4, 2016 at 1:36 AM, Maxime Ripard wrote: > Hi, > > On Thu, Nov 03, 2016 at 03:55:48PM +0800, Chen-Yu Tsai wrote: >> +/* headphone controls */ >> +static const char * const sun6i_codec_hp_src_enum_text[] = { >> + "DAC", "Mixer", >> +}; >> + >> +static SOC_ENUM_DOUBLE_DECL(sun6i_codec_hp_src_enum, >> + SUN6I_CODEC_OM_DACA_CTRL, >> + SUN6I_CODEC_OM_DACA_CTRL_LHPIS, >> + SUN6I_CODEC_OM_DACA_CTRL_RHPIS, >> + sun6i_codec_hp_src_enum_text); >> + >> +static const struct snd_kcontrol_new sun6i_codec_hp_src[] = { >> + SOC_DAPM_ENUM("Headphone Source Playback Route", >> + sun6i_codec_hp_src_enum), >> +}; > > What is that route exactly? A muxer? Yup. The following is part of the widgets list later in the code: + /* Headphone output path */ + SND_SOC_DAPM_MUX("Headphone Source Playback Route", + SND_SOC_NOPM, 0, 0, sun6i_codec_hp_src), ChenYu