Re: [alsa-devel] [PATCH v4 14/24] ASoC: qdsp6: q6asm: Add support to audio stream apis

Rohit Kumar
2018-04-09T11:53:58Z

On 3/10/2018 7:54 AM, srinivas.kandagatla@linaro.org wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>
> This patch adds support to open, write and media format commands
> in the q6asm module.
>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Reviewed-and-Tested-by: Rohit kumar <rohitkr@codeaurora.org>
> ---
> sound/soc/qcom/qdsp6/q6asm.c | 744 ++++++++++++++++++++++++++++++++++++++++++-
> sound/soc/qcom/qdsp6/q6asm.h | 49 +++
> 2 files changed, 792 insertions(+), 1 deletion(-)
>
> diff --git a/sound/soc/qcom/qdsp6/q6asm.c b/sound/soc/qcom/qdsp6/q6asm.c
> index 8a2ca2ff9ce8..ddbf526358bd 100644
> --- a/sound/soc/qcom/qdsp6/q6asm.c
> +++ b/sound/soc/qcom/qdsp6/q6asm.c
> @@ -9,6 +9,8 @@
> #include <linux/device.h>
> #include <linux/spinlock.h>
> #include <linux/of.h>
> +#include <linux/of_platform.h>
> +#include <uapi/sound/asound.h>
> #include <linux/delay.h>
> #include <linux/slab.h>
> #include <linux/mm.h>
> @@ -17,10 +19,36 @@
> #include "q6dsp-errno.h"
> #include "q6dsp-common.h"
>
> +#define ASM_STREAM_CMD_CLOSE 0x00010BCD
> +#define ASM_STREAM_CMD_FLUSH 0x00010BCE
> +#define ASM_SESSION_CMD_PAUSE 0x00010BD3
> +#define ASM_DATA_CMD_EOS 0x00010BDB
> +#define ASM_DEFAULT_POPP_TOPOLOGY 0x00010BE4
Use NULL topology instead of default topology.
#define ASM_NULL_POPP_TOPOLOGY                     0x00010C68
> +#define ASM_STREAM_CMD_FLUSH_READBUFS 0x00010C09
> +#define ASM_STREAM_CMD_SET_ENCDEC_PARAM 0x00010C10
[..]
> + q6asm_add_hdr(ac, &open.hdr, sizeof(open), true, ac->stream_id);
> +
> + open.hdr.opcode = ASM_STREAM_CMD_OPEN_WRITE_V3;
> + open.mode_flags = 0x00;
> + open.mode_flags |= ASM_LEGACY_STREAM_SESSION;
> +
> + /* source endpoint : matrix */
> + open.sink_endpointype = ASM_END_POINT_DEVICE_MATRIX;
> + open.bits_per_sample = bits_per_sample;
> + open.postprocopo_id = ASM_DEFAULT_POPP_TOPOLOGY;
open.postprocopo_id = ASM_NULL_POPP_TOPOLOGY;
> +
> + switch (format) {
> + case FORMAT_LINEAR_PCM:
> + open.dec_fmt_id = ASM_MEDIA_FMT_MULTI_CHANNEL_PCM_V2;
> + break;
> + default:
> + dev_err(ac->dev, "Invalid format 0x%x\n", format);
> + return -EINVAL;
> + }
> +
> + rc = q6asm_ac_send_cmd_sync(ac, &open);
> + if (rc < 0)
> + return rc;
> +
> + ac->io_mode |= ASM_TUN_WRITE_IO_MODE;
> +
> + return 0;
> +}
> +EXPORT_SYMBOL_GPL(q6asm_open_write);