Commit b18768f5 authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown

ASoC: soc-core: remove snd_soc_component_add/del()

soc-core has
snd_soc_add_component(), snd_soc_component_add(),
snd_soc_del_component(), snd_soc_component_del().

These are very confusing naming.
snd_soc_component_xxx() are called from snd_soc_xxx_component(),
and these are very small.
Let's merge these into snd_soc_xxx_component(), and
remove snd_soc_component_xxx().
Reported-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: default avatarRanjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/871rum3jmy.wl-kuninori.morimoto.gx@renesas.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 486c7978
...@@ -2746,34 +2746,6 @@ EXPORT_SYMBOL_GPL(snd_soc_component_exit_regmap); ...@@ -2746,34 +2746,6 @@ EXPORT_SYMBOL_GPL(snd_soc_component_exit_regmap);
#endif #endif
static void snd_soc_component_add(struct snd_soc_component *component)
{
mutex_lock(&client_mutex);
if (!component->driver->write && !component->driver->read) {
if (!component->regmap)
component->regmap = dev_get_regmap(component->dev,
NULL);
if (component->regmap)
snd_soc_component_setup_regmap(component);
}
/* see for_each_component */
list_add(&component->list, &component_list);
mutex_unlock(&client_mutex);
}
static void snd_soc_component_del(struct snd_soc_component *component)
{
struct snd_soc_card *card = component->card;
if (card)
snd_soc_unbind_card(card, false);
list_del(&component->list);
}
#define ENDIANNESS_MAP(name) \ #define ENDIANNESS_MAP(name) \
(SNDRV_PCM_FMTBIT_##name##LE | SNDRV_PCM_FMTBIT_##name##BE) (SNDRV_PCM_FMTBIT_##name##LE | SNDRV_PCM_FMTBIT_##name##BE)
static u64 endianness_format_map[] = { static u64 endianness_format_map[] = {
...@@ -2820,8 +2792,14 @@ static void snd_soc_try_rebind_card(void) ...@@ -2820,8 +2792,14 @@ static void snd_soc_try_rebind_card(void)
static void snd_soc_del_component_unlocked(struct snd_soc_component *component) static void snd_soc_del_component_unlocked(struct snd_soc_component *component)
{ {
struct snd_soc_card *card = component->card;
snd_soc_unregister_dais(component); snd_soc_unregister_dais(component);
snd_soc_component_del(component);
if (card)
snd_soc_unbind_card(card, false);
list_del(&component->list);
} }
int snd_soc_add_component(struct device *dev, int snd_soc_add_component(struct device *dev,
...@@ -2833,6 +2811,8 @@ int snd_soc_add_component(struct device *dev, ...@@ -2833,6 +2811,8 @@ int snd_soc_add_component(struct device *dev,
int ret; int ret;
int i; int i;
mutex_lock(&client_mutex);
ret = snd_soc_component_initialize(component, component_driver, dev); ret = snd_soc_component_initialize(component, component_driver, dev);
if (ret) if (ret)
goto err_free; goto err_free;
...@@ -2850,14 +2830,26 @@ int snd_soc_add_component(struct device *dev, ...@@ -2850,14 +2830,26 @@ int snd_soc_add_component(struct device *dev,
goto err_cleanup; goto err_cleanup;
} }
snd_soc_component_add(component); if (!component->driver->write && !component->driver->read) {
snd_soc_try_rebind_card(); if (!component->regmap)
component->regmap = dev_get_regmap(component->dev,
NULL);
if (component->regmap)
snd_soc_component_setup_regmap(component);
}
return 0; /* see for_each_component */
list_add(&component->list, &component_list);
err_cleanup: err_cleanup:
snd_soc_del_component_unlocked(component); if (ret < 0)
snd_soc_del_component_unlocked(component);
err_free: err_free:
mutex_unlock(&client_mutex);
if (ret == 0)
snd_soc_try_rebind_card();
return ret; return ret;
} }
EXPORT_SYMBOL_GPL(snd_soc_add_component); EXPORT_SYMBOL_GPL(snd_soc_add_component);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment