lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202203170449.Gb8UPhN9-lkp@intel.com>
Date:   Thu, 17 Mar 2022 05:00:08 +0800
From:   kernel test robot <lkp@...el.com>
To:     Benoit Parrot <bparrot@...com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Nikhil Devshatwar <nikhil.nd@...com>
Subject: [pinchartl-media:am572x/vip 3/129]
 drivers/media/platform/ti-vpe/vip.c:892:22: warning: variable 'list_length'
 set but not used

tree:   git://linuxtv.org/pinchartl/media.git am572x/vip
head:   2a2995588382c94f9346c69248af9c716e532b9b
commit: e2cd6cd91c127527f1c3a04d236a968740d99a45 [3/129] media: ti-vpe: Add the VIP driver
config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20220317/202203170449.Gb8UPhN9-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add pinchartl-media git://linuxtv.org/pinchartl/media.git
        git fetch --no-tags pinchartl-media am572x/vip
        git checkout e2cd6cd91c127527f1c3a04d236a968740d99a45
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   drivers/media/platform/ti-vpe/vip.c: In function 'populate_desc_list':
>> drivers/media/platform/ti-vpe/vip.c:892:22: warning: variable 'list_length' set but not used [-Wunused-but-set-variable]
     892 |         unsigned int list_length;
         |                      ^~~~~~~~~~~
   drivers/media/platform/ti-vpe/vip.c: In function 'vip_try_fmt_vid_cap':
   drivers/media/platform/ti-vpe/vip.c:1635:35: error: implicit declaration of function 'v4l2_subdev_alloc_pad_config'; did you mean 'v4l2_subdev_alloc_state'? [-Werror=implicit-function-declaration]
    1635 |                         pad_cfg = v4l2_subdev_alloc_pad_config(port->subdev);
         |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
         |                                   v4l2_subdev_alloc_state
   drivers/media/platform/ti-vpe/vip.c:1635:33: warning: assignment to 'struct v4l2_subdev_pad_config *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
    1635 |                         pad_cfg = v4l2_subdev_alloc_pad_config(port->subdev);
         |                                 ^
   In file included from include/media/v4l2-device.h:13,
                    from drivers/media/platform/ti-vpe/vip.h:17,
                    from drivers/media/platform/ti-vpe/vip.c:28:
   drivers/media/platform/ti-vpe/vip.c:1642:48: error: passing argument 2 of '((const struct v4l2_subdev_pad_ops *)v4l2_subdev_call_wrappers.pad)->set_fmt' from incompatible pointer type [-Werror=incompatible-pointer-types]
    1642 |                                                pad_cfg, &format);
         |                                                ^~~~~~~
         |                                                |
         |                                                struct v4l2_subdev_pad_config *
   include/media/v4l2-subdev.h:1190:65: note: in definition of macro 'v4l2_subdev_call'
    1190 |                                                         __sd, ##args);  \
         |                                                                 ^~~~
   drivers/media/platform/ti-vpe/vip.c:1642:48: note: expected 'struct v4l2_subdev_state *' but argument is of type 'struct v4l2_subdev_pad_config *'
    1642 |                                                pad_cfg, &format);
         |                                                ^~~~~~~
   include/media/v4l2-subdev.h:1190:65: note: in definition of macro 'v4l2_subdev_call'
    1190 |                                                         __sd, ##args);  \
         |                                                                 ^~~~
   drivers/media/platform/ti-vpe/vip.c:1642:48: error: passing argument 2 of '((const struct v4l2_subdev_pad_ops *)__sd->ops->pad)->set_fmt' from incompatible pointer type [-Werror=incompatible-pointer-types]
    1642 |                                                pad_cfg, &format);
         |                                                ^~~~~~~
         |                                                |
         |                                                struct v4l2_subdev_pad_config *
   include/media/v4l2-subdev.h:1192:60: note: in definition of macro 'v4l2_subdev_call'
    1192 |                         __result = __sd->ops->o->f(__sd, ##args);       \
         |                                                            ^~~~
   drivers/media/platform/ti-vpe/vip.c:1642:48: note: expected 'struct v4l2_subdev_state *' but argument is of type 'struct v4l2_subdev_pad_config *'
    1642 |                                                pad_cfg, &format);
         |                                                ^~~~~~~
   include/media/v4l2-subdev.h:1192:60: note: in definition of macro 'v4l2_subdev_call'
    1192 |                         __result = __sd->ops->o->f(__sd, ##args);       \
         |                                                            ^~~~
   drivers/media/platform/ti-vpe/vip.c:1659:25: error: implicit declaration of function 'v4l2_subdev_free_pad_config' [-Werror=implicit-function-declaration]
    1659 |                         v4l2_subdev_free_pad_config(pad_cfg);
         |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/media/platform/ti-vpe/vip.c: In function 'free_port':
   drivers/media/platform/ti-vpe/vip.c:3614:9: error: implicit declaration of function 'v4l2_async_notifier_unregister'; did you mean 'v4l2_async_nf_unregister'? [-Werror=implicit-function-declaration]
    3614 |         v4l2_async_notifier_unregister(&port->notifier);
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         |         v4l2_async_nf_unregister
   drivers/media/platform/ti-vpe/vip.c:3615:9: error: implicit declaration of function 'v4l2_async_notifier_cleanup'; did you mean 'v4l2_async_nf_cleanup'? [-Werror=implicit-function-declaration]
    3615 |         v4l2_async_notifier_cleanup(&port->notifier);
         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~
         |         v4l2_async_nf_cleanup
   drivers/media/platform/ti-vpe/vip.c: In function 'vip_create_streams':
>> drivers/media/platform/ti-vpe/vip.c:3636:42: warning: variable 'bus' set but not used [-Wunused-but-set-variable]
    3636 |         struct v4l2_fwnode_bus_parallel *bus;
         |                                          ^~~
   drivers/media/platform/ti-vpe/vip.c: In function 'vip_register_subdev_notif':
   drivers/media/platform/ti-vpe/vip.c:3784:9: error: implicit declaration of function 'v4l2_async_notifier_init'; did you mean 'v4l2_async_nf_init'? [-Werror=implicit-function-declaration]
    3784 |         v4l2_async_notifier_init(notifier);
         |         ^~~~~~~~~~~~~~~~~~~~~~~~
         |         v4l2_async_nf_init
   drivers/media/platform/ti-vpe/vip.c:3786:15: error: implicit declaration of function 'v4l2_async_notifier_add_fwnode_subdev'; did you mean 'v4l2_async_nf_add_fwnode_remote'? [-Werror=implicit-function-declaration]
    3786 |         asd = v4l2_async_notifier_add_fwnode_subdev(notifier, subdev,
         |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         |               v4l2_async_nf_add_fwnode_remote
   drivers/media/platform/ti-vpe/vip.c:3786:13: warning: assignment to 'struct v4l2_async_subdev *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
    3786 |         asd = v4l2_async_notifier_add_fwnode_subdev(notifier, subdev,
         |             ^
   drivers/media/platform/ti-vpe/vip.c:3796:15: error: implicit declaration of function 'v4l2_async_notifier_register'; did you mean 'v4l2_async_nf_register'? [-Werror=implicit-function-declaration]
    3796 |         ret = v4l2_async_notifier_register(dev->v4l2_dev, notifier);
         |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
         |               v4l2_async_nf_register
   drivers/media/platform/ti-vpe/vip.c: In function 'vip_probe':
>> drivers/media/platform/ti-vpe/vip.c:4016:25: warning: variable 'pinctrl' set but not used [-Wunused-but-set-variable]
    4016 |         struct pinctrl *pinctrl;
         |                         ^~~~~~~
   drivers/media/platform/ti-vpe/vip.c: In function 'add_out_dtd':
   drivers/media/platform/ti-vpe/vip.c:722:23: warning: this statement may fall through [-Wimplicit-fallthrough=]
     722 |                 plane = 1;
         |                 ~~~~~~^~~
   drivers/media/platform/ti-vpe/vip.c:724:9: note: here
     724 |         case VIP_SRCE_LUMA:
         |         ^~~~
   cc1: some warnings being treated as errors


vim +/list_length +892 drivers/media/platform/ti-vpe/vip.c

   887	
   888	static void populate_desc_list(struct vip_stream *stream)
   889	{
   890		struct vip_port *port = stream->port;
   891		struct vip_dev *dev = port->dev;
 > 892		unsigned int list_length;
   893	
   894		stream->desc_next = stream->desc_list.buf.addr;
   895		add_stream_dtds(stream);
   896	
   897		list_length = stream->desc_next - stream->desc_list.buf.addr;
   898		vpdma_map_desc_buf(dev->shared->vpdma, &stream->desc_list.buf);
   899	}
   900	

---
0-DAY CI Kernel Test Service
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