lkml.org 
[lkml]   [2018]   [May]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [alsa-devel] [PATCH v7 16/24] ASoC: qdsp6: q6asm: Add support to audio stream apis
From
Date


On 5/1/2018 5:38 PM, Srinivas Kandagatla wrote:
> +static int __q6asm_run(struct audio_client *ac, uint32_t flags,
> + uint32_t msw_ts, uint32_t lsw_ts, bool wait)
> +{
> + struct asm_session_cmd_run_v2 *run;
> + struct apr_pkt *pkt;
> + int pkt_size, rc;
> + void *p;
> +
> + pkt_size = APR_HDR_SIZE + sizeof(*run);
> + p = kzalloc(pkt_size, GFP_KERNEL);
Should be GFP_ATOMIC as this API is also called from interrupt context
> + if (!p)
> + return -ENOMEM;
> +
[..]
> +int q6asm_read(struct audio_client *ac)
> +{
> + struct asm_data_cmd_read_v2 *read;
> + struct audio_port_data *port;
> + struct audio_buffer *ab;
> + struct apr_pkt *pkt;
> + int pkt_size;
> + int rc = 0;
> + void *p;
> +
> + if (!(ac->io_mode & ASM_SYNC_IO_MODE))
> + return 0;
> +
> + pkt_size = APR_HDR_SIZE + sizeof(*read);
> + p = kzalloc(pkt_size, GFP_KERNEL);
same here. GFP_ATOMIC
> + if (!p)
> + return -ENOMEM;
[..]
> +int q6asm_write_async(struct audio_client *ac, uint32_t len, uint32_t msw_ts,
> + uint32_t lsw_ts, uint32_t flags)
> +{
> + struct asm_data_cmd_write_v2 *write;
> + struct audio_port_data *port;
> + struct audio_buffer *ab;
> + struct apr_pkt *pkt;
> + int pkt_size;
> + int rc = 0;
> + void *p;
> +
> + pkt_size = APR_HDR_SIZE + sizeof(*write);
> + p = kzalloc(pkt_size, GFP_KERNEL);
GFP_ATOMIC
> + if (!p)
> + return -ENOMEM;
>
Thanks,
Rohit

--
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.

\
 
 \ /
  Last update: 2018-05-04 09:13    [W:0.351 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site