Commit 69b6f196 authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Mark Brown
Browse files

ASoC: ux500_pcm: Use the same snd_pcm_hardware for playback and capture 



The snd_pcm_hardware structs for playback and capture in the ux500 PCM are
identical, so remove one of them and use the same snd_pcm_hardware struct for
both playback and capture. Also move the defines used to initialize the
snd_pcm_hardware fields from ux500_pcm.h to ux500_pcm.c since that's the only
place where they are used.

Also drop the assignment of the snd_pcm_hardware struct to runtime->hw since
that is what the call to snd_soc_set_runtime_hwparams() right above it already
does, so the second assignment is redundant.

Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent e6451c3f

sound/soc/ux500/ux500_pcm.c

+15 −36
Original line number Diff line number Diff line
@@ -28,28 +28,19 @@ 
#include "ux500_msp_i2s.h"

#include "ux500_pcm.h"



static struct snd_pcm_hardware ux500_pcm_hw_playback = {

	.info = SNDRV_PCM_INFO_INTERLEAVED |

		SNDRV_PCM_INFO_MMAP |

		SNDRV_PCM_INFO_RESUME |

		SNDRV_PCM_INFO_PAUSE,

	.formats = SNDRV_PCM_FMTBIT_S16_LE |

		SNDRV_PCM_FMTBIT_U16_LE |

		SNDRV_PCM_FMTBIT_S16_BE |

		SNDRV_PCM_FMTBIT_U16_BE,

	.rates = SNDRV_PCM_RATE_KNOT,

	.rate_min = UX500_PLATFORM_MIN_RATE_PLAYBACK,

	.rate_max = UX500_PLATFORM_MAX_RATE_PLAYBACK,

	.channels_min = UX500_PLATFORM_MIN_CHANNELS,

	.channels_max = UX500_PLATFORM_MAX_CHANNELS,

	.buffer_bytes_max = UX500_PLATFORM_BUFFER_BYTES_MAX,

	.period_bytes_min = UX500_PLATFORM_PERIODS_BYTES_MIN,

	.period_bytes_max = UX500_PLATFORM_PERIODS_BYTES_MAX,

	.periods_min = UX500_PLATFORM_PERIODS_MIN,

	.periods_max = UX500_PLATFORM_PERIODS_MAX,

};

#define UX500_PLATFORM_MIN_RATE 8000

#define UX500_PLATFORM_MAX_RATE 48000



#define UX500_PLATFORM_MIN_CHANNELS 1

#define UX500_PLATFORM_MAX_CHANNELS 8



#define UX500_PLATFORM_PERIODS_BYTES_MIN	128

#define UX500_PLATFORM_PERIODS_BYTES_MAX	(64 * PAGE_SIZE)

#define UX500_PLATFORM_PERIODS_MIN		2

#define UX500_PLATFORM_PERIODS_MAX		48

#define UX500_PLATFORM_BUFFER_BYTES_MAX		(2048 * PAGE_SIZE)



static struct snd_pcm_hardware ux500_pcm_hw_capture = {

static struct snd_pcm_hardware ux500_pcm_hw = {

	.info = SNDRV_PCM_INFO_INTERLEAVED |

		SNDRV_PCM_INFO_MMAP |

		SNDRV_PCM_INFO_RESUME |
@@ -59,8 +50,8 @@ static struct snd_pcm_hardware ux500_pcm_hw_capture = { 
		SNDRV_PCM_FMTBIT_S16_BE |

		SNDRV_PCM_FMTBIT_U16_BE,

	.rates = SNDRV_PCM_RATE_KNOT,

	.rate_min = UX500_PLATFORM_MIN_RATE_CAPTURE,

	.rate_max = UX500_PLATFORM_MAX_RATE_CAPTURE,

	.rate_min = UX500_PLATFORM_MIN_RATE,

	.rate_max = UX500_PLATFORM_MAX_RATE,

	.channels_min = UX500_PLATFORM_MIN_CHANNELS,

	.channels_max = UX500_PLATFORM_MAX_CHANNELS,

	.buffer_bytes_max = UX500_PLATFORM_BUFFER_BYTES_MAX,
@@ -90,8 +81,6 @@ static void ux500_pcm_dma_hw_free(struct device *dev, 


static int ux500_pcm_open(struct snd_pcm_substream *substream)

{

	int stream_id = substream->pstr->stream;

	struct snd_pcm_runtime *runtime = substream->runtime;

	struct snd_soc_pcm_runtime *rtd = substream->private_data;

	struct snd_soc_dai *dai = rtd->cpu_dai;

	struct device *dev = dai->dev;
@@ -104,17 +93,7 @@ static int ux500_pcm_open(struct snd_pcm_substream *substream) 
		snd_pcm_stream_str(substream));



	dev_dbg(dev, "%s: Set runtime hwparams.\n", __func__);

	if (stream_id == SNDRV_PCM_STREAM_PLAYBACK)

		snd_soc_set_runtime_hwparams(substream,

					&ux500_pcm_hw_playback);

	else

		snd_soc_set_runtime_hwparams(substream,

					&ux500_pcm_hw_capture);



	dev_dbg(dev, "%s: Set hw-struct for %s.\n", __func__,

		snd_pcm_stream_str(substream));

	runtime->hw = (stream_id == SNDRV_PCM_STREAM_PLAYBACK) ?

		ux500_pcm_hw_playback : ux500_pcm_hw_capture;

	snd_soc_set_runtime_hwparams(substream, &ux500_pcm_hw);



	mem_data_width = STEDMA40_HALFWORD_WIDTH;

sound/soc/ux500/ux500_pcm.h

+0 −14
Original line number Diff line number Diff line
@@ -18,20 +18,6 @@ 


#include <linux/workqueue.h>



#define UX500_PLATFORM_MIN_RATE_PLAYBACK 8000

#define UX500_PLATFORM_MAX_RATE_PLAYBACK 48000

#define UX500_PLATFORM_MIN_RATE_CAPTURE	8000

#define UX500_PLATFORM_MAX_RATE_CAPTURE	48000



#define UX500_PLATFORM_MIN_CHANNELS 1

#define UX500_PLATFORM_MAX_CHANNELS 8



#define UX500_PLATFORM_PERIODS_BYTES_MIN	128

#define UX500_PLATFORM_PERIODS_BYTES_MAX	(64 * PAGE_SIZE)

#define UX500_PLATFORM_PERIODS_MIN		2

#define UX500_PLATFORM_PERIODS_MAX		48

#define UX500_PLATFORM_BUFFER_BYTES_MAX		(2048 * PAGE_SIZE)



int ux500_pcm_register_platform(struct platform_device *pdev);

int ux500_pcm_unregister_platform(struct platform_device *pdev);

CRA Git | Maintained and supported by SUSTech CRA and CCSE