mcbsp.1 slave clkx ext problem

[Michael Trimarchi <michael@amarulasolutions.com>]

Hi

I'm trying to understand what is wrong here with this small
driver.

/* McBSP1 */
OMAP4_MUX(ABE_MCBSP1_CLKX, OMAP_MUX_MODE0 | OMAP_PIN_INPUT),
OMAP4_MUX(ABE_MCBSP1_DR, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLDOWN),
OMAP4_MUX(ABE_MCBSP1_DX, OMAP_MUX_MODE0 | OMAP_PIN_OUTPUT |
          OMAP_PULL_ENA),
OMAP4_MUX(ABE_MCBSP1_FSX, OMAP_MUX_MODE0 | OMAP_PIN_INPUT),

kernel 3.8.0

[  107.276702] omap-mcbsp omap-mcbsp.1: Configuring McBSP1  phys_base: 0x40122000
[  107.283691] omap-mcbsp omap-mcbsp.1: **** McBSP1 regs ****
[  107.283721] omap-mcbsp omap-mcbsp.1: DRR2:  0x88260a09
[  107.283721] omap-mcbsp omap-mcbsp.1: DRR1:  0x0000
[  107.283752] omap-mcbsp omap-mcbsp.1: DXR2:  0x0000
[  107.283752] omap-mcbsp omap-mcbsp.1: DXR1:  0x0000
[  107.283752] omap-mcbsp omap-mcbsp.1: SPCR2: 0x0233
[  107.283782] omap-mcbsp omap-mcbsp.1: SPCR1: 0x0030
[  107.283782] omap-mcbsp omap-mcbsp.1: RCR2:  0x80a1
[  107.283813] omap-mcbsp omap-mcbsp.1: RCR1:  0x00a0
[  107.283813] omap-mcbsp omap-mcbsp.1: XCR2:  0x80a1
[  107.283813] omap-mcbsp omap-mcbsp.1: XCR1:  0x00a0
[  107.283843] omap-mcbsp omap-mcbsp.1: SRGR2: 0x203f
[  107.283843] omap-mcbsp omap-mcbsp.1: SRGR1: 0x1f00
[  107.283843] omap-mcbsp omap-mcbsp.1: PCR0:  0x008f
[  107.283874] omap-mcbsp omap-mcbsp.1: ***********************
[  119.926177] omap-dma-engine omap-dma-engine: freeing channel for 33

Well basically it doesn't start the transfer. The clock is provided
using the clkx pin. clkx is rate * channel * bitsxword and come from
the codec. 

What is wrong? (I know that how I register the card is deprecated)

Regards Michael

/*
 * dacmax.c  --  SoC audio for pandaboard demokit
 *
 * Author: Michael Trimarchi <michael@amarulasolutions.com>
 *
 * Based on:
 * Author: Misael Lopez Cruz <x0052729@ti.com>
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * version 2 as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful, but
 * WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 * 02110-1301 USA
 *
 */

#include <linux/clk.h>
#include <linux/platform_device.h>
#include <sound/core.h>
#include <sound/pcm.h>
#include <sound/pcm_params.h>
#include <sound/soc.h>

#include <asm/mach-types.h>
#include <linux/gpio.h>
#include <linux/platform_data/asoc-ti-mcbsp.h>

#include <linux/module.h>
#include "omap-mcbsp.h"

static struct snd_soc_card snd_soc_dacmax;

static int dacmax_hw_params(struct snd_pcm_substream *substream,
	struct snd_pcm_hw_params *params)
{
	int ret;
	struct snd_soc_pcm_runtime *rtd = substream->private_data;
	struct snd_soc_dai *cpu_dai = rtd->cpu_dai;

	pr_info("%s: setting the params\n", __func__);

	ret = snd_soc_dai_set_sysclk(cpu_dai, OMAP_MCBSP_SYSCLK_CLKX_EXT, 0,
                                SND_SOC_CLOCK_IN);
	if (ret < 0) {
		printk(KERN_ERR "can't set CPU system clock " \
			        "OMAP_MCBSP_CLKR_SRC_CLKX\n");
	}

	return 0;
}

static struct snd_soc_ops dacmax_ops = {
	.hw_params = dacmax_hw_params,
};

static int dacmax_init(struct snd_soc_pcm_runtime *rtd)
{
	pr_info("%s: INIT\n", __func__);
	return 0;
}

/* Digital audio interface glue - connects codec <--> CPU */
static struct snd_soc_dai_link dacmax_dai = {
	.name = "DACMAX-I2S",
	.stream_name = "DACMAX-Audio",
	.cpu_dai_name = "omap-mcbsp.1",
	.codec_dai_name = "pcm1792a-hifi",
	.platform_name = "omap-pcm-audio",
	.codec_name = "pcm1792a",
	.dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF |
		   SND_SOC_DAIFMT_CBM_CFM,
	.init = dacmax_init,
	.ops = &dacmax_ops,
};

/* Audio machine driver */
static struct snd_soc_card snd_soc_dacmax = {
	.name = "DACMAX",
	.driver_name = "OMAP4",
	.long_name = "TI OMAP4 Board",
	.dai_link = &dacmax_dai,
	.num_links = 1,
};

static struct platform_device *dacmax_snd_device;

static int __init dacmax_soc_init(void)
{
	int ret;

	printk(KERN_INFO "DACMAX SoC init\n");

	dacmax_snd_device = platform_device_alloc("soc-audio", -1);
	if (!dacmax_snd_device) {
		printk(KERN_ERR "Platform device allocation failed\n");
		return -ENOMEM;
	}

	platform_set_drvdata(dacmax_snd_device, &snd_soc_dacmax);

	ret = platform_device_add(dacmax_snd_device);
	if (ret)
		goto err1;

	return 0;

err1:
	printk(KERN_ERR "Unable to add platform device\n");
	platform_device_put(dacmax_snd_device);

	return ret;
}
module_init(dacmax_soc_init);

static void __exit dacmax_soc_exit(void)
{
	platform_device_unregister(dacmax_snd_device);
}
module_exit(dacmax_soc_exit);

MODULE_AUTHOR("Michael Trimarchi <michael@amurulasolutions.com>");
MODULE_DESCRIPTION("ALSA SoC DACMAX");
MODULE_LICENSE("GPL");