[ALSA] Fix PM support

Modules: ARM AACI PL041 driver,ARM PXA2XX driver

Fix PM support of aaci and pxa2xx drivers.

Signed-off-by: Takashi Iwai <tiwai@suse.de>

2 files changed, 19 insertions, 29 deletions

diff --git a/sound/arm/aaci.c b/sound/arm/aaci.c
index 691f6dd81c30..5e9a81ab990b 100644
--- a/sound/arm/aaci.c
+++ b/sound/arm/aaci.c
@@ -635,19 +635,14 @@ static struct snd_pcm_ops aaci_playback_ops = {
 static int aaci_do_suspend(struct snd_card *card, unsigned int state)
 {
         struct aaci *aaci = card->private_data;
-        if (aaci->card->power_state != SNDRV_CTL_POWER_D3cold) {
-                snd_pcm_suspend_all(aaci->pcm);
-                snd_power_change_state(aaci->card, SNDRV_CTL_POWER_D3cold);
-        }
+        snd_power_change_state(card, SNDRV_CTL_POWER_D3cold);
+        snd_pcm_suspend_all(aaci->pcm);
         return 0;
 }
 
 static int aaci_do_resume(struct snd_card *card, unsigned int state)
 {
-        struct aaci *aaci = card->private_data;
-        if (aaci->card->power_state != SNDRV_CTL_POWER_D0) {
-                snd_power_change_state(aaci->card, SNDRV_CTL_POWER_D0);
-        }
+        snd_power_change_state(card, SNDRV_CTL_POWER_D0);
         return 0;
 }
 
@@ -780,7 +775,6 @@ static struct aaci * __devinit aaci_init_card(struct amba_device *dev)
                 return ERR_PTR(-ENOMEM);
 
         card->private_free = aaci_free_card;
-        snd_card_set_pm_callback(card, aaci_do_suspend, aaci_do_resume, NULL);
 
         strlcpy(card->driver, DRIVER_NAME, sizeof(card->driver));
         strlcpy(card->shortname, "ARM AC'97 Interface", sizeof(card->shortname));
diff --git a/sound/arm/pxa2xx-ac97.c b/sound/arm/pxa2xx-ac97.c
index d9efc37393ee..dda64beb202b 100644
--- a/sound/arm/pxa2xx-ac97.c
+++ b/sound/arm/pxa2xx-ac97.c
@@ -247,30 +247,28 @@ static struct pxa2xx_pcm_client pxa2xx_ac97_pcm_client = {
 
 static int pxa2xx_ac97_do_suspend(struct snd_card *card, pm_message_t state)
 {
-        if (card->power_state != SNDRV_CTL_POWER_D3cold) {
-                pxa2xx_audio_ops_t *platform_ops = card->dev->platform_data;
-                snd_pcm_suspend_all(pxa2xx_ac97_pcm);
-                snd_ac97_suspend(pxa2xx_ac97_ac97);
-                snd_power_change_state(card, SNDRV_CTL_POWER_D3cold);
-                if (platform_ops && platform_ops->suspend)
-                        platform_ops->suspend(platform_ops->priv);
-                GCR |= GCR_ACLINK_OFF;
-                pxa_set_cken(CKEN2_AC97, 0);
-        }
+        pxa2xx_audio_ops_t *platform_ops = card->dev->platform_data;
+
+        snd_power_change_state(card, SNDRV_CTL_POWER_D3cold);
+        snd_pcm_suspend_all(pxa2xx_ac97_pcm);
+        snd_ac97_suspend(pxa2xx_ac97_ac97);
+        if (platform_ops && platform_ops->suspend)
+                platform_ops->suspend(platform_ops->priv);
+        GCR |= GCR_ACLINK_OFF;
+        pxa_set_cken(CKEN2_AC97, 0);
 
         return 0;
 }
 
 static int pxa2xx_ac97_do_resume(struct snd_card *card)
 {
-        if (card->power_state != SNDRV_CTL_POWER_D0) {
-                pxa2xx_audio_ops_t *platform_ops = card->dev->platform_data;
-                pxa_set_cken(CKEN2_AC97, 1);
-                if (platform_ops && platform_ops->resume)
-                        platform_ops->resume(platform_ops->priv);
-                snd_ac97_resume(pxa2xx_ac97_ac97);
-                snd_power_change_state(card, SNDRV_CTL_POWER_D0);
-        }
+        pxa2xx_audio_ops_t *platform_ops = card->dev->platform_data;
+
+        pxa_set_cken(CKEN2_AC97, 1);
+        if (platform_ops && platform_ops->resume)
+                platform_ops->resume(platform_ops->priv);
+        snd_ac97_resume(pxa2xx_ac97_ac97);
+        snd_power_change_state(card, SNDRV_CTL_POWER_D0);
 
         return 0;
 }
@@ -349,8 +347,6 @@ static int pxa2xx_ac97_probe(struct platform_device *dev)
         snprintf(card->longname, sizeof(card->longname),
                  "%s (%s)", dev->dev.driver->name, card->mixername);
 
-        snd_card_set_pm_callback(card, pxa2xx_ac97_do_suspend,
-                                 pxa2xx_ac97_do_resume, NULL);
         ret = snd_card_register(card);
         if (ret == 0) {
                 platform_set_drvdata(dev, card);