[alsa-devel] [PATCH v2 4/4] Rename exports in mpc5200_dma.c

Mark Brown broonie at opensource.wolfsonmicro.com
Mon Apr 27 21:35:34 CEST 2009


On Mon, Apr 27, 2009 at 12:44:47PM -0400, Jon Smirl wrote:
> Rename the public DMA exports into the global name space so that the DMA code can be built as a module.

> +EXPORT_SYMBOL_GPL(mpc5200_audio_dma_startup);
> +EXPORT_SYMBOL_GPL(mpc5200_audio_dma_hw_free);
> +EXPORT_SYMBOL_GPL(mpc5200_audio_dma_trigger);
> +EXPORT_SYMBOL_GPL(mpc5200_audio_dma_shutdown);

Why are you exporting these?  They should all be accessed via...

> -struct snd_soc_platform psc_dma_pcm_soc_platform = {
> +struct snd_soc_platform mpc5200_audio_dma_platform = {
>  	.name		= "mpc5200-psc-audio",
>  	.pcm_ops	= &psc_dma_pcm_ops,
>  	.pcm_new	= &psc_dma_pcm_new,
>  	.pcm_free	= &psc_dma_pcm_free,
>  };
> -
> +EXPORT_SYMBOL_GPL(mpc5200_audio_dma_platform);

...this unless there's a reason not to do so.

> -int psc_dma_startup(struct snd_pcm_substream *substream,
> +int mpc5200_audio_dma_startup(struct snd_pcm_substream *substream,
>  			   struct snd_soc_dai *dai);

These should go.


More information about the Alsa-devel mailing list