From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
To: Vinod Koul <vkoul@kernel.org>, Mark Brown <broonie@kernel.org>,
Takashi Iwai <tiwai@suse.com>
Cc: linux-arm-msm@vger.kernel.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Patrick Lai <plai@codeaurora.org>,
Banajit Goswami <bgoswami@codeaurora.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Jaroslav Kysela <perex@perex.cz>,
alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [RESEND PATCH v2 4/9] ASoC: qcom: q6asm: add support to wma config
Date: Fri, 13 Mar 2020 12:15:23 +0000 [thread overview]
Message-ID: <cda84c80-a8bb-ecb5-3bda-ec1d610fd374@linaro.org> (raw)
In-Reply-To: <20200313101627.1561365-5-vkoul@kernel.org>
On 13/03/2020 10:16, Vinod Koul wrote:
> Qualcomm DSPs expect wma v9 and wma v10 configs to be set for wma
> decoders, so add the API to program the respective wma config to the DSP
>
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
Reviewed-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
> sound/soc/qcom/qdsp6/q6asm.c | 123 +++++++++++++++++++++++++++++++++++
> sound/soc/qcom/qdsp6/q6asm.h | 17 +++++
> 2 files changed, 140 insertions(+)
>
> diff --git a/sound/soc/qcom/qdsp6/q6asm.c b/sound/soc/qcom/qdsp6/q6asm.c
> index 64eb7b6ba305..4cec95c657ba 100644
> --- a/sound/soc/qcom/qdsp6/q6asm.c
> +++ b/sound/soc/qcom/qdsp6/q6asm.c
> @@ -39,6 +39,8 @@
> #define ASM_MEDIA_FMT_MULTI_CHANNEL_PCM_V2 0x00010DA5
> #define ASM_MEDIA_FMT_MP3 0x00010BE9
> #define ASM_MEDIA_FMT_FLAC 0x00010C16
> +#define ASM_MEDIA_FMT_WMA_V9 0x00010DA8
> +#define ASM_MEDIA_FMT_WMA_V10 0x00010DA7
> #define ASM_DATA_CMD_WRITE_V2 0x00010DAB
> #define ASM_DATA_CMD_READ_V2 0x00010DAC
> #define ASM_SESSION_CMD_SUSPEND 0x00010DEC
> @@ -104,6 +106,33 @@ struct asm_flac_fmt_blk_v2 {
> u16 reserved;
> } __packed;
>
> +struct asm_wmastdv9_fmt_blk_v2 {
> + struct asm_data_cmd_media_fmt_update_v2 fmt_blk;
> + u16 fmtag;
> + u16 num_channels;
> + u32 sample_rate;
> + u32 bytes_per_sec;
> + u16 blk_align;
> + u16 bits_per_sample;
> + u32 channel_mask;
> + u16 enc_options;
> + u16 reserved;
> +} __packed;
> +
> +struct asm_wmaprov10_fmt_blk_v2 {
> + struct asm_data_cmd_media_fmt_update_v2 fmt_blk;
> + u16 fmtag;
> + u16 num_channels;
> + u32 sample_rate;
> + u32 bytes_per_sec;
> + u16 blk_align;
> + u16 bits_per_sample;
> + u32 channel_mask;
> + u16 enc_options;
> + u16 advanced_enc_options1;
> + u32 advanced_enc_options2;
> +} __packed;
> +
> struct asm_stream_cmd_set_encdec_param {
> u32 param_id;
> u32 param_size;
> @@ -894,6 +923,24 @@ int q6asm_open_write(struct audio_client *ac, uint32_t format,
> case SND_AUDIOCODEC_FLAC:
> open->dec_fmt_id = ASM_MEDIA_FMT_FLAC;
> break;
> + case SND_AUDIOCODEC_WMA:
> + switch (codec_profile) {
> + case SND_AUDIOPROFILE_WMA9:
> + open->dec_fmt_id = ASM_MEDIA_FMT_WMA_V9;
> + break;
> + case SND_AUDIOPROFILE_WMA10:
> + case SND_AUDIOPROFILE_WMA9_PRO:
> + case SND_AUDIOPROFILE_WMA9_LOSSLESS:
> + case SND_AUDIOPROFILE_WMA10_LOSSLESS:
> + open->dec_fmt_id = ASM_MEDIA_FMT_WMA_V10;
> + break;
> + default:
> + dev_err(ac->dev, "Invalid codec profile 0x%x\n",
> + codec_profile);
> + rc = -EINVAL;
> + goto err;
> + }
> + break;
> default:
> dev_err(ac->dev, "Invalid format 0x%x\n", format);
> rc = -EINVAL;
> @@ -1075,6 +1122,82 @@ int q6asm_stream_media_format_block_flac(struct audio_client *ac,
> return rc;
> }
> EXPORT_SYMBOL_GPL(q6asm_stream_media_format_block_flac);
> +
> +int q6asm_stream_media_format_block_wma_v9(struct audio_client *ac,
> + struct q6asm_wma_cfg *cfg)
> +{
> + struct asm_wmastdv9_fmt_blk_v2 *fmt;
> + struct apr_pkt *pkt;
> + void *p;
> + int rc, pkt_size;
> +
> + pkt_size = APR_HDR_SIZE + sizeof(*fmt);
> + p = kzalloc(pkt_size, GFP_KERNEL);
> + if (!p)
> + return -ENOMEM;
> +
> + pkt = p;
> + fmt = p + APR_HDR_SIZE;
> +
> + q6asm_add_hdr(ac, &pkt->hdr, pkt_size, true, ac->stream_id);
> +
> + pkt->hdr.opcode = ASM_DATA_CMD_MEDIA_FMT_UPDATE_V2;
> + fmt->fmt_blk.fmt_blk_size = sizeof(*fmt) - sizeof(fmt->fmt_blk);
> + fmt->fmtag = cfg->fmtag;
> + fmt->num_channels = cfg->num_channels;
> + fmt->sample_rate = cfg->sample_rate;
> + fmt->bytes_per_sec = cfg->bytes_per_sec;
> + fmt->blk_align = cfg->block_align;
> + fmt->bits_per_sample = cfg->bits_per_sample;
> + fmt->channel_mask = cfg->channel_mask;
> + fmt->enc_options = cfg->enc_options;
> + fmt->reserved = 0;
> +
> + rc = q6asm_ac_send_cmd_sync(ac, pkt);
> + kfree(pkt);
> +
> + return rc;
> +}
> +EXPORT_SYMBOL_GPL(q6asm_stream_media_format_block_wma_v9);
> +
> +int q6asm_stream_media_format_block_wma_v10(struct audio_client *ac,
> + struct q6asm_wma_cfg *cfg)
> +{
> + struct asm_wmaprov10_fmt_blk_v2 *fmt;
> + struct apr_pkt *pkt;
> + void *p;
> + int rc, pkt_size;
> +
> + pkt_size = APR_HDR_SIZE + sizeof(*fmt);
> + p = kzalloc(pkt_size, GFP_KERNEL);
> + if (!p)
> + return -ENOMEM;
> +
> + pkt = p;
> + fmt = p + APR_HDR_SIZE;
> +
> + q6asm_add_hdr(ac, &pkt->hdr, pkt_size, true, ac->stream_id);
> +
> + pkt->hdr.opcode = ASM_DATA_CMD_MEDIA_FMT_UPDATE_V2;
> + fmt->fmt_blk.fmt_blk_size = sizeof(*fmt) - sizeof(fmt->fmt_blk);
> + fmt->fmtag = cfg->fmtag;
> + fmt->num_channels = cfg->num_channels;
> + fmt->sample_rate = cfg->sample_rate;
> + fmt->bytes_per_sec = cfg->bytes_per_sec;
> + fmt->blk_align = cfg->block_align;
> + fmt->bits_per_sample = cfg->bits_per_sample;
> + fmt->channel_mask = cfg->channel_mask;
> + fmt->enc_options = cfg->enc_options;
> + fmt->advanced_enc_options1 = cfg->adv_enc_options;
> + fmt->advanced_enc_options2 = cfg->adv_enc_options2;
> +
> + rc = q6asm_ac_send_cmd_sync(ac, pkt);
> + kfree(pkt);
> +
> + return rc;
> +}
> +EXPORT_SYMBOL_GPL(q6asm_stream_media_format_block_wma_v10);
> +
> /**
> * q6asm_enc_cfg_blk_pcm_format_support() - setup pcm configuration for capture
> *
> diff --git a/sound/soc/qcom/qdsp6/q6asm.h b/sound/soc/qcom/qdsp6/q6asm.h
> index 1cff7f68b95d..5d9fbc75688c 100644
> --- a/sound/soc/qcom/qdsp6/q6asm.h
> +++ b/sound/soc/qcom/qdsp6/q6asm.h
> @@ -45,6 +45,19 @@ struct q6asm_flac_cfg {
> u16 md5_sum;
> };
>
> +struct q6asm_wma_cfg {
> + u32 fmtag;
> + u32 num_channels;
> + u32 sample_rate;
> + u32 bytes_per_sec;
> + u32 block_align;
> + u32 bits_per_sample;
> + u32 channel_mask;
> + u32 enc_options;
> + u32 adv_enc_options;
> + u32 adv_enc_options2;
> +};
> +
> typedef void (*q6asm_cb) (uint32_t opcode, uint32_t token,
> void *payload, void *priv);
> struct audio_client;
> @@ -69,6 +82,10 @@ int q6asm_media_format_block_multi_ch_pcm(struct audio_client *ac,
> uint16_t bits_per_sample);
> int q6asm_stream_media_format_block_flac(struct audio_client *ac,
> struct q6asm_flac_cfg *cfg);
> +int q6asm_stream_media_format_block_wma_v9(struct audio_client *ac,
> + struct q6asm_wma_cfg *cfg);
> +int q6asm_stream_media_format_block_wma_v10(struct audio_client *ac,
> + struct q6asm_wma_cfg *cfg);
> int q6asm_run(struct audio_client *ac, uint32_t flags, uint32_t msw_ts,
> uint32_t lsw_ts);
> int q6asm_run_nowait(struct audio_client *ac, uint32_t flags, uint32_t msw_ts,
>
next prev parent reply other threads:[~2020-03-13 12:15 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-13 10:16 [RESEND PATCH v2 0/9] ALSA: compress: Add wma, alac and ape support Vinod Koul
2020-03-13 10:16 ` [RESEND PATCH v2 1/9] ALSA: compress: add wma codec profiles Vinod Koul
2020-03-13 10:16 ` [RESEND PATCH v2 2/9] ALSA: compress: Add wma decoder params Vinod Koul
2020-03-13 10:16 ` [RESEND PATCH v2 3/9] ASoC: qcom: q6asm: pass codec profile to q6asm_open_write Vinod Koul
2020-03-13 12:15 ` Srinivas Kandagatla
2020-03-13 10:16 ` [RESEND PATCH v2 4/9] ASoC: qcom: q6asm: add support to wma config Vinod Koul
2020-03-13 12:15 ` Srinivas Kandagatla [this message]
2020-03-13 10:16 ` [RESEND PATCH v2 5/9] ASoC: qcom: q6asm-dai: add support to wma decoder Vinod Koul
2020-03-13 12:15 ` Srinivas Kandagatla
2020-03-13 10:16 ` [RESEND PATCH v2 6/9] ALSA: compress: add alac & ape decoder params Vinod Koul
2020-03-13 10:16 ` [RESEND PATCH v2 7/9] ASoC: qcom: q6asm: add support for alac and ape configs Vinod Koul
2020-03-13 12:15 ` Srinivas Kandagatla
2020-03-13 10:16 ` [RESEND PATCH v2 8/9] ASoC: qcom: q6asm-dai: add support for ALAC and APE decoders Vinod Koul
2020-03-13 12:15 ` Srinivas Kandagatla
2020-03-13 12:40 ` Vinod Koul
2020-03-13 10:16 ` [RESEND PATCH v2 9/9] ALSA: compress: bump the version Vinod Koul
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cda84c80-a8bb-ecb5-3bda-ec1d610fd374@linaro.org \
--to=srinivas.kandagatla@linaro.org \
--cc=alsa-devel@alsa-project.org \
--cc=bgoswami@codeaurora.org \
--cc=bjorn.andersson@linaro.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=perex@perex.cz \
--cc=plai@codeaurora.org \
--cc=tiwai@suse.com \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®