CC: [email protected]
BCC: [email protected]
CC: [email protected]
TO: Tomi Valkeinen <[email protected]>
CC: Mauro Carvalho Chehab <[email protected]>
CC: [email protected]
CC: Hans Verkuil <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   ed4643521e6af8ab8ed1e467630a85884d2696cf
commit: 0d346d2a6f54f06f36b224fd27cd6eafe8c83be9 media: v4l2-subdev: add 
subdev-wide state struct
date:   9 months ago
:::::: branch date: 5 hours ago
:::::: commit date: 9 months ago
config: arm-randconfig-m031-20220323 
(https://download.01.org/0day-ci/archive/20220324/[email protected]/config)
compiler: arm-linux-gnueabi-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]>

smatch warnings:
drivers/media/v4l2-core/v4l2-subdev.c:907 v4l2_subdev_alloc_state() warn: 
Please consider using kvcalloc instead

vim +907 drivers/media/v4l2-core/v4l2-subdev.c

9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  896  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  897  struct v4l2_subdev_state 
*v4l2_subdev_alloc_state(struct v4l2_subdev *sd)
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  898  {
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  899        struct 
v4l2_subdev_state *state;
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  900        int ret;
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  901  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  902        state = 
kzalloc(sizeof(*state), GFP_KERNEL);
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  903        if (!state)
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  904                return 
ERR_PTR(-ENOMEM);
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  905  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  906        if 
(sd->entity.num_pads) {
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10 @907                state->pads = 
kvmalloc_array(sd->entity.num_pads,
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  908                                
             sizeof(*state->pads),
758d90e161382c1 Tomasz Figa      2017-06-19  909                                
             GFP_KERNEL | __GFP_ZERO);
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  910                if 
(!state->pads) {
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  911                        ret = 
-ENOMEM;
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  912                        goto 
err;
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  913                }
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  914        }
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  915  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  916        ret = 
v4l2_subdev_call(sd, pad, init_cfg, state);
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  917        if (ret < 0 && ret != 
-ENOIOCTLCMD)
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  918                goto err;
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  919  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  920        return state;
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  921  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  922  err:
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  923        if (state && 
state->pads)
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  924                
kvfree(state->pads);
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  925  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  926        kfree(state);
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  927  
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  928        return ERR_PTR(ret);
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  929  }
0d346d2a6f54f06 Tomi Valkeinen   2021-06-10  930  
EXPORT_SYMBOL_GPL(v4l2_subdev_alloc_state);
9b02cbb3ede89b5 Laurent Pinchart 2015-04-24  931  

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

Reply via email to