ALSA: ca0106: Use the standard snd_ctl_add_followers() helper

Instead of open-code, use the new standard helper to manage vmaster
stuff for code simplification.

Also, handle the errors from the helper more properly instead of
silently ignoring.

Link: https://lore.kernel.org/r/20230721071643.3631-4-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2023-07-21 09:16:42 +02:00
parent 1caf64d91f
commit b7bb11fa36

View File

@ -751,17 +751,6 @@ static const char * const follower_sws[] = {
NULL
};
static void add_followers(struct snd_card *card,
struct snd_kcontrol *master, const char * const *list)
{
for (; *list; list++) {
struct snd_kcontrol *follower =
snd_ctl_find_id_mixer(card, *list);
if (follower)
snd_ctl_add_follower(master, follower);
}
}
int snd_ca0106_mixer(struct snd_ca0106 *emu)
{
int err;
@ -843,7 +832,9 @@ int snd_ca0106_mixer(struct snd_ca0106 *emu)
err = snd_ctl_add(card, vmaster);
if (err < 0)
return err;
add_followers(card, vmaster, follower_vols);
err = snd_ctl_add_followers(card, vmaster, follower_vols);
if (err < 0)
return err;
if (emu->details->spi_dac) {
vmaster = snd_ctl_make_virtual_master("Master Playback Switch",
@ -853,7 +844,9 @@ int snd_ca0106_mixer(struct snd_ca0106 *emu)
err = snd_ctl_add(card, vmaster);
if (err < 0)
return err;
add_followers(card, vmaster, follower_sws);
err = snd_ctl_add_followers(card, vmaster, follower_sws);
if (err < 0)
return err;
}
strcpy(card->mixername, "CA0106");