Re: [PATCH] v4l: add support for CSI-1 bus

From: kbuild test robot
Date: Wed Dec 28 2016 - 16:42:24 EST


Hi Sakari,

[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.10-rc1 next-20161224]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Pavel-Machek/v4l-add-support-for-CSI-1-bus/20161229-043410
base: git://linuxtv.org/media_tree.git master
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

drivers/media/platform/pxa_camera.c: In function 'pxa_mbus_config_compatible':
>> drivers/media/platform/pxa_camera.c:595:2: warning: enumeration value 'V4L2_MBUS_CSI1' not handled in switch [-Wswitch]
switch (cfg->type) {
^~~~~~
>> drivers/media/platform/pxa_camera.c:595:2: warning: enumeration value 'V4L2_MBUS_CCP2' not handled in switch [-Wswitch]

vim +/V4L2_MBUS_CSI1 +595 drivers/media/platform/pxa_camera.c

34b27b13 Hans Verkuil 2016-09-11 579
34b27b13 Hans Verkuil 2016-09-11 580 static const struct pxa_mbus_pixelfmt *pxa_mbus_get_fmtdesc(
34b27b13 Hans Verkuil 2016-09-11 581 u32 code)
34b27b13 Hans Verkuil 2016-09-11 582 {
34b27b13 Hans Verkuil 2016-09-11 583 return pxa_mbus_find_fmtdesc(code, mbus_fmt, ARRAY_SIZE(mbus_fmt));
34b27b13 Hans Verkuil 2016-09-11 584 }
34b27b13 Hans Verkuil 2016-09-11 585
34b27b13 Hans Verkuil 2016-09-11 586 static unsigned int pxa_mbus_config_compatible(const struct v4l2_mbus_config *cfg,
34b27b13 Hans Verkuil 2016-09-11 587 unsigned int flags)
34b27b13 Hans Verkuil 2016-09-11 588 {
34b27b13 Hans Verkuil 2016-09-11 589 unsigned long common_flags;
34b27b13 Hans Verkuil 2016-09-11 590 bool hsync = true, vsync = true, pclk, data, mode;
34b27b13 Hans Verkuil 2016-09-11 591 bool mipi_lanes, mipi_clock;
34b27b13 Hans Verkuil 2016-09-11 592
34b27b13 Hans Verkuil 2016-09-11 593 common_flags = cfg->flags & flags;
34b27b13 Hans Verkuil 2016-09-11 594
34b27b13 Hans Verkuil 2016-09-11 @595 switch (cfg->type) {
34b27b13 Hans Verkuil 2016-09-11 596 case V4L2_MBUS_PARALLEL:
34b27b13 Hans Verkuil 2016-09-11 597 hsync = common_flags & (V4L2_MBUS_HSYNC_ACTIVE_HIGH |
34b27b13 Hans Verkuil 2016-09-11 598 V4L2_MBUS_HSYNC_ACTIVE_LOW);
34b27b13 Hans Verkuil 2016-09-11 599 vsync = common_flags & (V4L2_MBUS_VSYNC_ACTIVE_HIGH |
34b27b13 Hans Verkuil 2016-09-11 600 V4L2_MBUS_VSYNC_ACTIVE_LOW);
34b27b13 Hans Verkuil 2016-09-11 601 /* fall through */
34b27b13 Hans Verkuil 2016-09-11 602 case V4L2_MBUS_BT656:
34b27b13 Hans Verkuil 2016-09-11 603 pclk = common_flags & (V4L2_MBUS_PCLK_SAMPLE_RISING |

:::::: The code at line 595 was first introduced by commit
:::::: 34b27b13a2cd43e0d27e3ce13b96119efbbbedbc [media] pxa_camera: merge soc_mediabus.c into pxa_camera.c

:::::: TO: Hans Verkuil <hans.verkuil@xxxxxxxxx>
:::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip