RE: [alsa-devel] [PATCHv2 6/6] ASoC: OMAP4: Add McPDM platform driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Liam Girdwood wrote:
> On Fri, 2010-01-22 at 17:15 -0600, Candelaria Villareal, Jorge wrote:
> > From: Misael Lopez Cruz <x0052729@xxxxxx>
> > 
> > McPDM platform driver is configured to use sDMA in order to transfer
> > to/from memory. Support for interfacing with ABE will be 
> added later.
> > 
> > McPDM dai currently supports up to 4 downlink channels and 2 uplink
> > channels simultaneously, as well as 88.2 and 96 KHz, and a sample
> > size of 32 bits.
> > 
> > Signed-off-by: Misael Lopez Cruz <x0052729@xxxxxx>
> > Signed-off-by: Margarita Olaya <x0080101@xxxxxx>
> > Signed-off-by: Jorge Eduardo Candelaria <x0107209@xxxxxx>
> > ---
> >  sound/soc/omap/Makefile     |    2 +-
> >  sound/soc/omap/omap-mcpdm.c |  250 
> +++++++++++++++++++++++++++++++++++++++++++
> >  sound/soc/omap/omap-mcpdm.h |   29 +++++
> >  3 files changed, 280 insertions(+), 1 deletions(-)
> >  create mode 100644 sound/soc/omap/omap-mcpdm.c
> >  create mode 100644 sound/soc/omap/omap-mcpdm.h
> > 
> > diff --git a/sound/soc/omap/Makefile b/sound/soc/omap/Makefile
> > index bf8b388..d6382fa 100644
> > --- a/sound/soc/omap/Makefile
> > +++ b/sound/soc/omap/Makefile
> > @@ -1,7 +1,7 @@
> >  # OMAP Platform Support
> >  snd-soc-omap-objs := omap-pcm.o
> >  snd-soc-omap-mcbsp-objs := omap-mcbsp.o
> > -snd-soc-omap-mcpdm-objs := mcpdm.o
> > +snd-soc-omap-mcpdm-objs := mcpdm.o omap-mcpdm.o
> >  
> >  obj-$(CONFIG_SND_OMAP_SOC) += snd-soc-omap.o
> >  obj-$(CONFIG_SND_OMAP_SOC_MCBSP) += snd-soc-omap-mcbsp.o
> > diff --git a/sound/soc/omap/omap-mcpdm.c 
> b/sound/soc/omap/omap-mcpdm.c
> > new file mode 100644
> > index 0000000..180cca6
> > --- /dev/null
> > +++ b/sound/soc/omap/omap-mcpdm.c
> > @@ -0,0 +1,250 @@
> > +/*
> > + * omap-mcpdm.c  --  OMAP ALSA SoC DAI driver using McPDM port
> > + *
> > + * Copyright (C) 2009 Texas Instruments
> > + *
> > + * Author: Misael Lopez Cruz <x0052729@xxxxxx>
> > + * Contact: Jorge Eduardo Candelaria <x0107209@xxxxxx>
> > + *          Margarita Olaya <magi.olaya@xxxxxx>
> > + *
> > + * 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/init.h>
> > +#include <linux/module.h>
> > +#include <linux/device.h>
> > +#include <sound/core.h>
> > +#include <sound/pcm.h>
> > +#include <sound/pcm_params.h>
> > +#include <sound/initval.h>
> > +#include <sound/soc.h>
> > +
> > +#include <mach/control.h>
> > +#include <mach/dma.h>
> > +#include <mach/mcbsp.h>
> > +#include "mcpdm.h"
> > +#include "omap-mcpdm.h"
> > +#include "omap-pcm.h"
> > +
> > +struct omap_mcpdm_data {
> > +	struct omap_mcpdm_link *links;
> > +	int active;
> > +};
> > +
> > +static struct omap_mcpdm_link omap_mcpdm_links[] = {
> > +	/* downlink */
> > +	{
> > +		.irq_mask = MCPDM_DN_IRQ_EMPTY | MCPDM_DN_IRQ_FULL,
> > +		.threshold = 1,
> > +		.format = PDMOUTFORMAT_LJUST,
> > +	},
> > +	/* uplink */
> > +	{
> > +		.irq_mask = MCPDM_UP_IRQ_EMPTY | MCPDM_UP_IRQ_FULL,
> > +		.threshold = 1,
> > +		.format = PDMOUTFORMAT_LJUST,
> > +	},
> > +};
> > +
> > +static struct omap_mcpdm_data mcpdm_data = {
> > +	.links = omap_mcpdm_links,
> > +	.active = 0,
> > +};
> > +
> > +/*
> > + * Stream DMA parameters
> > + */
> > +static struct omap_pcm_dma_data omap_mcpdm_dai_dma_params[] = {
> > +	{
> > +		.name = "Audio downlink",
> > +		.dma_req = OMAP44XX_DMA_MCPDM_DL,
> > +		.data_type = OMAP_DMA_DATA_TYPE_S32,
> > +		.sync_mode = OMAP_DMA_SYNC_PACKET,
> > +		.packet_size = 16,
> > +		.port_addr = OMAP44XX_MCPDM_L3_BASE + MCPDM_DN_DATA,
> > +	},
> > +	{
> > +		.name = "Audio uplink",
> > +		.dma_req = OMAP44XX_DMA_MCPDM_UP,
> > +		.data_type = OMAP_DMA_DATA_TYPE_S32,
> > +		.sync_mode = OMAP_DMA_SYNC_PACKET,
> > +		.packet_size = 16,
> > +		.port_addr = OMAP44XX_MCPDM_L3_BASE + MCPDM_UP_DATA,
> > +	},
> > +};
> > +
> > +static int omap_mcpdm_dai_startup(struct snd_pcm_substream 
> *substream,
> > +				  struct snd_soc_dai *dai)
> > +{
> > +	struct snd_soc_pcm_runtime *rtd = substream->private_data;
> > +	struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
> > +	int err = 0;
> > +
> > +	if (!cpu_dai->active)
> > +		err = omap_mcpdm_request();
> 
> Will anything else use this hw interface other than ALSA audio ?
> If not, the request is probably better in the machine driver probe().

omap_mcpdm_request will enable the functional clock. Isn't it better
for the clock to be enabled only when is about to get used?

> 
> > +
> > +	return err;
> > +}
> > +
> > +static void omap_mcpdm_dai_shutdown(struct 
> snd_pcm_substream *substream,
> > +				    struct snd_soc_dai *dai)
> > +{
> > +	struct snd_soc_pcm_runtime *rtd = substream->private_data;
> > +	struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
> > +
> > +	if (!cpu_dai->active)
> > +		omap_mcpdm_free();
> 
> ditto.

Also, omap_mcpdm_free should disable the clock once it is not needed.

> 
> > +}
> > +
> > +static int omap_mcpdm_dai_trigger(struct snd_pcm_substream 
> *substream, int cmd,
> > +				  struct snd_soc_dai *dai)
> > +{
> > +	struct snd_soc_pcm_runtime *rtd = substream->private_data;
> > +	struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
> > +	struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data;
> > +	int stream = substream->stream;
> > +	int err = 0;
> > +
> > +	switch (cmd) {
> > +	case SNDRV_PCM_TRIGGER_START:
> > +	case SNDRV_PCM_TRIGGER_RESUME:
> > +	case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
> > +		if (!mcpdm_priv->active++)
> > +			omap_mcpdm_start(stream);
> > +		break;
> > +
> > +	case SNDRV_PCM_TRIGGER_STOP:
> > +	case SNDRV_PCM_TRIGGER_SUSPEND:
> > +	case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
> > +		if (!--mcpdm_priv->active)
> > +			omap_mcpdm_stop(stream);
> > +		break;
> > +	default:
> > +		err = -EINVAL;
> > +	}
> > +
> > +	return err;
> > +}
> > +
> > +static int omap_mcpdm_dai_hw_params(struct 
> snd_pcm_substream *substream,
> > +				    struct snd_pcm_hw_params *params,
> > +				    struct snd_soc_dai *dai)
> > +{
> > +	struct snd_soc_pcm_runtime *rtd = substream->private_data;
> > +	struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
> > +	struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data;
> > +	struct omap_mcpdm_link *mcpdm_links = mcpdm_priv->links;
> > +	int stream = substream->stream;
> > +	int channels, err, link_mask = 0;
> > +
> > +	cpu_dai->dma_data = &omap_mcpdm_dai_dma_params[stream];
> > +
> > +	channels = params_channels(params);
> > +	switch (channels) {
> > +	case 4:
> > +		if (stream)
> 
> stream is either PLAYBACK or CAPTURE here.

Downlink and uplink were chosen instead to avoid confusion
because that is how it is referred in McPDM technical
documentation. But I do understand that we are talking
about ALSA stream, not McPDM...

> 
> > +			/* up to 2 channels for uplink */

So I should refer to CAPTURE instead of UPLINK in here.

> > +			return -EINVAL;
> > +		link_mask |= 1 << 3;
> > +	case 3:
> > +		if (stream)
> > +			/* up to 2 channels for uplink */
> > +			return -EINVAL;
> > +		link_mask |= 1 << 2;
> > +	case 2:
> > +		link_mask |= 1 << 1;
> > +	case 1:
> > +		link_mask |= 1 << 0;
> > +		break;
> > +	default:
> > +		/* unsupported number of channels */
> > +		return -EINVAL;
> > +	}
> > +
> > +	if (stream) {
> 
> ditto

Here, however, the methods set_uplink & set_downlink are
Named after McPDM data paths. In this case is it preferred
to name everything using ALSA convention? Or would it be ok
to use downlink/uplink on certain cases?

> 
> > +		mcpdm_links[stream].channels = link_mask << 0;
> > +		err = omap_mcpdm_set_uplink(&mcpdm_links[stream]);
> > +	} else {
> > +		mcpdm_links[stream].channels = link_mask << 3;
> > +		err = omap_mcpdm_set_downlink(&mcpdm_links[stream]);
> > +	}
> > +
> > +	return err;
> > +}
> > +
> > +static int omap_mcpdm_dai_hw_free(struct snd_pcm_substream 
> *substream,
> > +				  struct snd_soc_dai *dai)
> > +{
> > +	struct snd_soc_pcm_runtime *rtd = substream->private_data;
> > +	struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
> > +	struct omap_mcpdm_data *mcpdm_priv = cpu_dai->private_data;
> > +	struct omap_mcpdm_link *mcpdm_links = mcpdm_priv->links;
> > +	int err;
> > +
> > +	if (substream->stream)
> 
> 
> ditto

This is the same case, uplink and downlink refer to McPDM data
path names.
 
> > +		err = 
> omap_mcpdm_clr_uplink(&mcpdm_links[substream->stream]);
> > +	else
> > +		err = 
> omap_mcpdm_clr_downlink(&mcpdm_links[substream->stream]);
> > +
> > +	return err;
> > +}
> > +
> > +static struct snd_soc_dai_ops omap_mcpdm_dai_ops = {
> > +	.startup	= omap_mcpdm_dai_startup,
> > +	.shutdown	= omap_mcpdm_dai_shutdown,
> > +	.trigger	= omap_mcpdm_dai_trigger,
> > +	.hw_params	= omap_mcpdm_dai_hw_params,
> > +	.hw_free	= omap_mcpdm_dai_hw_free,
> > +};
> > +
> > +#define OMAP_MCPDM_RATES	(SNDRV_PCM_RATE_88200 | 
> SNDRV_PCM_RATE_96000)
> 
> Is this the link rate or sample rate ? What about slower rates ?

