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@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;