ASoC: Fix and cleaup dma config

SVN-Revision: 21618

diff --git a/target/linux/xburst/patches-2.6.34/061-asoc.patch b/target/linux/xburst/patches-2.6.34/061-asoc.patch
index b7e8aa7f0f41a82c501cc202e28c40b47b52f2b6..9cabcec1db92d06cc2a3228df1f2cbd9d7fec3dc 100644 (file)
--- a/target/linux/xburst/patches-2.6.34/061-asoc.patch
+++ b/target/linux/xburst/patches-2.6.34/061-asoc.patch
@@ -1,6 +1,6 @@
-From 6548719b6a1ce4e202d58625fbfb94e6c894ced9 Mon Sep 17 00:00:00 2001
+From d12d1c5f8615192b545faa69a553f64d4d9cc9ab Mon Sep 17 00:00:00 2001

 From: Lars-Peter Clausen <lars@metafoo.de>
 From: Lars-Peter Clausen <lars@metafoo.de>

-Date: Sat, 24 Apr 2010 12:35:07 +0200
+Date: Fri, 28 May 2010 19:54:02 +0200

 Subject: [PATCH] Add jz4740 SoC sound drivers
 
 ---
 Subject: [PATCH] Add jz4740 SoC sound drivers
 
 ---


@@ -8,11 +8,11 @@ Subject: [PATCH] Add jz4740 SoC sound drivers
  sound/soc/Makefile            |    1 +
  sound/soc/jz4740/Kconfig      |   13 +
  sound/soc/jz4740/Makefile     |    9 +
  sound/soc/Makefile            |    1 +
  sound/soc/jz4740/Kconfig      |   13 +
  sound/soc/jz4740/Makefile     |    9 +

- sound/soc/jz4740/jz4740-i2s.c |  572 +++++++++++++++++++++++++++++++++++++++++
+ sound/soc/jz4740/jz4740-i2s.c |  568 +++++++++++++++++++++++++++++++++++++++++

  sound/soc/jz4740/jz4740-i2s.h |   18 ++
  sound/soc/jz4740/jz4740-pcm.c |  350 +++++++++++++++++++++++++
  sound/soc/jz4740/jz4740-pcm.h |   22 ++
  sound/soc/jz4740/jz4740-i2s.h |   18 ++
  sound/soc/jz4740/jz4740-pcm.c |  350 +++++++++++++++++++++++++
  sound/soc/jz4740/jz4740-pcm.h |   22 ++

- 8 files changed, 986 insertions(+), 0 deletions(-)
+ 8 files changed, 982 insertions(+), 0 deletions(-)

  create mode 100644 sound/soc/jz4740/Kconfig
  create mode 100644 sound/soc/jz4740/Makefile
  create mode 100644 sound/soc/jz4740/jz4740-i2s.c
  create mode 100644 sound/soc/jz4740/Kconfig
  create mode 100644 sound/soc/jz4740/Makefile
  create mode 100644 sound/soc/jz4740/jz4740-i2s.c


@@ -77,10 +77,10 @@ index 0000000..1be8d19
 +
 diff --git a/sound/soc/jz4740/jz4740-i2s.c b/sound/soc/jz4740/jz4740-i2s.c
 new file mode 100644
 +
 diff --git a/sound/soc/jz4740/jz4740-i2s.c b/sound/soc/jz4740/jz4740-i2s.c
 new file mode 100644

-index 0000000..d3daa27
+index 0000000..2b139fd

 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-i2s.c
 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-i2s.c