This is the sample rate. Slower rates are not supported by McPDM
nor by the audio codec. When ABE is present, it supports other
input rates, like 48000, but then ABE needs to upsample to 96000
before sending to McPDM module.

> 
> > +#define OMAP_MCPDM_FORMATS	(SNDRV_PCM_FMTBIT_S32_LE)
> > +
> 
> Is this the link format or sample format ? What about 16bit ?

Again, only ABE supports 16 bit, but needs to convert to 32 bit
before sending to McPDM. With current McPDM patches, we are
bypassing ABE.

> 
> > +struct snd_soc_dai omap_mcpdm_dai = {
> > +	.name = "omap-mcpdm",
> > +	.id = -1,
> > +	.playback = {
> > +		.channels_min = 1,
> > +		.channels_max = 4,
> > +		.rates = OMAP_MCPDM_RATES,
> > +		.formats = OMAP_MCPDM_FORMATS,
> > +	},
> > +	.capture = {
> > +		.channels_min = 1,
> > +		.channels_max = 2,
> > +		.rates = OMAP_MCPDM_RATES,
> > +		.formats = OMAP_MCPDM_FORMATS,
> > +	},
> > +	.ops = &omap_mcpdm_dai_ops,
> > +	.private_data = &mcpdm_data,
> > +};
> > +EXPORT_SYMBOL_GPL(omap_mcpdm_dai);
> > +
> > +static int __init snd_omap_mcpdm_init(void)
> > +{
> > +	return snd_soc_register_dai(&omap_mcpdm_dai);
> > +}
> > +module_init(snd_omap_mcpdm_init);
> > +
> > +static void __exit snd_omap_mcpdm_exit(void)
> > +{
> > +	snd_soc_unregister_dai(&omap_mcpdm_dai);
> > +}
> > +module_exit(snd_omap_mcpdm_exit);
> > +
> > +MODULE_AUTHOR("Misael Lopez Cruz <x0052729@xxxxxx>");
> > +MODULE_DESCRIPTION("OMAP PDM SoC Interface");
> > +MODULE_LICENSE("GPL");
> > diff --git a/sound/soc/omap/omap-mcpdm.h 
> b/sound/soc/omap/omap-mcpdm.h
> > new file mode 100644
> > index 0000000..73b80d5
> > --- /dev/null
> > +++ b/sound/soc/omap/omap-mcpdm.h
> > @@ -0,0 +1,29 @@
> > +/*
> > + * omap-mcpdm.h
> > + *
> > + * Copyright (C) 2009 Texas Instruments
> > + *
> > + * Contact: Misael Lopez Cruz <x0052729@xxxxxx>
> > + *
> > + * 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
> > + *
> > + */
> > +
> > +#ifndef __OMAP_MCPDM_H__
> > +#define __OMAP_MCPDM_H__
> > +
> > +extern struct snd_soc_dai omap_mcpdm_dai;
> > +
> > +#endif	/* End of __OMAP_MCPDM_H__ */
> 
> 
> ��.n��������+%������w��{.n�����{�������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux