CC: [email protected]
CC: [email protected]
TO: Srinivas Kandagatla <[email protected]>
CC: Mark Brown <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   61556703b610a104de324e4f061dc6cf7b218b46
commit: 84ab3b9f19f6ff0bb5df6c6deea75ab4c1d2aff8 ASoC: q6dsp: q6afe: update 
q6afe_set_param to support global clocks
date:   5 months ago
:::::: branch date: 28 hours ago
:::::: commit date: 5 months ago
config: h8300-randconfig-m031-20210205 (attached as .config)
compiler: h8300-linux-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

New smatch warnings:
sound/soc/qcom/qdsp6/q6afe.c:991 q6afe_set_param() warn: is 'p' large enough 
for 'struct apr_pkt'? 0

Old smatch warnings:
sound/soc/qcom/qdsp6/q6afe.c:1045 q6afe_port_set_param_v2() warn: is 'p' large 
enough for 'struct apr_pkt'? 0

vim +991 sound/soc/qcom/qdsp6/q6afe.c

7fa2d70f976657 Srinivas Kandagatla 2018-05-18   969  
84ab3b9f19f6ff Srinivas Kandagatla 2020-09-10   970  static int 
q6afe_set_param(struct q6afe *afe, struct q6afe_port *port,
84ab3b9f19f6ff Srinivas Kandagatla 2020-09-10   971                        void 
*data, int param_id, int module_id, int psize,
84ab3b9f19f6ff Srinivas Kandagatla 2020-09-10   972                        int 
token)
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   973  {
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   974     struct 
afe_svc_cmd_set_param *param;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   975     struct 
afe_port_param_data_v2 *pdata;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   976     struct apr_pkt *pkt;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   977     int ret, pkt_size;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   978     void *p, *pl;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   979  
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   980     pkt_size = APR_HDR_SIZE 
+ sizeof(*param) + sizeof(*pdata) + psize;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   981     p = kzalloc(pkt_size, 
GFP_KERNEL);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   982     if (!p)
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   983             return -ENOMEM;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   984  
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   985     pkt = p;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   986     param = p + 
APR_HDR_SIZE;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   987     pdata = p + 
APR_HDR_SIZE + sizeof(*param);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   988     pl = p + APR_HDR_SIZE + 
sizeof(*param) + sizeof(*pdata);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   989     memcpy(pl, data, psize);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   990  
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  @991     pkt->hdr.hdr_field = 
APR_HDR_FIELD(APR_MSG_TYPE_SEQ_CMD,
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   992                             
           APR_HDR_LEN(APR_HDR_SIZE),
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   993                             
           APR_PKT_VER);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   994     pkt->hdr.pkt_size = 
pkt_size;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   995     pkt->hdr.src_port = 0;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   996     pkt->hdr.dest_port = 0;
84ab3b9f19f6ff Srinivas Kandagatla 2020-09-10   997     pkt->hdr.token = token;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   998     pkt->hdr.opcode = 
AFE_SVC_CMD_SET_PARAM;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18   999  
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1000     param->payload_size = 
sizeof(*pdata) + psize;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1001     
param->payload_address_lsw = 0x00;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1002     
param->payload_address_msw = 0x00;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1003     param->mem_map_handle = 
0x00;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1004     pdata->module_id = 
module_id;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1005     pdata->param_id = 
param_id;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1006     pdata->param_size = 
psize;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1007  
342a4f8ca12b1c Srinivas Kandagatla 2020-09-10  1008     ret = 
afe_apr_send_pkt(afe, pkt, port, AFE_SVC_CMD_SET_PARAM);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1009     if (ret)
84ab3b9f19f6ff Srinivas Kandagatla 2020-09-10  1010             
dev_err(afe->dev, "AFE set params failed %d\n", ret);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1011  
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1012     kfree(pkt);
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1013     return ret;
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1014  }
a4ae3af59bd585 Srinivas Kandagatla 2018-05-18  1015  

:::::: The code at line 991 was first introduced by commit
:::::: a4ae3af59bd58514b5417d7d802a37e82bfcfd8f ASoC: qdsp6: q6afe: Add support 
to MI2S sysclks

:::::: TO: Srinivas Kandagatla <[email protected]>
:::::: CC: Mark Brown <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to