Commit 942de47b authored by Mark Brown's avatar Mark Brown Committed by Russell King

[ARM] 4834/3: Convert ASoC pxa2xx-ac97 driver to use the clock API

Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: default avatarTakashi Iwai <tiwai@suse.de>
Acked-by: default avatareric miao <eric.miao@marvell.com>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 93873fbf
...@@ -15,6 +15,7 @@ ...@@ -15,6 +15,7 @@
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/wait.h> #include <linux/wait.h>
#include <linux/clk.h>
#include <linux/delay.h> #include <linux/delay.h>
#include <sound/core.h> #include <sound/core.h>
...@@ -35,6 +36,10 @@ ...@@ -35,6 +36,10 @@
static DEFINE_MUTEX(car_mutex); static DEFINE_MUTEX(car_mutex);
static DECLARE_WAIT_QUEUE_HEAD(gsr_wq); static DECLARE_WAIT_QUEUE_HEAD(gsr_wq);
static volatile long gsr_bits; static volatile long gsr_bits;
static struct clk *ac97_clk;
#ifdef CONFIG_PXA27x
static struct clk *ac97conf_clk;
#endif
/* /*
* Beware PXA27x bugs: * Beware PXA27x bugs:
...@@ -159,9 +164,9 @@ static void pxa2xx_ac97_cold_reset(struct snd_ac97 *ac97) ...@@ -159,9 +164,9 @@ static void pxa2xx_ac97_cold_reset(struct snd_ac97 *ac97)
gsr_bits = 0; gsr_bits = 0;
#ifdef CONFIG_PXA27x #ifdef CONFIG_PXA27x
/* PXA27x Developers Manual section 13.5.2.2.1 */ /* PXA27x Developers Manual section 13.5.2.2.1 */
pxa_set_cken(CKEN_AC97CONF, 1); clk_enable(ac97conf_clk);
udelay(5); udelay(5);
pxa_set_cken(CKEN_AC97CONF, 0); clk_disable(ac97conf_clk);
GCR = GCR_COLD_RST; GCR = GCR_COLD_RST;
udelay(50); udelay(50);
#else #else
...@@ -255,7 +260,7 @@ static int pxa2xx_ac97_suspend(struct platform_device *pdev, ...@@ -255,7 +260,7 @@ static int pxa2xx_ac97_suspend(struct platform_device *pdev,
struct snd_soc_cpu_dai *dai) struct snd_soc_cpu_dai *dai)
{ {
GCR |= GCR_ACLINK_OFF; GCR |= GCR_ACLINK_OFF;
pxa_set_cken(CKEN_AC97, 0); clk_disable(ac97_clk);
return 0; return 0;
} }
...@@ -270,7 +275,7 @@ static int pxa2xx_ac97_resume(struct platform_device *pdev, ...@@ -270,7 +275,7 @@ static int pxa2xx_ac97_resume(struct platform_device *pdev,
/* Use GPIO 113 as AC97 Reset on Bulverde */ /* Use GPIO 113 as AC97 Reset on Bulverde */
pxa_gpio_mode(113 | GPIO_ALT_FN_2_OUT); pxa_gpio_mode(113 | GPIO_ALT_FN_2_OUT);
#endif #endif
pxa_set_cken(CKEN_AC97, 1); clk_enable(ac97_clk);
return 0; return 0;
} }
...@@ -294,16 +299,32 @@ static int pxa2xx_ac97_probe(struct platform_device *pdev) ...@@ -294,16 +299,32 @@ static int pxa2xx_ac97_probe(struct platform_device *pdev)
#ifdef CONFIG_PXA27x #ifdef CONFIG_PXA27x
/* Use GPIO 113 as AC97 Reset on Bulverde */ /* Use GPIO 113 as AC97 Reset on Bulverde */
pxa_gpio_mode(113 | GPIO_ALT_FN_2_OUT); pxa_gpio_mode(113 | GPIO_ALT_FN_2_OUT);
ac97conf_clk = clk_get(&pdev->dev, "AC97CONFCLK");
if (IS_ERR(ac97conf_clk)) {
ret = PTR_ERR(ac97conf_clk);
ac97conf_clk = NULL;
goto err_irq;
}
#endif #endif
pxa_set_cken(CKEN_AC97, 1); ac97_clk = clk_get(&pdev->dev, "AC97CLK");
if (IS_ERR(ac97_clk)) {
ret = PTR_ERR(ac97_clk);
ac97_clk = NULL;
goto err_irq;
}
return 0; return 0;
err: err_irq:
if (CKEN & (1 << CKEN_AC97)) { GCR |= GCR_ACLINK_OFF;
GCR |= GCR_ACLINK_OFF; #ifdef CONFIG_PXA27x
free_irq(IRQ_AC97, NULL); if (ac97conf_clk) {
pxa_set_cken(CKEN_AC97, 0); clk_put(ac97conf_clk);
ac97conf_clk = NULL;
} }
#endif
free_irq(IRQ_AC97, NULL);
err:
return ret; return ret;
} }
...@@ -311,7 +332,13 @@ static void pxa2xx_ac97_remove(struct platform_device *pdev) ...@@ -311,7 +332,13 @@ static void pxa2xx_ac97_remove(struct platform_device *pdev)
{ {
GCR |= GCR_ACLINK_OFF; GCR |= GCR_ACLINK_OFF;
free_irq(IRQ_AC97, NULL); free_irq(IRQ_AC97, NULL);
pxa_set_cken(CKEN_AC97, 0); #ifdef CONFIG_PXA27x
clk_put(ac97conf_clk);
ac97conf_clk = NULL;
#endif
clk_disable(ac97_clk);
clk_put(ac97_clk);
ac97_clk = NULL;
} }
static int pxa2xx_ac97_hw_params(struct snd_pcm_substream *substream, static int pxa2xx_ac97_hw_params(struct snd_pcm_substream *substream,
......
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