CC: [email protected]
In-Reply-To: <[email protected]>
References: <[email protected]>
TO: Srinivas Kandagatla <[email protected]>
TO: [email protected]
TO: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]
CC: Srinivas Kandagatla <[email protected]>

Hi Srinivas,

I love your patch! Perhaps something to improve:

[auto build test WARNING on char-misc/char-misc-testing]
[also build test WARNING on robh/for-next linux/master linus/master v5.17-rc1 
next-20220127]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    
https://github.com/0day-ci/linux/commits/Srinivas-Kandagatla/misc-fastrpc-Add-missing-DSP-FastRPC-features/20220126-215705
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git 
515a2f507491e7c3818e74ef4f4e088c1fecb190
:::::: branch date: 20 hours ago
:::::: commit date: 20 hours ago
config: openrisc-randconfig-m031-20220124 
(https://download.01.org/0day-ci/archive/20220127/[email protected]/config)
compiler: or1k-linux-gcc (GCC) 11.2.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:
drivers/misc/fastrpc.c:1636 fastrpc_device_register() warn: passing devm_ 
allocated variable to kfree. 'fdev'

Old smatch warnings:
drivers/misc/fastrpc.c:504 fastrpc_context_alloc() warn: should 'ret << 4' be a 
64 bit type?

vim +/fdev +1636 drivers/misc/fastrpc.c

f6f9279f2bf0e37 Srinivas Kandagatla 2019-02-08  1619  
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1620  static int 
fastrpc_device_register(struct device *dev, struct fastrpc_channel_ctx *cctx,
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1621                            
   const char *domain)
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1622  {
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1623    struct fastrpc_device 
*fdev;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1624    int err;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1625  
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1626    fdev = 
devm_kzalloc(dev, sizeof(*fdev), GFP_KERNEL);
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1627    if (!fdev)
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1628            return -ENOMEM;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1629  
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1630    fdev->cctx = cctx;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1631    fdev->miscdev.minor = 
MISC_DYNAMIC_MINOR;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1632    fdev->miscdev.fops = 
&fastrpc_fops;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1633    fdev->miscdev.name = 
devm_kasprintf(dev, GFP_KERNEL, "fastrpc-%s", domain);
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1634    err = 
misc_register(&fdev->miscdev);
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1635    if (err)
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26 @1636            kfree(fdev);
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1637    else
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1638            cctx->fdevice = 
fdev;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1639  
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1640    return err;
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1641  }
99d9d7a1c5f2dae Srinivas Kandagatla 2022-01-26  1642  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to