mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-27 19:29:37 +00:00
ASoC: component: Remove misplaced prefix handling in pin control functions
[ Upstream commit31428c7874
] When the component level pin control functions were added they for some no longer obvious reason handled adding prefixing of widget names. This meant that when the lack of prefix handling in the DAPM level pin operations was fixed byae4fc53224
(ASoC: dapm: use component prefix when checking widget names) the one device using the component level API ended up with the prefix being applied twice, causing all lookups to fail. Fix this by removing the redundant prefixing from the component code, which has the nice side effect of also making that code much simpler. Reported-by: Richard Fitzgerald <rf@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> Tested-by: Lucas Tanure <tanureal@opensource.cirrus.com> Link: https://lore.kernel.org/r/20210726194123.54585-1-broonie@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
34cc80ec12
commit
0af6a9f82c
1 changed files with 27 additions and 36 deletions
|
@ -135,86 +135,75 @@ int snd_soc_component_set_bias_level(struct snd_soc_component *component,
|
|||
return soc_component_ret(component, ret);
|
||||
}
|
||||
|
||||
static int soc_component_pin(struct snd_soc_component *component,
|
||||
const char *pin,
|
||||
int (*pin_func)(struct snd_soc_dapm_context *dapm,
|
||||
const char *pin))
|
||||
{
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
char *full_name;
|
||||
int ret;
|
||||
|
||||
if (!component->name_prefix) {
|
||||
ret = pin_func(dapm, pin);
|
||||
goto end;
|
||||
}
|
||||
|
||||
full_name = kasprintf(GFP_KERNEL, "%s %s", component->name_prefix, pin);
|
||||
if (!full_name) {
|
||||
ret = -ENOMEM;
|
||||
goto end;
|
||||
}
|
||||
|
||||
ret = pin_func(dapm, full_name);
|
||||
kfree(full_name);
|
||||
end:
|
||||
return soc_component_ret(component, ret);
|
||||
}
|
||||
|
||||
int snd_soc_component_enable_pin(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_enable_pin);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_enable_pin(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_enable_pin);
|
||||
|
||||
int snd_soc_component_enable_pin_unlocked(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_enable_pin_unlocked);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_enable_pin_unlocked(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_enable_pin_unlocked);
|
||||
|
||||
int snd_soc_component_disable_pin(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_disable_pin);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_disable_pin(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_disable_pin);
|
||||
|
||||
int snd_soc_component_disable_pin_unlocked(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_disable_pin_unlocked);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_disable_pin_unlocked(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_disable_pin_unlocked);
|
||||
|
||||
int snd_soc_component_nc_pin(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_nc_pin);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_nc_pin(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_nc_pin);
|
||||
|
||||
int snd_soc_component_nc_pin_unlocked(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_nc_pin_unlocked);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_nc_pin_unlocked(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_nc_pin_unlocked);
|
||||
|
||||
int snd_soc_component_get_pin_status(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_get_pin_status);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_get_pin_status(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_get_pin_status);
|
||||
|
||||
int snd_soc_component_force_enable_pin(struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_force_enable_pin);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_force_enable_pin(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_force_enable_pin);
|
||||
|
||||
|
@ -222,7 +211,9 @@ int snd_soc_component_force_enable_pin_unlocked(
|
|||
struct snd_soc_component *component,
|
||||
const char *pin)
|
||||
{
|
||||
return soc_component_pin(component, pin, snd_soc_dapm_force_enable_pin_unlocked);
|
||||
struct snd_soc_dapm_context *dapm =
|
||||
snd_soc_component_get_dapm(component);
|
||||
return snd_soc_dapm_force_enable_pin_unlocked(dapm, pin);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_component_force_enable_pin_unlocked);
|
||||
|
||||
|
|
Loading…
Reference in a new issue