mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2024-12-28 16:53:49 +00:00
ASoC: soc-card: Use snd_ctl_find_id_mixer() instead of open-coding
Use the snd_ctl_find_id_mixer[_locked]() wrapper in snd_soc_card_get_kcontrol[_locked]() instead of open-coding a custom list walk of the card controls list. Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com> Reviewed-by: Takashi Iwai <tiwai@suse.de> Link: https://msgid.link/r/20240401100210.61277-3-rf@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
08ea486a61
commit
897cc72b08
@ -32,33 +32,20 @@ static inline int _soc_card_ret(struct snd_soc_card *card,
|
||||
struct snd_kcontrol *snd_soc_card_get_kcontrol_locked(struct snd_soc_card *soc_card,
|
||||
const char *name)
|
||||
{
|
||||
struct snd_card *card = soc_card->snd_card;
|
||||
struct snd_kcontrol *kctl;
|
||||
|
||||
/* must be held read or write */
|
||||
lockdep_assert_held(&card->controls_rwsem);
|
||||
|
||||
if (unlikely(!name))
|
||||
return NULL;
|
||||
|
||||
list_for_each_entry(kctl, &card->controls, list)
|
||||
if (!strncmp(kctl->id.name, name, sizeof(kctl->id.name)))
|
||||
return kctl;
|
||||
return NULL;
|
||||
return snd_ctl_find_id_mixer_locked(soc_card->snd_card, name);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_card_get_kcontrol_locked);
|
||||
|
||||
struct snd_kcontrol *snd_soc_card_get_kcontrol(struct snd_soc_card *soc_card,
|
||||
const char *name)
|
||||
{
|
||||
struct snd_card *card = soc_card->snd_card;
|
||||
struct snd_kcontrol *kctl;
|
||||
if (unlikely(!name))
|
||||
return NULL;
|
||||
|
||||
down_read(&card->controls_rwsem);
|
||||
kctl = snd_soc_card_get_kcontrol_locked(soc_card, name);
|
||||
up_read(&card->controls_rwsem);
|
||||
|
||||
return kctl;
|
||||
return snd_ctl_find_id_mixer(soc_card->snd_card, name);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(snd_soc_card_get_kcontrol);
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user