-@@ -0,0 +1,572 @@
+@@ -0,0 +1,568 @@

 +/*
 + *  Copyright (C) 2010, Lars-Peter Clausen <lars@metafoo.de>
 + *
 +/*
 + *  Copyright (C) 2010, Lars-Peter Clausen <lars@metafoo.de>
 + *


@@ -173,37 +173,18 @@ index 0000000..d3daa27
 +      struct clk *clk_aic;
 +      struct clk *clk_i2s;
 +
 +      struct clk *clk_aic;
 +      struct clk *clk_i2s;
 +

-+      struct jz4740_pcm_config capture_pcm_config;
-+      struct jz4740_pcm_config playback_pcm_config;
++      struct jz4740_pcm_config pcm_config_playback;
++      struct jz4740_pcm_config pcm_config_capture;

 +};
 +
 +};
 +

-+static struct jz4740_dma_config jz4740_i2s_dma_playback_config = {
-+      .src_width = JZ4740_DMA_WIDTH_16BIT,
-+      .dst_width = JZ4740_DMA_WIDTH_32BIT,
-+      .transfer_size = JZ4740_DMA_TRANSFER_SIZE_16BYTE,
-+      .request_type = JZ4740_DMA_TYPE_AIC_TRANSMIT,
-+      .flags = JZ4740_DMA_SRC_AUTOINC,
-+      .mode = JZ4740_DMA_MODE_SINGLE,
-+};
-+
-+static struct jz4740_dma_config jz4740_i2s_dma_capture_config = {
-+      .src_width = JZ4740_DMA_WIDTH_32BIT,
-+      .dst_width = JZ4740_DMA_WIDTH_16BIT,
-+      .transfer_size = JZ4740_DMA_TRANSFER_SIZE_16BYTE,
-+      .request_type = JZ4740_DMA_TYPE_AIC_RECEIVE,
-+      .flags = JZ4740_DMA_DST_AUTOINC,
-+      .mode = JZ4740_DMA_MODE_SINGLE,
-+};
-+
-+

 +static inline uint32_t jz4740_i2s_read(const struct jz4740_i2s *i2s,
 +static inline uint32_t jz4740_i2s_read(const struct jz4740_i2s *i2s,

-+              unsigned int reg)
++      unsigned int reg)

 +{
 +      return readl(i2s->base + reg);
 +}
 +
 +static inline void jz4740_i2s_write(const struct jz4740_i2s *i2s,
 +{
 +      return readl(i2s->base + reg);
 +}
 +
 +static inline void jz4740_i2s_write(const struct jz4740_i2s *i2s,

-+              unsigned int reg, uint32_t value)
++      unsigned int reg, uint32_t value)

 +{
 +      writel(value, i2s->base + reg);
 +}
 +{
 +      writel(value, i2s->base + reg);
 +}


@@ -214,7 +195,7 @@ index 0000000..d3daa27
 +}
 +
 +static int jz4740_i2s_startup(struct snd_pcm_substream *substream,
 +}
 +
 +static int jz4740_i2s_startup(struct snd_pcm_substream *substream,

-+              struct snd_soc_dai *dai)
++      struct snd_soc_dai *dai)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      uint32_t conf, ctrl;
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      uint32_t conf, ctrl;


@@ -222,22 +203,22 @@ index 0000000..d3daa27
 +      if (dai->active)
 +              return 0;
 +
 +      if (dai->active)
 +              return 0;
 +

-+      conf = jz4740_i2s_read(i2s, JZ_REG_AIC_CONF);
-+      ctrl = jz4740_i2s_read(i2s, JZ_REG_AIC_CTRL);

 +
 +

-+      conf |= JZ_AIC_CONF_ENABLE;
++      ctrl = jz4740_i2s_read(i2s, JZ_REG_AIC_CTRL);

 +      ctrl |= JZ_AIC_CTRL_FLUSH;
 +      ctrl |= JZ_AIC_CTRL_FLUSH;

-+
-+

 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);

++

 +      clk_enable(i2s->clk_i2s);
 +      clk_enable(i2s->clk_i2s);

++
++      conf = jz4740_i2s_read(i2s, JZ_REG_AIC_CONF);
++      conf |= JZ_AIC_CONF_ENABLE;

 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CONF, conf);
 +
 +      return 0;
 +}
 +
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CONF, conf);
 +
 +      return 0;
 +}
 +

-+static void jz4740_i2s_shutdown(struct snd_pcm_substream *substream, struct
-+                              snd_soc_dai *dai)
++static void jz4740_i2s_shutdown(struct snd_pcm_substream *substream,
++      struct snd_soc_dai *dai)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      uint32_t conf;
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      uint32_t conf;


@@ -254,7 +235,7 @@ index 0000000..d3daa27
 +
 +
 +static int jz4740_i2s_trigger(struct snd_pcm_substream *substream, int cmd,
 +
 +
 +static int jz4740_i2s_trigger(struct snd_pcm_substream *substream, int cmd,

-+                             struct snd_soc_dai *dai)
++      struct snd_soc_dai *dai)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      bool playback = (substream->stream == SNDRV_PCM_STREAM_PLAYBACK);
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      bool playback = (substream->stream == SNDRV_PCM_STREAM_PLAYBACK);


@@ -262,11 +243,10 @@ index 0000000..d3daa27
 +      uint32_t ctrl;
 +      uint32_t mask;
 +
 +      uint32_t ctrl;
 +      uint32_t mask;
 +

-+      if (playback) {
-+          mask = JZ_AIC_CTRL_ENABLE_PLAYBACK | JZ_AIC_CTRL_ENABLE_TX_DMA;
-+      } else {
-+          mask = JZ_AIC_CTRL_ENABLE_CAPTURE | JZ_AIC_CTRL_ENABLE_RX_DMA;
-+      }
++      if (playback)
++              mask = JZ_AIC_CTRL_ENABLE_PLAYBACK | JZ_AIC_CTRL_ENABLE_TX_DMA;
++      else
++              mask = JZ_AIC_CTRL_ENABLE_CAPTURE | JZ_AIC_CTRL_ENABLE_RX_DMA;

 +
 +      ctrl = jz4740_i2s_read(i2s, JZ_REG_AIC_CTRL);
 +
 +
 +      ctrl = jz4740_i2s_read(i2s, JZ_REG_AIC_CTRL);
 +


@@ -274,15 +254,15 @@ index 0000000..d3daa27
 +      case SNDRV_PCM_TRIGGER_START:
 +      case SNDRV_PCM_TRIGGER_RESUME:
 +      case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
 +      case SNDRV_PCM_TRIGGER_START:
 +      case SNDRV_PCM_TRIGGER_RESUME:
 +      case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:

-+          ctrl |= mask;
-+          break;
++              ctrl |= mask;
++              break;

 +      case SNDRV_PCM_TRIGGER_STOP:
 +      case SNDRV_PCM_TRIGGER_SUSPEND:
 +      case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
 +      case SNDRV_PCM_TRIGGER_STOP:
 +      case SNDRV_PCM_TRIGGER_SUSPEND:
 +      case SNDRV_PCM_TRIGGER_PAUSE_PUSH:

-+          ctrl &= ~mask;
-+          break;
++              ctrl &= ~mask;
++              break;

 +      default:
 +      default:

-+          return -EINVAL;
++              return -EINVAL;

 +      }
 +
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);
 +      }
 +
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);


@@ -291,8 +271,7 @@ index 0000000..d3daa27
 +}
 +
 +
 +}
 +
 +

-+static int jz4740_i2s_set_fmt(struct snd_soc_dai *dai,
-+                             unsigned int fmt)
++static int jz4740_i2s_set_fmt(struct snd_soc_dai *dai, unsigned int fmt)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +


@@ -305,36 +284,36 @@ index 0000000..d3daa27
 +
 +      switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) {
 +      case SND_SOC_DAIFMT_CBS_CFS:
 +
 +      switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) {
 +      case SND_SOC_DAIFMT_CBS_CFS:

-+          conf |= JZ_AIC_CONF_BIT_CLK_MASTER | JZ_AIC_CONF_SYNC_CLK_MASTER;
-+          format |= JZ_AIC_I2S_FMT_ENABLE_SYS_CLK;
-+          break;
++              conf |= JZ_AIC_CONF_BIT_CLK_MASTER | JZ_AIC_CONF_SYNC_CLK_MASTER;
++              format |= JZ_AIC_I2S_FMT_ENABLE_SYS_CLK;
++              break;

 +      case SND_SOC_DAIFMT_CBM_CFS:
 +      case SND_SOC_DAIFMT_CBM_CFS:

-+          conf |= JZ_AIC_CONF_SYNC_CLK_MASTER;
-+          break;
++              conf |= JZ_AIC_CONF_SYNC_CLK_MASTER;
++              break;

 +      case SND_SOC_DAIFMT_CBS_CFM:
 +      case SND_SOC_DAIFMT_CBS_CFM:

-+          conf |= JZ_AIC_CONF_BIT_CLK_MASTER;
-+          break;
++              conf |= JZ_AIC_CONF_BIT_CLK_MASTER;
++              break;

 +      case SND_SOC_DAIFMT_CBM_CFM:
 +      case SND_SOC_DAIFMT_CBM_CFM:

-+          break;
++              break;

 +      default:
 +      default:

-+          return -EINVAL;
++              return -EINVAL;

 +      }
 +
 +      switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) {
 +      case SND_SOC_DAIFMT_MSB:
 +      }
 +
 +      switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) {
 +      case SND_SOC_DAIFMT_MSB:

-+          format |= JZ_AIC_I2S_FMT_MSB;
-+          break;
++              format |= JZ_AIC_I2S_FMT_MSB;
++              break;

 +      case SND_SOC_DAIFMT_I2S:
 +      case SND_SOC_DAIFMT_I2S:

-+          break;
++              break;

 +      default:
 +      default:

-+          return -EINVAL;
++              return -EINVAL;

 +      }
 +
 +      switch (fmt & SND_SOC_DAIFMT_INV_MASK) {
 +      case SND_SOC_DAIFMT_NB_NF:
 +      }
 +
 +      switch (fmt & SND_SOC_DAIFMT_INV_MASK) {
 +      case SND_SOC_DAIFMT_NB_NF:

-+          break;
++              break;

 +      default:
 +      default:

-+          return -EINVAL;
++              return -EINVAL;

 +      }
 +
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CONF, conf);
 +      }
 +
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CONF, conf);


@@ -344,8 +323,7 @@ index 0000000..d3daa27
 +}
 +
 +static int jz4740_i2s_hw_params(struct snd_pcm_substream *substream,
 +}
 +
 +static int jz4740_i2s_hw_params(struct snd_pcm_substream *substream,

-+                               struct snd_pcm_hw_params *params,
-+                               struct snd_soc_dai *dai)
++      struct snd_pcm_hw_params *params, struct snd_soc_dai *dai)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      bool playback = (substream->stream == SNDRV_PCM_STREAM_PLAYBACK);
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      bool playback = (substream->stream == SNDRV_PCM_STREAM_PLAYBACK);


@@ -358,11 +336,11 @@ index 0000000..d3daa27
 +
 +      switch (params_format(params)) {
 +      case SNDRV_PCM_FORMAT_S8:
 +
 +      switch (params_format(params)) {
 +      case SNDRV_PCM_FORMAT_S8:

-+          sample_size = 0;
++              sample_size = 0;

 +              dma_width = JZ4740_DMA_WIDTH_8BIT;
 +              dma_width = JZ4740_DMA_WIDTH_8BIT;

-+          break;
++              break;

 +      case SNDRV_PCM_FORMAT_S16:
 +      case SNDRV_PCM_FORMAT_S16:

-+          sample_size = 1;
++              sample_size = 1;

 +              dma_width = JZ4740_DMA_WIDTH_16BIT;
 +              break;
 +      default:
 +              dma_width = JZ4740_DMA_WIDTH_16BIT;
 +              break;
 +      default:


@@ -379,7 +357,7 @@ index 0000000..d3daa27
 +
 +      switch (params_channels(params)) {
 +      case 2:
 +
 +      switch (params_channels(params)) {
 +      case 2:

-+          break;
++              break;

 +      case 1:
 +              if (playback) {
 +                      ctrl |= JZ_AIC_CTRL_MONO_TO_STEREO;
 +      case 1:
 +              if (playback) {
 +                      ctrl |= JZ_AIC_CTRL_MONO_TO_STEREO;


@@ -392,23 +370,20 @@ index 0000000..d3daa27
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);
 +
 +      if (playback) {
 +      jz4740_i2s_write(i2s, JZ_REG_AIC_CTRL, ctrl);
 +
 +      if (playback) {

-+              jz4740_i2s_dma_playback_config.src_width = dma_width;
-+              pcm_config = &i2s->playback_pcm_config;
-+              pcm_config->dma_config = &jz4740_i2s_dma_playback_config;
++              pcm_config = &i2s->pcm_config_playback;
++              pcm_config->dma_config.dst_width = dma_width;

 +      } else {
 +      } else {

-+              jz4740_i2s_dma_capture_config.dst_width = dma_width;
-+              pcm_config = &i2s->capture_pcm_config;
-+              pcm_config->dma_config = &jz4740_i2s_dma_capture_config;
++              pcm_config = &i2s->pcm_config_capture;
++              pcm_config->dma_config.src_width = dma_width;

 +      }
 +      }

-+      pcm_config->fifo_addr = i2s->phys_base + JZ_REG_AIC_FIFO;
++

 +
 +      snd_soc_dai_set_dma_data(dai, substream, pcm_config);
 +
 +      return 0;
 +}
 +
 +
 +      snd_soc_dai_set_dma_data(dai, substream, pcm_config);
 +
 +      return 0;
 +}
 +

-+static int jz4740_i2s_set_clkdiv(struct snd_soc_dai *dai,
-+                                int div_id, int div)
++static int jz4740_i2s_set_clkdiv(struct snd_soc_dai *dai, int div_id, int div)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +


@@ -426,7 +401,7 @@ index 0000000..d3daa27
 +}
 +
 +static int jz4740_i2s_set_sysclk(struct snd_soc_dai *dai, int clk_id,
 +}
 +
 +static int jz4740_i2s_set_sysclk(struct snd_soc_dai *dai, int clk_id,

-+                                unsigned int freq, int dir)
++      unsigned int freq, int dir)

 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      int ret = 0;
 +{
 +      struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
 +      int ret = 0;


@@ -524,13 +499,13 @@ index 0000000..d3daa27
 +      .playback = {
 +              .channels_min = 1,
 +              .channels_max = 2,
 +      .playback = {
 +              .channels_min = 1,
 +              .channels_max = 2,

-+              .rates = SNDRV_PCM_RATE_8000_44100,
++              .rates = SNDRV_PCM_RATE_8000_48000,

 +              .formats = JZ4740_I2S_FMTS,
 +      },
 +      .capture = {
 +              .channels_min = 2,
 +              .channels_max = 2,
 +              .formats = JZ4740_I2S_FMTS,
 +      },
 +      .capture = {
 +              .channels_min = 2,
 +              .channels_max = 2,

-+              .rates = SNDRV_PCM_RATE_8000_44100,
++              .rates = SNDRV_PCM_RATE_8000_48000,

 +              .formats = JZ4740_I2S_FMTS,
 +      },
 +      .symmetric_rates = 1,
 +              .formats = JZ4740_I2S_FMTS,
 +      },
 +      .symmetric_rates = 1,


@@ -540,6 +515,29 @@ index 0000000..d3daa27
 +};
 +EXPORT_SYMBOL_GPL(jz4740_i2s_dai);
 +
 +};
 +EXPORT_SYMBOL_GPL(jz4740_i2s_dai);
 +

++static void __devinit jz4740_i2c_init_pcm_config(struct jz4740_i2s *i2s)
++{
++      struct jz4740_dma_config *dma_config;
++
++      /* Playback */
++      dma_config = &i2s->pcm_config_playback.dma_config;
++      dma_config->src_width = JZ4740_DMA_WIDTH_32BIT,
++      dma_config->transfer_size = JZ4740_DMA_TRANSFER_SIZE_16BYTE;
++      dma_config->request_type = JZ4740_DMA_TYPE_AIC_TRANSMIT;
++      dma_config->flags = JZ4740_DMA_SRC_AUTOINC;
++      dma_config->mode = JZ4740_DMA_MODE_SINGLE;
++      i2s->pcm_config_playback.fifo_addr = i2s->phys_base + JZ_REG_AIC_FIFO;
++
++      /* Capture */
++      dma_config = &i2s->pcm_config_capture.dma_config;
++      dma_config->dst_width = JZ4740_DMA_WIDTH_32BIT,
++      dma_config->transfer_size = JZ4740_DMA_TRANSFER_SIZE_16BYTE;
++      dma_config->request_type = JZ4740_DMA_TYPE_AIC_RECEIVE;
++      dma_config->flags = JZ4740_DMA_DST_AUTOINC;
++      dma_config->mode = JZ4740_DMA_MODE_SINGLE;
++      i2s->pcm_config_capture.fifo_addr = i2s->phys_base + JZ_REG_AIC_FIFO;
++}
++

 +static int __devinit jz4740_i2s_dev_probe(struct platform_device *pdev)
 +{
 +      struct jz4740_i2s *i2s;
 +static int __devinit jz4740_i2s_dev_probe(struct platform_device *pdev)
 +{
 +      struct jz4740_i2s *i2s;


@@ -574,20 +572,13 @@ index 0000000..d3daa27
 +
 +      i2s->phys_base = i2s->mem->start;
 +
 +
 +      i2s->phys_base = i2s->mem->start;
 +

-+      jz4740_i2s_dai.private_data = i2s;
-+
-+      ret = snd_soc_register_dai(&jz4740_i2s_dai);
-+

 +      i2s->clk_aic = clk_get(&pdev->dev, "aic");
 +      i2s->clk_aic = clk_get(&pdev->dev, "aic");

-+

 +      if (IS_ERR(i2s->clk_aic)) {
 +              ret = PTR_ERR(i2s->clk_aic);
 +              goto err_iounmap;
 +      }
 +
 +      if (IS_ERR(i2s->clk_aic)) {
 +              ret = PTR_ERR(i2s->clk_aic);
 +              goto err_iounmap;
 +      }
 +

-+

 +      i2s->clk_i2s = clk_get(&pdev->dev, "i2s");
 +      i2s->clk_i2s = clk_get(&pdev->dev, "i2s");

-+

 +      if (IS_ERR(i2s->clk_i2s)) {
 +              ret = PTR_ERR(i2s->clk_i2s);
 +              goto err_iounmap;
 +      if (IS_ERR(i2s->clk_i2s)) {
 +              ret = PTR_ERR(i2s->clk_i2s);
 +              goto err_iounmap;


@@ -595,6 +586,11 @@ index 0000000..d3daa27
 +
 +      clk_enable(i2s->clk_aic);
 +
 +
 +      clk_enable(i2s->clk_aic);
 +

++      jz4740_i2c_init_pcm_config(i2s);
++
++      jz4740_i2s_dai.private_data = i2s;
++      ret = snd_soc_register_dai(&jz4740_i2s_dai);
++

 +      platform_set_drvdata(pdev, i2s);
 +
 +      return 0;
 +      platform_set_drvdata(pdev, i2s);
 +
 +      return 0;


@@ -679,7 +675,7 @@ index 0000000..da22ed8
 +#endif
 diff --git a/sound/soc/jz4740/jz4740-pcm.c b/sound/soc/jz4740/jz4740-pcm.c
 new file mode 100644
 +#endif
 diff --git a/sound/soc/jz4740/jz4740-pcm.c b/sound/soc/jz4740/jz4740-pcm.c
 new file mode 100644

-index 0000000..e55ffbf
+index 0000000..fd1c203

 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-pcm.c
 @@ -0,0 +1,350 @@
 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-pcm.c
 @@ -0,0 +1,350 @@


@@ -735,7 +731,7 @@ index 0000000..e55ffbf
 +      .rates                  = SNDRV_PCM_RATE_8000_48000,
 +      .channels_min           = 1,
 +      .channels_max           = 2,
 +      .rates                  = SNDRV_PCM_RATE_8000_48000,
 +      .channels_min           = 1,
 +      .channels_max           = 2,

-+      .period_bytes_min       = 32,
++      .period_bytes_min       = 16,

 +      .period_bytes_max       = 2 * PAGE_SIZE,
 +      .periods_min            = 2,
 +      .periods_max            = 128,
 +      .period_bytes_max       = 2 * PAGE_SIZE,
 +      .periods_min            = 2,
 +      .periods_max            = 128,


@@ -805,7 +801,7 @@ index 0000000..e55ffbf
 +      if (!prtd->dma)
 +              return -EBUSY;
 +
 +      if (!prtd->dma)
 +              return -EBUSY;
 +

-+      jz4740_dma_configure(prtd->dma, config->dma_config);
++      jz4740_dma_configure(prtd->dma, &config->dma_config);

 +      prtd->fifo_addr = config->fifo_addr;
 +
 +      jz4740_dma_set_complete_cb(prtd->dma, jz4740_pcm_dma_transfer_done);
 +      prtd->fifo_addr = config->fifo_addr;
 +
 +      jz4740_dma_set_complete_cb(prtd->dma, jz4740_pcm_dma_transfer_done);


@@ -1035,7 +1031,7 @@ index 0000000..e55ffbf
 +MODULE_LICENSE("GPL");
 diff --git a/sound/soc/jz4740/jz4740-pcm.h b/sound/soc/jz4740/jz4740-pcm.h
 new file mode 100644
 +MODULE_LICENSE("GPL");
 diff --git a/sound/soc/jz4740/jz4740-pcm.h b/sound/soc/jz4740/jz4740-pcm.h
 new file mode 100644

-index 0000000..2a11800
+index 0000000..e3f221e

 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-pcm.h
 @@ -0,0 +1,22 @@
 --- /dev/null
 +++ b/sound/soc/jz4740/jz4740-pcm.h
 @@ -0,0 +1,22 @@


@@ -1056,7 +1052,7 @@ index 0000000..2a11800
 +extern struct snd_soc_platform jz4740_soc_platform;
 +
 +struct jz4740_pcm_config {
 +extern struct snd_soc_platform jz4740_soc_platform;
 +
 +struct jz4740_pcm_config {

-+      struct jz4740_dma_config *dma_config;
++      struct jz4740_dma_config dma_config;

 +      phys_addr_t fifo_addr;
 +};
 +
 +      phys_addr_t fifo_addr;
 +};
 +