Received: by 2002:a25:e74b:0:0:0:0:0 with SMTP id e72csp75415ybh; Tue, 21 Jul 2020 16:48:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzlvugQ2Uil0Y82PxaGtQfMiQNnUnY8RuxwbYGWMsDQdgm3TZIDlJtF474EQwqcHJEVFZcl X-Received: by 2002:a05:6402:1a3c:: with SMTP id be28mr29055017edb.140.1595375285249; Tue, 21 Jul 2020 16:48:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1595375285; cv=none; d=google.com; s=arc-20160816; b=XgPDkV0oqo2Wzj9VcZv0v6Dv3f+zRmS4Ptm25NelsBJXuSmmzqzupzTQ6Q/HXlCJfW ihPJ9INJv0eRtLHH9hcbKUdlysnG7BksUgN5MwlNwKDkT77TVvawFfw1jzWtZohEHWIJ AEsHdidENc2LmLfGyYhXc3C9Kc/yKh/AYOhJS0pjn1SY+6EpdJCt/hMrbtGkJfTzC4V7 govg/nf4zHWK3k2CA+iIS6TpsWfMoLtP+0sG0k2AerEhNPDUOkj1tC9D86RBo1WJqRCs esquqOWew2LQBktDjjG7Q/X53IYHc7uRrUY0CAxOjqn6mWTS+nc3rhT+ksyGl3rlRoQ+ ZLlw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:ironport-sdr:ironport-sdr; bh=1oFKlKK6wLbNkAX6XvaZ5OIpL+7gQEZwt71Au8+FRMQ=; b=QOLKdh1iPbcaXteZMziYugh2TIsyzpTtnjdPFZfvi4Sh5GxD/imEvolQDuTIe4gsuV L1tUSUnq85ygH/W72Ivm0CMm8qkczRRonVu2WDZKwpkSw0YLU1DBh1QZGluxu5Op1Q2f OBTowhDPGqtRtrSZNQp7MVCbat3mrqX6LSfL9FKvBdjIyRrGLQS2qp0ZT0bRmpOL5RqP au5KwMdxv5o3BYNU2Zp2sWmUSIBY47r6cniR6UxDSN4pr+C+VdAZE7y4raudx9e2uUuE lhNuve/2FDD1l8yuEmW8+C+Imjq7BvnZuLUzmkegWezOBhWnX5G0z1ftFrc9rpWUeg9E QiAQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ni7si13035356ejb.725.2020.07.21.16.47.19; Tue, 21 Jul 2020 16:48:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731560AbgGUXqO (ORCPT + 99 others); Tue, 21 Jul 2020 19:46:14 -0400 Received: from mga12.intel.com ([192.55.52.136]:44150 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731459AbgGUXqL (ORCPT ); Tue, 21 Jul 2020 19:46:11 -0400 IronPort-SDR: QofjuauXwdeRD+lvoHRO6ObQNubHBJ++pcLx5uSlSjaxNGA8JnOdmoxYk7g8Qjye6fGtqdVUmz dUAWyJIpHmGw== X-IronPort-AV: E=McAfee;i="6000,8403,9689"; a="129815373" X-IronPort-AV: E=Sophos;i="5.75,380,1589266800"; d="scan'208";a="129815373" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Jul 2020 16:46:11 -0700 IronPort-SDR: O+Zbmqj3QdWHOWpt1LnUKLcLzu7x9zcOB2XDaVewCuA3B352SntT5b+n2dzMbRz84RDYXShJYy g9EQedM6yeEw== X-IronPort-AV: E=Sophos;i="5.75,380,1589266800"; d="scan'208";a="362539392" Received: from krlloyd-mobl.amr.corp.intel.com (HELO [10.255.231.6]) ([10.255.231.6]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Jul 2020 16:46:10 -0700 Subject: Re: [RFC PATCH v2 6/6] ASoC: q6asm-dai: add support to set_codec_params To: Srinivas Kandagatla , broonie@kernel.org Cc: alsa-devel@alsa-project.org, ckeepax@opensource.cirrus.com, tiwai@suse.com, lgirdwood@gmail.com, linux-kernel@vger.kernel.org, vkoul@kernel.org References: <20200721170007.4554-1-srinivas.kandagatla@linaro.org> <20200721170007.4554-7-srinivas.kandagatla@linaro.org> From: Pierre-Louis Bossart Message-ID: <970e7065-4731-e553-f07f-896aaa57c5dc@linux.intel.com> Date: Tue, 21 Jul 2020 15:09:26 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20200721170007.4554-7-srinivas.kandagatla@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 7/21/20 12:00 PM, Srinivas Kandagatla wrote: > Make use of new set_codec_params callback to allow decoder switching > during gapless playback. > > Signed-off-by: Srinivas Kandagatla > --- > sound/soc/qcom/qdsp6/q6asm-dai.c | 33 ++++++++++++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/sound/soc/qcom/qdsp6/q6asm-dai.c b/sound/soc/qcom/qdsp6/q6asm-dai.c > index b5c719682919..a8cfb1996614 100644 > --- a/sound/soc/qcom/qdsp6/q6asm-dai.c > +++ b/sound/soc/qcom/qdsp6/q6asm-dai.c > @@ -876,6 +876,37 @@ static int __q6asm_dai_compr_set_codec_params(struct snd_soc_component *componen > return 0; > } > > +static int q6asm_dai_compr_set_codec_params(struct snd_soc_component *component, > + struct snd_compr_stream *stream, > + struct snd_codec *codec) > +{ > + struct snd_compr_runtime *runtime = stream->runtime; > + struct q6asm_dai_rtd *prtd = runtime->private_data; > + int ret; > + > + ret = q6asm_open_write(prtd->audio_client, prtd->next_track_stream_id, > + codec->id, codec->profile, prtd->bits_per_sample, > + true); > + if (ret < 0) { > + pr_err("q6asm_open_write failed\n"); > + return ret; > + } > + > + ret = __q6asm_dai_compr_set_codec_params(component, stream, codec, > + prtd->next_track_stream_id); > + if (ret < 0) { > + pr_err("q6asm_open_write failed\n"); > + return ret; > + } > + > + ret = q6asm_stream_remove_initial_silence(prtd->audio_client, > + prtd->next_track_stream_id, > + prtd->initial_samples_drop); > + prtd->next_track_stream_id = 0; same comment as in the other patchset, the stream_id toggles between 1 and 2, it's not clear to me what 0 means. off-by-one bug or feature?