From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EE525C2BB1D for ; Fri, 13 Mar 2020 12:15:41 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C6B7320724 for ; Fri, 13 Mar 2020 12:15:41 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="ivrnIzrt" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726776AbgCMMPk (ORCPT ); Fri, 13 Mar 2020 08:15:40 -0400 Received: from mail-wm1-f67.google.com ([209.85.128.67]:37370 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726633AbgCMMPk (ORCPT ); Fri, 13 Mar 2020 08:15:40 -0400 Received: by mail-wm1-f67.google.com with SMTP id a141so10043208wme.2 for ; Fri, 13 Mar 2020 05:15:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=bEk7S7Dnd0BzCyFDN1a5hqfpt0sia9NmxQj/OgqMtaY=; b=ivrnIzrtSb317R27s6UIuB8l1eRMFATY6xEueaZ9KFH+WmhD6bTED40vT+v2Iw+RIT +EVGP7Dblt0JJSWK97d1PnXU9wER9/oKLqyqwtD7B84efCSQtQCHA8jaJBpnSf66/qIY 3HS86wRhMXfS7F4BHpu5Njizex3YV3/kwUq5iDgd/6OJ/vAh5NP7Z3VvwEC5UMeEIiyf vtvv1LEKrW/DZIVP6V+anPSt97bHI6iGJRqYCzc5ErM6NJmnZwMy0S9J79jfjq6IXtFj UcAVJXqKp5P/6bMtMdWPXl86CIF3O4K/FxOCeXjs1v1/ZrRN08Toh4KimCwUjhdsF8Bp BfvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=bEk7S7Dnd0BzCyFDN1a5hqfpt0sia9NmxQj/OgqMtaY=; b=Qc0Z8O4gmfDBaOWlIASbXs+FCG+R8+LldPhSaSrQhUAktvE5ktKvOfgez3nwKPdoIR xZFbLF/2PGfDWeHvP91A/izHoZBylPsBA1tYkBrfbR5EO6+P5REajf6yBNTMqlRLvHab 6ZJisnt4zEpWvijdJCX1j/xLDlKThg5/x2cVvu6hTws7tgEFluRWwfBifIkrDVoFk0QS F/YzKMJsavVM1d+aPXjeGjYNy1GMpMlEm9UdVobhTUapABrz0prrR2SnsEjYV9Gx6Ci0 ZNazVyF5K60mSR1AQoWgNWhLrrBVcIlpLNS+3U2TclE5bBA5GWKvLpoDDcDLrn4AbIvU cIig== X-Gm-Message-State: ANhLgQ1Q8TU/0yqg36uyFyS7AAyYW9G707EslGIJh5trqF7wxrWUINxc MDrNkaHomMJ48DcbZKWa36ThQrn91pA= X-Google-Smtp-Source: ADFU+vsU8bP/cl627H526e80+fYdUA5wijE1PisL9EbmsVGYHyWfszDXSnlk7wtJ3xCHDtXQZgg1/Q== X-Received: by 2002:a05:600c:208:: with SMTP id 8mr10952728wmi.79.1584101735900; Fri, 13 Mar 2020 05:15:35 -0700 (PDT) Received: from [192.168.86.34] (cpc89974-aztw32-2-0-cust43.18-1.cable.virginm.net. [86.30.250.44]) by smtp.googlemail.com with ESMTPSA id x9sm40720370wrx.0.2020.03.13.05.15.34 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 13 Mar 2020 05:15:35 -0700 (PDT) Subject: Re: [RESEND PATCH v2 5/9] ASoC: qcom: q6asm-dai: add support to wma decoder To: Vinod Koul , Mark Brown , Takashi Iwai Cc: linux-arm-msm@vger.kernel.org, Bjorn Andersson , Patrick Lai , Banajit Goswami , Liam Girdwood , Jaroslav Kysela , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org References: <20200313101627.1561365-1-vkoul@kernel.org> <20200313101627.1561365-6-vkoul@kernel.org> From: Srinivas Kandagatla Message-ID: Date: Fri, 13 Mar 2020 12:15:34 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <20200313101627.1561365-6-vkoul@kernel.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 13/03/2020 10:16, Vinod Koul wrote: > Qualcomm DSPs also supports the wma decoder, so add support for wma > decoder and convert the snd_codec_params to qdsp format. > > Signed-off-by: Vinod Koul Reviewed-by: Srinivas Kandagatla > --- > sound/soc/qcom/qdsp6/q6asm-dai.c | 67 +++++++++++++++++++++++++++++++- > 1 file changed, 66 insertions(+), 1 deletion(-) > > diff --git a/sound/soc/qcom/qdsp6/q6asm-dai.c b/sound/soc/qcom/qdsp6/q6asm-dai.c > index 8f245d03b6f5..53c250778eea 100644 > --- a/sound/soc/qcom/qdsp6/q6asm-dai.c > +++ b/sound/soc/qcom/qdsp6/q6asm-dai.c > @@ -627,10 +627,13 @@ static int q6asm_dai_compr_set_params(struct snd_compr_stream *stream, > int dir = stream->direction; > struct q6asm_dai_data *pdata; > struct q6asm_flac_cfg flac_cfg; > + struct q6asm_wma_cfg wma_cfg; > + unsigned int wma_v9 = 0; > struct device *dev = c->dev; > int ret; > union snd_codec_options *codec_options; > struct snd_dec_flac *flac; > + struct snd_dec_wma *wma; > > codec_options = &(prtd->codec_param.codec.options); > > @@ -692,6 +695,67 @@ static int q6asm_dai_compr_set_params(struct snd_compr_stream *stream, > return -EIO; > } > break; > + > + case SND_AUDIOCODEC_WMA: > + wma = &codec_options->wma_d; > + > + memset(&wma_cfg, 0x0, sizeof(struct q6asm_wma_cfg)); > + > + wma_cfg.sample_rate = params->codec.sample_rate; > + wma_cfg.num_channels = params->codec.ch_in; > + wma_cfg.bytes_per_sec = params->codec.bit_rate / 8; > + wma_cfg.block_align = params->codec.align; > + wma_cfg.bits_per_sample = prtd->bits_per_sample; > + wma_cfg.enc_options = wma->encoder_option; > + wma_cfg.adv_enc_options = wma->adv_encoder_option; > + wma_cfg.adv_enc_options2 = wma->adv_encoder_option2; > + > + if (wma_cfg.num_channels == 1) > + wma_cfg.channel_mask = 4; /* Mono Center */ > + else if (wma_cfg.num_channels == 2) > + wma_cfg.channel_mask = 3; /* Stereo FL/FR */ > + else > + return -EINVAL; > + > + /* check the codec profile */ > + switch (params->codec.profile) { > + case SND_AUDIOPROFILE_WMA9: > + wma_cfg.fmtag = 0x161; > + wma_v9 = 1; > + break; > + > + case SND_AUDIOPROFILE_WMA10: > + wma_cfg.fmtag = 0x166; > + break; > + > + case SND_AUDIOPROFILE_WMA9_PRO: > + wma_cfg.fmtag = 0x162; > + break; > + > + case SND_AUDIOPROFILE_WMA9_LOSSLESS: > + wma_cfg.fmtag = 0x163; > + break; > + > + case SND_AUDIOPROFILE_WMA10_LOSSLESS: > + wma_cfg.fmtag = 0x167; > + break; > + > + default: > + dev_err(dev, "Unknown WMA profile:%x\n", > + params->codec.profile); > + return -EIO; > + } > + > + if (wma_v9) > + ret = q6asm_stream_media_format_block_wma_v9( > + prtd->audio_client, &wma_cfg); > + else > + ret = q6asm_stream_media_format_block_wma_v10( > + prtd->audio_client, &wma_cfg); > + if (ret < 0) { > + dev_err(dev, "WMA9 CMD failed:%d\n", ret); > + return -EIO; > + } > default: > break; > } > @@ -791,9 +855,10 @@ static int q6asm_dai_compr_get_caps(struct snd_compr_stream *stream, > caps->max_fragment_size = COMPR_PLAYBACK_MAX_FRAGMENT_SIZE; > caps->min_fragments = COMPR_PLAYBACK_MIN_NUM_FRAGMENTS; > caps->max_fragments = COMPR_PLAYBACK_MAX_NUM_FRAGMENTS; > - caps->num_codecs = 2; > + caps->num_codecs = 3; > caps->codecs[0] = SND_AUDIOCODEC_MP3; > caps->codecs[1] = SND_AUDIOCODEC_FLAC; > + caps->codecs[2] = SND_AUDIOCODEC_WMA; > > return 0; > } >