summaryrefslogtreecommitdiffstats
path: root/sound/soc/imx
diff options
context:
space:
mode:
authorjavier Martin <javier.martin@vista-silicon.com>2009-08-05 22:29:39 +0100
committerMark Brown <broonie@opensource.wolfsonmicro.com>2009-08-05 22:31:55 +0100
commitfbb474dedab871c7382858c374acd287dad4ab38 (patch)
treeed8f33421582b930889f613862285e277ffbdb01 /sound/soc/imx
parent2ccafed43a75b836a15bd36ef0acf466ba36604d (diff)
downloadop-kernel-dev-fbb474dedab871c7382858c374acd287dad4ab38.zip
op-kernel-dev-fbb474dedab871c7382858c374acd287dad4ab38.tar.gz
ASoC: Fix review issues in i.MX2x PCM driver
Signed-off-by: javier Martin <javier.martin@vista-silicon.com> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/imx')
-rw-r--r--sound/soc/imx/mx1_mx2-pcm.c13
-rw-r--r--sound/soc/imx/mx1_mx2-pcm.h26
2 files changed, 9 insertions, 30 deletions
diff --git a/sound/soc/imx/mx1_mx2-pcm.c b/sound/soc/imx/mx1_mx2-pcm.c
index 94807f8..16efd8a 100644
--- a/sound/soc/imx/mx1_mx2-pcm.c
+++ b/sound/soc/imx/mx1_mx2-pcm.c
@@ -126,7 +126,7 @@ static int dma_new_period(struct snd_pcm_substream *substream)
dma_size, dev_addr,
prtd->dma_params->transfer_type);
if (ret < 0) {
- printk(KERN_ERR "Error configuring DMA\n");
+ printk(KERN_ERR "Error %d configuring DMA\n", ret);
return ret;
}
imx_dma_enable(prtd->dma_ch);
@@ -216,7 +216,8 @@ static int mx1_mx2_pcm_hw_params(struct snd_pcm_substream *substream,
ret = snd_pcm_lib_malloc_pages(substream,
params_buffer_bytes(hw_params));
if (ret < 0) {
- printk(KERN_ERR "%s: failed to malloc pcm pages\n", __func__);
+ printk(KERN_ERR "%s: Error %d failed to malloc pcm pages \n",
+ __func__, ret);
return ret;
}
@@ -324,7 +325,7 @@ static int mx1_mx2_pcm_open(struct snd_pcm_substream *substream)
prtd->dma_ch = imx_dma_request_by_prio(prtd->dma_params->name,
DMA_PRIO_HIGH);
if (prtd->dma_ch < 0) {
- printk(KERN_ERR "Error requesting dma channel\n");
+ printk(KERN_ERR "Error %d requesting dma channel\n", ret);
return ret;
}
imx_dma_config_burstlen(prtd->dma_ch,
@@ -336,8 +337,8 @@ static int mx1_mx2_pcm_open(struct snd_pcm_substream *substream)
prtd->dma_params->event_id, 0);
if (ret) {
- pr_debug(KERN_ERR "Error configuring dma channel %d\n",
- prtd->dma_ch);
+ pr_debug(KERN_ERR "Error %d configuring dma channel %d\n",
+ ret, prtd->dma_ch);
return ret;
}
@@ -346,7 +347,7 @@ static int mx1_mx2_pcm_open(struct snd_pcm_substream *substream)
audio_dma_irq, NULL,
(void *)substream);
if (ret < 0) {
- printk(KERN_ERR "Error setting dma callback function\n");
+ printk(KERN_ERR "Error %d setting dma callback function\n", ret);
return ret;
}
return 0;
diff --git a/sound/soc/imx/mx1_mx2-pcm.h b/sound/soc/imx/mx1_mx2-pcm.h
index e1e3a3f..2e52810 100644
--- a/sound/soc/imx/mx1_mx2-pcm.h
+++ b/sound/soc/imx/mx1_mx2-pcm.h
@@ -6,30 +6,8 @@
* published by the Free Software Foundation.
*/
-#ifndef _MXC_PCM_H
-#define _MXC_PCM_H
-
-/* AUDMUX register definitions */
-#define AUDMUX_IO_BASE_ADDR IO_ADDRESS(AUDMUX_BASE_ADDR)
-
-#define DAM_HPCR1 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x00)))
-#define DAM_HPCR2 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x04)))
-#define DAM_HPCR3 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x08)))
-#define DAM_PPCR1 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x10)))
-#define DAM_PPCR2 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x14)))
-#define DAM_PPCR3 (*((volatile u32 *)(AUDMUX_IO_BASE_ADDR + 0x1C)))
-
-#define AUDMUX_HPCR_TFSDIR (1 << 31)
-#define AUDMUX_HPCR_TCLKDIR (1 << 30)
-#define AUDMUX_HPCR_TFCSEL(x) (((x) & 0xff) << 26)
-#define AUDMUX_HPCR_RXDSEL(x) (((x) & 0x7) << 13)
-#define AUDMUX_HPCR_SYN (1 << 12)
-
-#define AUDMUX_PPCR_TFSDIR (1 << 31)
-#define AUDMUX_PPCR_TCLKDIR (1 << 30)
-#define AUDMUX_PPCR_TFCSEL(x) (((x) & 0xff) << 26)
-#define AUDMUX_PPCR_RXDSEL(x) (((x) & 0x7) << 13)
-#define AUDMUX_PPCR_SYN (1 << 12)
+#ifndef _MX1_MX2_PCM_H
+#define _MX1_MX2_PCM_H
/* DMA information for mx1_mx2 platforms */
struct mx1_mx2_pcm_dma_params {
OpenPOWER on IntegriCloud