[alsa-devel] [PATCH][alsa-lib] topology: fix unused-const-variable warning

Liam Girdwood liam.r.girdwood at linux.intel.com
Fri Dec 23 09:49:24 CET 2016


This looks fine by me, Mengdong is this still unused or is it needed by
a future patch ?

On Sat, 2016-12-17 at 09:39 +0900, Takashi Sakamoto wrote:
> Ping to Liam. I wish this patch would be included to next release of
> this library.
> 
> On 2016年11月30日 00:44, Takashi Sakamoto wrote:
> > Last year, unused static const variable was added, then compiler generates
> > a below warning.
> > 
> > dapm.c:43:30: warning: ‘widget_control_map’ defined but not used [-Wunused-const-variable=]
> >  static const struct map_elem widget_control_map[] = {
> >                               ^~~~~~~~~~~~~~~~~~
> > 
> > This commit removes it.
> > 
> > Fixes: 01a0e1a1c219 ("topology: Add DAPM object parser")
> > Signed-off-by: Takashi Sakamoto <o-takashi at sakamocchi.jp>
> > ---
> >  src/topology/dapm.c | 8 --------
> >  1 file changed, 8 deletions(-)
> > 
> > diff --git a/src/topology/dapm.c b/src/topology/dapm.c
> > index e830751..8c585a7 100644
> > --- a/src/topology/dapm.c
> > +++ b/src/topology/dapm.c
> > @@ -39,14 +39,6 @@ static const struct map_elem widget_map[] = {
> >  	{"dai_link", SND_SOC_TPLG_DAPM_DAI_LINK},
> >  };
> >  
> > -/* mapping of widget kcontrol text names to types */
> > -static const struct map_elem widget_control_map[] = {
> > -	{"volsw", SND_SOC_TPLG_DAPM_CTL_VOLSW},
> > -	{"enum_double", SND_SOC_TPLG_DAPM_CTL_ENUM_DOUBLE},
> > -	{"enum_virt", SND_SOC_TPLG_DAPM_CTL_ENUM_VIRT},
> > -	{"enum_value", SND_SOC_TPLG_DAPM_CTL_ENUM_VALUE},
> > -};
> > -
> >  static int lookup_widget(const char *w)
> >  {
> >  	unsigned int i;
> > 




More information about the Alsa-devel mailing list