linux-kernel - Re: [PATCH] media: i2c: adv748x: add enuminput control to adv748x hdmi subdev

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 16 Jun 2020 00:12:57 +0800
From:   kernel test robot <lkp@...el.com>
To:     Ramzi BEN MEFTAH <rbmeftah@...adit-jv.com>,
        Kieran Bingham <kieran.bingham@...asonboard.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        linux-kernel@...r.kernel.org
Cc:     kbuild-all@...ts.01.org, linux-media@...r.kernel.org,
        Michael Rodin <mrodin@...adit-jv.com>,
        efriedrich@...adit-jv.com, erosca@...adit-jv.com
Subject: Re: [PATCH] media: i2c: adv748x: add enuminput control to adv748x
 hdmi subdev

Hi Ramzi,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on v5.8-rc1 next-20200615]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Ramzi-BEN-MEFTAH/media-i2c-adv748x-add-enuminput-control-to-adv748x-hdmi-subdev/20200615-212859
base:   git://linuxtv.org/media_tree.git master
config: i386-randconfig-a016-20200615 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0
reproduce (this is a W=1 build):
        # save the attached .config to linux build tree
        make W=1 ARCH=i386 

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

All error/warnings (new ones prefixed by >>, old ones prefixed by <<):

>> drivers/media/i2c/adv748x/adv748x-hdmi.c:405:3: error: 'const struct v4l2_subdev_video_ops' has no member named 'enuminput'
405 |  .enuminput = adv748x_hdmi_enuminput,
|   ^~~~~~~~~
>> drivers/media/i2c/adv748x/adv748x-hdmi.c:405:15: error: initialization of 'int (*)(struct v4l2_subdev *, int)' from incompatible pointer type 'int (*)(struct v4l2_subdev *, struct v4l2_input *)' [-Werror=incompatible-pointer-types]
405 |  .enuminput = adv748x_hdmi_enuminput,
|               ^~~~~~~~~~~~~~~~~~~~~~
drivers/media/i2c/adv748x/adv748x-hdmi.c:405:15: note: (near initialization for 'adv748x_video_ops_hdmi.s_stream')
>> drivers/media/i2c/adv748x/adv748x-hdmi.c:406:14: warning: initialized field overwritten [-Woverride-init]
406 |  .s_stream = adv748x_hdmi_s_stream,
|              ^~~~~~~~~~~~~~~~~~~~~
drivers/media/i2c/adv748x/adv748x-hdmi.c:406:14: note: (near initialization for 'adv748x_video_ops_hdmi.s_stream')
cc1: some warnings being treated as errors

vim +405 drivers/media/i2c/adv748x/adv748x-hdmi.c

   399	
   400	static const struct v4l2_subdev_video_ops adv748x_video_ops_hdmi = {
   401		.s_dv_timings = adv748x_hdmi_s_dv_timings,
   402		.g_dv_timings = adv748x_hdmi_g_dv_timings,
   403		.query_dv_timings = adv748x_hdmi_query_dv_timings,
   404		.g_input_status = adv748x_hdmi_g_input_status,
 > 405		.enuminput = adv748x_hdmi_enuminput,
 > 406		.s_stream = adv748x_hdmi_s_stream,
   407		.g_pixelaspect = adv748x_hdmi_g_pixelaspect,
   408	};
   409	

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

Download attachment ".config.gz" of type "application/gzip" (39862 bytes)