From mboxrd@z Thu Jan  1 00:00:00 1970
From: Liam Girdwood <liam.r.girdwood@intel.com>
Subject: Re: [PATCH 2/2] ASoC: compress: Add suport for DPCM
 into	compressed audio
Date: Fri, 17 Jan 2014 08:28:04 +0000
Message-ID: <1389947284.2353.5.camel@loki>
References: <1389804717-5985-1-git-send-email-liam.r.girdwood@linux.intel.com>
 <1389804717-5985-2-git-send-email-liam.r.girdwood@linux.intel.com>
 <s5hbnzbsysd.wl%tiwai@suse.de> <1389892081.2298.61.camel@loki>
 <20140117035326.GK25466@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <alsa-devel-bounces@alsa-project.org>
Received: from mga11.intel.com (mga11.intel.com [192.55.52.93])
 by alsa0.perex.cz (Postfix) with ESMTP id 033DD261A7B
 for <alsa-devel@alsa-project.org>; Fri, 17 Jan 2014 09:28:09 +0100 (CET)
In-Reply-To: <20140117035326.GK25466@intel.com>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/options/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel/>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Errors-To: alsa-devel-bounces@alsa-project.org
Sender: alsa-devel-bounces@alsa-project.org
To: Vinod Koul <vinod.koul@intel.com>
Cc: Takashi Iwai <tiwai@suse.de>, alsa-devel@alsa-project.org, Liam Girdwood <liam.r.girdwood@linux.jf.intel.com>, Mark Brown <broonie@kernel.org>
List-Id: alsa-devel@alsa-project.org

On Fri, 2014-01-17 at 09:23 +0530, Vinod Koul wrote:
> On Thu, Jan 16, 2014 at 05:08:01PM +0000, Liam Girdwood wrote:
> > On Thu, 2014-01-16 at 16:59 +0100, Takashi Iwai wrote:
> > > At Wed, 15 Jan 2014 16:51:57 +0000,
> > > Liam Girdwood wrote:
> > > > 
> > 
> > > 
> > > > +	mutex_lock_nested(&fe->card->mutex, SND_SOC_CARD_CLASS_RUNTIME);
> > > > +
> > > > +	if (platform->driver->compr_ops && platform->driver->compr_ops->set_params) {
> > > > +		ret = platform->driver->compr_ops->set_params(cstream, params);
> > > > +		if (ret < 0)
> > > > +			goto out;
> > > > +	}
> > > > +
> > > > +	if (fe->dai_link->compr_ops && fe->dai_link->compr_ops->set_params) {
> > > > +		ret = fe->dai_link->compr_ops->set_params(cstream);
> > > > +		if (ret < 0)
> > > > +			goto out;
> > > > +	}
> > > > +
> > > > +	memcpy(&fe->dpcm[fe_substream->stream].hw_params, params,
> > > > +			sizeof(struct snd_pcm_hw_params));
> > > 
> > > Does this really work?  params is snd_compr_params and hw_parmas is
> > > snd_pcm_hw_params, isn't it?
> > > 
> > 
> > Hmmm, no one reported a failure here to me. It is complete nonsense
> > though so I dont know why the testers didn't run into this. I'll fix it
> > for V2.
> I think that due to the fixup fn noone noticed this. The issue will be seen only
> by folks who dont have the fixup function.

I now know why we didn't see this on HSW. We have a fixed BE SSP
configuration so we always ignore the BE hw_params. So we ignored the
junk data ;)

Liam

---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.