Skip to contentSkip to conversation

Make ALSA omap-aic23.c compile again

PATCH In review Dirk Behme 2 participants 2 messages Jan 17, 2006 – Jan 20, 2006 Sound

lore
Share & tools
lore

Dirk Behme

author
#0 lore raw

PATCH 1/1 Make ALSA omap-aic23.c compile again

ARM: OMAP: Remove errors and warnings and make omap-aic23.c compile again:

sound/arm/omap-aic23.c:661: error: structure has no member named `pm_private_data'
sound/arm/omap-aic23.c:680: error: structure has no member named `pm_private_data'
sound/arm/omap-aic23.c:772: warning: implicit declaration of function `clk_get_usecount'
sound/arm/omap-aic23.c:863: warning: implicit declaration of function `snd_card_set_pm_callback'

Note: Needs "Fix clk_get_usecount() warning" patch

Signed-off-by Dirk Behme <dirk.behme_at_de.bosch.com>
sound/arm/omap-aic23.c renamed from sound/arm/omap-aic23.c_orig +5 -5
@@ -59,6 +59,7 @@
 #include <asm/arch/dma.h>
 #include <asm/arch/aic23.h>
 #include <asm/arch/mcbsp.h>
+#include <asm/arch/clock.h>
 
 #include <sound/core.h>
 #include <sound/pcm.h>
@@ -658,7 +659,7 @@ static int __init snd_card_omap_aic23_pc
 
 static int snd_omap_aic23_suspend(snd_card_t * card, pm_message_t state)
 {
-	struct snd_card_omap_aic23 *chip = card->pm_private_data;
+	struct snd_card_omap_aic23 *chip = card->private_data;
 	ADEBUG();
 
 	if (chip->card->power_state != SNDRV_CTL_POWER_D3hot) {
@@ -677,7 +678,7 @@ static int snd_omap_aic23_suspend(snd_ca
  */
 static int snd_omap_aic23_resume(snd_card_t * card)
 {
-	struct snd_card_omap_aic23 *chip = card->pm_private_data;
+	struct snd_card_omap_aic23 *chip = card->private_data;
 	ADEBUG();
 	
 	if (chip->card->power_state != SNDRV_CTL_POWER_D0) {
@@ -860,14 +861,13 @@ static int __init snd_omap_aic23_probe(s
 	if ((err = snd_card_omap_aic23_pcm(omap_aic23, 0)) < 0)
 		goto nodev;
 
-	snd_card_set_pm_callback(card, snd_omap_aic23_suspend,
-				 snd_omap_aic23_resume, omap_aic23);
-
 	strcpy(card->driver, "AIC23");
 	strcpy(card->shortname, "OSK AIC23");
 	sprintf(card->longname, "OMAP OSK with AIC23");
 
 	snd_omap_init_mixer();
+
+	snd_card_set_dev(card, &pdev->dev);
 	
 	if ((err = snd_card_register(card)) == 0) {
 		printk(KERN_INFO "OSK audio support initialized\n");
#1 lore raw
Quoted 53 lines · Dirk Behme

ARM: OMAP: Remove errors and warnings and make omap-aic23.c compile again:

sound/arm/omap-aic23.c:661: error: structure has no member named `pm_private_data'
sound/arm/omap-aic23.c:680: error: structure has no member named `pm_private_data'
sound/arm/omap-aic23.c:772: warning: implicit declaration of function `clk_get_usecount'
sound/arm/omap-aic23.c:863: warning: implicit declaration of function `snd_card_set_pm_callback'

Note: Needs "Fix clk_get_usecount() warning" patch

Signed-off-by Dirk Behme <dirk.behme_at_de.bosch.com>
sound/arm/omap-aic23.c renamed from sound/arm/omap-aic23.c_orig +3 -5
@@ -59,6 +59,7 @@
 #include <asm/arch/dma.h>
 #include <asm/arch/aic23.h>
 #include <asm/arch/mcbsp.h>
+#include <asm/arch/clock.h>
 
 #include <sound/core.h>
 #include <sound/pcm.h>
@@ -658,7 +659,7 @@ static int __init snd_card_omap_aic23_pc
 
 static int snd_omap_aic23_suspend(snd_card_t * card, pm_message_t state)
 {
-	struct snd_card_omap_aic23 *chip = card->pm_private_data;
+	struct snd_card_omap_aic23 *chip = card->private_data;
 	ADEBUG();
 
 	if (chip->card->power_state != SNDRV_CTL_POWER_D3hot) {
@@ -677,7 +678,7 @@ static int snd_omap_aic23_suspend(snd_ca
  */
 static int snd_omap_aic23_resume(snd_card_t * card)
 {
-	struct snd_card_omap_aic23 *chip = card->pm_private_data;
+	struct snd_card_omap_aic23 *chip = card->private_data;
 	ADEBUG();
 
 	if (chip->card->power_state != SNDRV_CTL_POWER_D0) {
@@ -860,14 +861,13 @@ static int __init snd_omap_aic23_probe(s
 	if ((err = snd_card_omap_aic23_pcm(omap_aic23, 0)) < 0)
 		goto nodev;
 
-	snd_card_set_pm_callback(card, snd_omap_aic23_suspend,
-				 snd_omap_aic23_resume, omap_aic23);
-

sound/arm/omap-aic23.c static int __init snd_omap_aic23_probe(s · line 873

Quoted 10 lines
sound/arm/omap-aic23.c renamed from sound/arm/omap-aic23.c_orig +2 -0
 	strcpy(card->driver, "AIC23");
 	strcpy(card->shortname, "OSK AIC23");
 	sprintf(card->longname, "OMAP OSK with AIC23");
 
 	snd_omap_init_mixer();
+
+	snd_card_set_dev(card, &pdev->dev);
 
 	if ((err = snd_card_register(card)) == 0) {
 		printk(KERN_INFO "OSK audio support initialized\n");

And pushing this one too.

Tony