Commit 3b6baa5a authored by Takashi Iwai's avatar Takashi Iwai Committed by Jaroslav Kysela

[ALSA] Remove delayed work properly at free and suspend

Remove delayed work properly at free and suspend in ac97 codec and
ak4114 drivers.
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarJaroslav Kysela <perex@suse.cz>
parent 965ac42c
...@@ -66,6 +66,7 @@ static void snd_ak4114_free(struct ak4114 *chip) ...@@ -66,6 +66,7 @@ static void snd_ak4114_free(struct ak4114 *chip)
{ {
chip->init = 1; /* don't schedule new work */ chip->init = 1; /* don't schedule new work */
mb(); mb();
cancel_delayed_work(&chip->work);
flush_scheduled_work(); flush_scheduled_work();
kfree(chip); kfree(chip);
} }
...@@ -97,6 +98,7 @@ int snd_ak4114_create(struct snd_card *card, ...@@ -97,6 +98,7 @@ int snd_ak4114_create(struct snd_card *card,
chip->read = read; chip->read = read;
chip->write = write; chip->write = write;
chip->private_data = private_data; chip->private_data = private_data;
INIT_DELAYED_WORK(&chip->work, ak4114_stats);
for (reg = 0; reg < 7; reg++) for (reg = 0; reg < 7; reg++)
chip->regmap[reg] = pgm[reg]; chip->regmap[reg] = pgm[reg];
...@@ -149,7 +151,6 @@ void snd_ak4114_reinit(struct ak4114 *chip) ...@@ -149,7 +151,6 @@ void snd_ak4114_reinit(struct ak4114 *chip)
reg_write(chip, AK4114_REG_PWRDN, old | AK4114_RST | AK4114_PWN); reg_write(chip, AK4114_REG_PWRDN, old | AK4114_RST | AK4114_PWN);
/* bring up statistics / event queing */ /* bring up statistics / event queing */
chip->init = 0; chip->init = 0;
INIT_DELAYED_WORK(&chip->work, ak4114_stats);
schedule_delayed_work(&chip->work, HZ / 10); schedule_delayed_work(&chip->work, HZ / 10);
} }
......
...@@ -990,6 +990,7 @@ static int snd_ac97_free(struct snd_ac97 *ac97) ...@@ -990,6 +990,7 @@ static int snd_ac97_free(struct snd_ac97 *ac97)
if (ac97) { if (ac97) {
#ifdef CONFIG_SND_AC97_POWER_SAVE #ifdef CONFIG_SND_AC97_POWER_SAVE
cancel_delayed_work(&ac97->power_work); cancel_delayed_work(&ac97->power_work);
flush_scheduled_work();
#endif #endif
snd_ac97_proc_done(ac97); snd_ac97_proc_done(ac97);
if (ac97->bus) if (ac97->bus)
...@@ -2415,6 +2416,10 @@ void snd_ac97_suspend(struct snd_ac97 *ac97) ...@@ -2415,6 +2416,10 @@ void snd_ac97_suspend(struct snd_ac97 *ac97)
return; return;
if (ac97->build_ops->suspend) if (ac97->build_ops->suspend)
ac97->build_ops->suspend(ac97); ac97->build_ops->suspend(ac97);
#ifdef CONFIG_SND_AC97_POWER_SAVE
cancel_delayed_work(&ac97->power_work);
flush_scheduled_work();
#endif
snd_ac97_powerdown(ac97); snd_ac97_powerdown(ac97);
} }
......
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