Re: [PATCH v6 2/5] media: v4l2: Extend VIDIOC_ENUM_FMT to support MC-centric devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

I love your patch! Perhaps something to improve:

[auto build test WARNING on v5.6-rc6]
[also build test WARNING on next-20200318]
[cannot apply to linuxtv-media/master]
[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/Laurent-Pinchart/v4l2-dev-ioctl-Add-V4L2_CAP_IO_MC/20200319-084846
base:    fb33c6510d5595144d585aa194d377cf74d31911
config: parisc-allyesconfig (attached as .config)
compiler: hppa-linux-gcc (GCC) 9.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=9.2.0 make.cross ARCH=parisc 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

   drivers/media//pci/cx18/cx18-ioctl.c: In function 'cx18_enum_fmt_vid_cap':
>> drivers/media//pci/cx18/cx18-ioctl.c:470:5: warning: braces around scalar initializer
     470 |     "HM12 (YUV 4:1:1)", V4L2_PIX_FMT_HM12, { 0, 0, 0, 0 }
         |     ^~~~~~~~~~~~~~~~~~
   drivers/media//pci/cx18/cx18-ioctl.c:470:5: note: (near initialization for 'formats[0].mbus_code')
>> drivers/media//pci/cx18/cx18-ioctl.c:470:49: warning: excess elements in scalar initializer
     470 |     "HM12 (YUV 4:1:1)", V4L2_PIX_FMT_HM12, { 0, 0, 0, 0 }
         |                                                 ^
   drivers/media//pci/cx18/cx18-ioctl.c:470:49: note: (near initialization for 'formats[0].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:470:52: warning: excess elements in scalar initializer
     470 |     "HM12 (YUV 4:1:1)", V4L2_PIX_FMT_HM12, { 0, 0, 0, 0 }
         |                                                    ^
   drivers/media//pci/cx18/cx18-ioctl.c:470:52: note: (near initialization for 'formats[0].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:470:55: warning: excess elements in scalar initializer
     470 |     "HM12 (YUV 4:1:1)", V4L2_PIX_FMT_HM12, { 0, 0, 0, 0 }
         |                                                       ^
   drivers/media//pci/cx18/cx18-ioctl.c:470:55: note: (near initialization for 'formats[0].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:473:5: warning: braces around scalar initializer
     473 |     "MPEG", V4L2_PIX_FMT_MPEG, { 0, 0, 0, 0 }
         |     ^~~~~~
   drivers/media//pci/cx18/cx18-ioctl.c:473:5: note: (near initialization for 'formats[1].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:473:37: warning: excess elements in scalar initializer
     473 |     "MPEG", V4L2_PIX_FMT_MPEG, { 0, 0, 0, 0 }
         |                                     ^
   drivers/media//pci/cx18/cx18-ioctl.c:473:37: note: (near initialization for 'formats[1].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:473:40: warning: excess elements in scalar initializer
     473 |     "MPEG", V4L2_PIX_FMT_MPEG, { 0, 0, 0, 0 }
         |                                        ^
   drivers/media//pci/cx18/cx18-ioctl.c:473:40: note: (near initialization for 'formats[1].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:473:43: warning: excess elements in scalar initializer
     473 |     "MPEG", V4L2_PIX_FMT_MPEG, { 0, 0, 0, 0 }
         |                                           ^
   drivers/media//pci/cx18/cx18-ioctl.c:473:43: note: (near initialization for 'formats[1].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:476:5: warning: braces around scalar initializer
     476 |     "UYVY 4:2:2", V4L2_PIX_FMT_UYVY, { 0, 0, 0, 0 }
         |     ^~~~~~~~~~~~
   drivers/media//pci/cx18/cx18-ioctl.c:476:5: note: (near initialization for 'formats[2].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:476:43: warning: excess elements in scalar initializer
     476 |     "UYVY 4:2:2", V4L2_PIX_FMT_UYVY, { 0, 0, 0, 0 }
         |                                           ^
   drivers/media//pci/cx18/cx18-ioctl.c:476:43: note: (near initialization for 'formats[2].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:476:46: warning: excess elements in scalar initializer
     476 |     "UYVY 4:2:2", V4L2_PIX_FMT_UYVY, { 0, 0, 0, 0 }
         |                                              ^
   drivers/media//pci/cx18/cx18-ioctl.c:476:46: note: (near initialization for 'formats[2].mbus_code')
   drivers/media//pci/cx18/cx18-ioctl.c:476:49: warning: excess elements in scalar initializer
     476 |     "UYVY 4:2:2", V4L2_PIX_FMT_UYVY, { 0, 0, 0, 0 }
         |                                                 ^
   drivers/media//pci/cx18/cx18-ioctl.c:476:49: note: (near initialization for 'formats[2].mbus_code')

vim +470 drivers/media//pci/cx18/cx18-ioctl.c

1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  464  
3b6fe58f0f1888 drivers/media/video/cx18/cx18-ioctl.c Andy Walls       2008-06-21  465  static int cx18_enum_fmt_vid_cap(struct file *file, void *fh,
3b6fe58f0f1888 drivers/media/video/cx18/cx18-ioctl.c Andy Walls       2008-06-21  466  					struct v4l2_fmtdesc *fmt)
3b6fe58f0f1888 drivers/media/video/cx18/cx18-ioctl.c Andy Walls       2008-06-21  467  {
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  468  	static const struct v4l2_fmtdesc formats[] = {
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  469  		{ 0, V4L2_BUF_TYPE_VIDEO_CAPTURE, 0,
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03 @470  		  "HM12 (YUV 4:1:1)", V4L2_PIX_FMT_HM12, { 0, 0, 0, 0 }
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  471  		},
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  472  		{ 1, V4L2_BUF_TYPE_VIDEO_CAPTURE, V4L2_FMT_FLAG_COMPRESSED,
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  473  		  "MPEG", V4L2_PIX_FMT_MPEG, { 0, 0, 0, 0 }
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  474  		},
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  475  		{ 2, V4L2_BUF_TYPE_VIDEO_CAPTURE, 0,
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  476  		  "UYVY 4:2:2", V4L2_PIX_FMT_UYVY, { 0, 0, 0, 0 }
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  477  		},
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  478  	};
1bf5842fe3b61d drivers/media/video/cx18/cx18-ioctl.c Simon Farnsworth 2011-05-03  479  
b7101de3fff596 drivers/media/video/cx18/cx18-ioctl.c Steven Toth      2011-04-06  480  	if (fmt->index > ARRAY_SIZE(formats) - 1)
1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  481  		return -EINVAL;
1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  482  	*fmt = formats[fmt->index];
1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  483  	return 0;
1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  484  }
1c1e45d17b663d drivers/media/video/cx18/cx18-ioctl.c Hans Verkuil     2008-04-28  485  

:::::: The code at line 470 was first introduced by commit
:::::: 1bf5842fe3b61d2dbbced96dbd27ad26fe93444a [media] cx18: Clean up mmap() support for raw YUV

:::::: TO: Simon Farnsworth <simon.farnsworth@xxxxxxxxxxxx>
:::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux