[alsa-devel] [PATCH] ASoC: omap-pcm: Allow only formats with 1, 2, and 4 byte physical size

[Peter Ujfalusi]

On 03/03/2015 01:28 PM, Jyri Sarha wrote:
> sDMA support only transfer elements with 1, 2, and 4 byte physical
> size. Initialize the pcm driver accordingly.

Acked-by: Peter Ujfalusi <peter.ujfalusi at ti.com>

> 
> Signed-off-by: Jyri Sarha <jsarha at ti.com>
> ---
>  sound/soc/omap/omap-pcm.c | 21 ++++++++++++++++++++-
>  1 file changed, 20 insertions(+), 1 deletion(-)
> 
> diff --git a/sound/soc/omap/omap-pcm.c b/sound/soc/omap/omap-pcm.c
> index f4b05bc..e49ee23 100644
> --- a/sound/soc/omap/omap-pcm.c
> +++ b/sound/soc/omap/omap-pcm.c
> @@ -39,7 +39,7 @@
>  #define pcm_omap1510()	0
>  #endif
>  
> -static const struct snd_pcm_hardware omap_pcm_hardware = {
> +static struct snd_pcm_hardware omap_pcm_hardware = {
>  	.info			= SNDRV_PCM_INFO_MMAP |
>  				  SNDRV_PCM_INFO_MMAP_VALID |
>  				  SNDRV_PCM_INFO_INTERLEAVED |
> @@ -53,6 +53,24 @@ static const struct snd_pcm_hardware omap_pcm_hardware = {
>  	.buffer_bytes_max	= 128 * 1024,
>  };
>  
> +/* sDMA supports only 1, 2, and 4 byte transfer elements. */
> +static void omap_pcm_limit_supported_formats(void)
> +{
> +	int i;
> +
> +	for (i = 0; i < SNDRV_PCM_FORMAT_LAST; i++) {
> +		switch (snd_pcm_format_physical_width(i)) {
> +		case 8:
> +		case 16:
> +		case 32:
> +			omap_pcm_hardware.formats |= (1LL << i);
> +			break;
> +		default:
> +			break;
> +		}
> +	}
> +}
> +
>  /* this may get called several times by oss emulation */
>  static int omap_pcm_hw_params(struct snd_pcm_substream *substream,
>  			      struct snd_pcm_hw_params *params)
> @@ -235,6 +253,7 @@ static struct snd_soc_platform_driver omap_soc_platform = {
>  
>  int omap_pcm_platform_register(struct device *dev)
>  {
> +	omap_pcm_limit_supported_formats();
>  	return devm_snd_soc_register_platform(dev, &omap_soc_platform);
>  }
>  EXPORT_SYMBOL_GPL(omap_pcm_platform_register);
> 


-- 
Péter