Re: [PATCH V2] ASoC: fsl_asrc: add constraint for the asrc of older version

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

 



Hi

> On Fri, Mar 01, 2019 at 08:37:08AM +0000, S.j. Wang wrote:
> > There is constraint for the channel number setting on the
> 
> nit: "a constraint"
> 
> > asrc of older version (e.g. imx35), the channel number should be even,
> > odd number isn't valid.
> 
> > +static int fsl_asrc_dai_startup(struct snd_pcm_substream *substream,
> > +				struct snd_soc_dai *dai)
> > +{
> > +	struct fsl_asrc *asrc_priv = snd_soc_dai_get_drvdata(dai);
> > +
> > +	/* channel_bits = 3 means older version on imx35*/
> 
> Space between '5' and '*'. And better to make it clear:
> 
> 	/* Odd channel number is not valid for older ASRC (channel_bits==3)
> */
> 
> > +	if (asrc_priv->channel_bits == 3)
> > +		snd_pcm_hw_constraint_step(substream->runtime, 0,
> > +
> SNDRV_PCM_HW_PARAM_CHANNELS, 2);
> 
> For your next version,
> 
> Acked-by: Nicolin Chen <nicoleotsuka@xxxxxxxxx>

Ok, thanks, will send v3.

> 
> Cheers
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux