Commit dadfb986 authored by Takashi Iwai's avatar Takashi Iwai

ALSA: ak4531: Fix assignment in if condition

AK4531 codec driver code contains a few assignments in if condition,
which is a bad coding style that may confuse readers and occasionally
lead to bugs.

This patch is merely for coding-style fixes, no functional changes.

Link: https://lore.kernel.org/r/20210608140540.17885-17-tiwai@suse.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent e2e30a32
...@@ -384,7 +384,8 @@ int snd_ak4531_mixer(struct snd_card *card, ...@@ -384,7 +384,8 @@ int snd_ak4531_mixer(struct snd_card *card,
return -ENOMEM; return -ENOMEM;
*ak4531 = *_ak4531; *ak4531 = *_ak4531;
mutex_init(&ak4531->reg_mutex); mutex_init(&ak4531->reg_mutex);
if ((err = snd_component_add(card, "AK4531")) < 0) { err = snd_component_add(card, "AK4531");
if (err < 0) {
snd_ak4531_free(ak4531); snd_ak4531_free(ak4531);
return err; return err;
} }
...@@ -398,13 +399,15 @@ int snd_ak4531_mixer(struct snd_card *card, ...@@ -398,13 +399,15 @@ int snd_ak4531_mixer(struct snd_card *card,
ak4531->write(ak4531, idx, ak4531->regs[idx] = snd_ak4531_initial_map[idx]); /* recording source is mixer */ ak4531->write(ak4531, idx, ak4531->regs[idx] = snd_ak4531_initial_map[idx]); /* recording source is mixer */
} }
for (idx = 0; idx < ARRAY_SIZE(snd_ak4531_controls); idx++) { for (idx = 0; idx < ARRAY_SIZE(snd_ak4531_controls); idx++) {
if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_ak4531_controls[idx], ak4531))) < 0) { err = snd_ctl_add(card, snd_ctl_new1(&snd_ak4531_controls[idx], ak4531));
if (err < 0) {
snd_ak4531_free(ak4531); snd_ak4531_free(ak4531);
return err; return err;
} }
} }
snd_ak4531_proc_init(card, ak4531); snd_ak4531_proc_init(card, ak4531);
if ((err = snd_device_new(card, SNDRV_DEV_CODEC, ak4531, &ops)) < 0) { err = snd_device_new(card, SNDRV_DEV_CODEC, ak4531, &ops);
if (err < 0) {
snd_ak4531_free(ak4531); snd_ak4531_free(ak4531);
return err; return err;
} }
......
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