Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1700758

Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio sample sizes

Show all headers | View raw

Hello Srinivas,

On 08/01/2017 12:49 AM, srinivas.kandagatla@linaro.org wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> 
> ADV7533 only supports audio samples word width from 16-24 bits.
> This patch restricts the audio sample sizes to the supported ones,
> so that sound card does not report wrong list of supported hwparms.
> 
> Without this patch aplay would fail when playing a 32 bit audio.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
>  drivers/gpu/drm/bridge/adv7511/adv7511_audio.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c b/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> index 67469c26bae8..d01d0aa0eef7 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> @@ -214,6 +214,8 @@ static struct hdmi_codec_pdata codec_data = {
>  	.ops = &adv7511_codec_ops,
>  	.max_i2s_channels = 2,
>  	.i2s = 1,
> +	.formats = (SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FMTBIT_S16_BE |
> +		    SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S24_BE),
>  };

I'm not sure that this limitation is needed.
I already used ADV7513 and i did not observe this limitation.

I had a look to ADV7533 data-sheet. it should support 32 and 64 bits I2S
format bus, with 16 or 24 bits precision sample. So it should support
SNDRV_PCM_FMTBIT_S32_LE and SNDRV_PCM_FMTBIT_S32_BE

As example, if you configure bus in Left justified format with 24 bits
sample length, 32 bits application samples should be truncated to 24
bits samples at ADV7533 I2S interface level (LSB dropped).

Perhaps i missed something... but look like more an I2S bus
configuration issue in your DT card description, that that a miss in the
drivers.

Regards
Arnaud
>  
>  int adv7511_audio_init(struct device *dev, struct adv7511 *adv7511)
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/2] ASoC: hdmi-codec: Allow driver to restrict sample size srinivas.kandagatla@linaro.org - 2017-08-01 00:50 +0200
  [PATCH v2 2/2] drm/bridge: adv7511: restrict audio sample sizes srinivas.kandagatla@linaro.org - 2017-08-01 01:00 +0200
    Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Arnaud Pouliquen <arnaud.pouliquen@st.com> - 2017-08-01 10:50 +0200
      Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Mark Brown <broonie@kernel.org> - 2017-08-01 14:30 +0200
        Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2017-08-01 15:00 +0200
          Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Mark Brown <broonie@kernel.org> - 2017-08-01 15:20 +0200
          Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Arnaud Pouliquen <arnaud.pouliquen@st.com> - 2017-08-01 18:00 +0200
      Re: [alsa-devel] [PATCH v2 2/2] drm/bridge: adv7511: restrict audio  sample sizes Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2017-08-01 14:30 +0200
  [PATCH v2 1/2] ASoC: hdmi-codec: Allow drivers to restrict sample sizes. srinivas.kandagatla@linaro.org - 2017-08-01 01:00 +0200

csiph-web