Re: [PATCH v2 12/19] media: davinci: allow build vpbe_display with COMPILE_TEST

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

 



Hi Mauro,

I love your patch! Perhaps something to improve:

[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.16 next-20180406]
[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/Mauro-Carvalho-Chehab/Make-all-media-drivers-build-with-COMPILE_TEST/20180406-163048
base:   git://linuxtv.org/media_tree.git master
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   In file included from drivers/media/platform/davinci/vpbe_display.c:35:0:
   drivers/media/platform/davinci/vpbe_display.c: In function 'register_device':
>> drivers/media/platform/davinci/vpbe_display.c:1358:5: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
        (int)vpbe_display_layer,
        ^
   include/media/v4l2-common.h:69:44: note: in definition of macro 'v4l2_printk'
     printk(level "%s: " fmt, (dev)->name , ## arg)
                                               ^~~
>> drivers/media/platform/davinci/vpbe_display.c:1356:2: note: in expansion of macro 'v4l2_info'
     v4l2_info(&disp_dev->vpbe_dev->v4l2_dev,
     ^~~~~~~~~
   drivers/media/platform/davinci/vpbe_display.c:1359:5: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
        (int)&vpbe_display_layer->video_dev);
        ^
   include/media/v4l2-common.h:69:44: note: in definition of macro 'v4l2_printk'
     printk(level "%s: " fmt, (dev)->name , ## arg)
                                               ^~~
>> drivers/media/platform/davinci/vpbe_display.c:1356:2: note: in expansion of macro 'v4l2_info'
     v4l2_info(&disp_dev->vpbe_dev->v4l2_dev,
     ^~~~~~~~~

vim +1358 drivers/media/platform/davinci/vpbe_display.c

a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1347  
4c62e976 drivers/media/platform/davinci/vpbe_display.c Greg Kroah-Hartman 2012-12-21  1348  static int register_device(struct vpbe_layer *vpbe_display_layer,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1349  			   struct vpbe_display *disp_dev,
4c62e976 drivers/media/platform/davinci/vpbe_display.c Greg Kroah-Hartman 2012-12-21  1350  			   struct platform_device *pdev)
4c62e976 drivers/media/platform/davinci/vpbe_display.c Greg Kroah-Hartman 2012-12-21  1351  {
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1352  	int err;
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1353  
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1354  	v4l2_info(&disp_dev->vpbe_dev->v4l2_dev,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1355  		  "Trying to register VPBE display device.\n");
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17 @1356  	v4l2_info(&disp_dev->vpbe_dev->v4l2_dev,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1357  		  "layer=%x,layer->video_dev=%x\n",
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17 @1358  		  (int)vpbe_display_layer,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1359  		  (int)&vpbe_display_layer->video_dev);
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1360  
266c9c2d drivers/media/platform/davinci/vpbe_display.c Prabhakar Lad      2014-10-12  1361  	vpbe_display_layer->video_dev.queue = &vpbe_display_layer->buffer_queue;
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1362  	err = video_register_device(&vpbe_display_layer->video_dev,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1363  				    VFL_TYPE_GRABBER,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1364  				    -1);
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1365  	if (err)
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1366  		return -ENODEV;
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1367  
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1368  	vpbe_display_layer->disp_dev = disp_dev;
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1369  	/* set the driver data in platform device */
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1370  	platform_set_drvdata(pdev, disp_dev);
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1371  	video_set_drvdata(&vpbe_display_layer->video_dev,
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1372  			  vpbe_display_layer);
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1373  
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1374  	return 0;
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1375  }
a2c25b44 drivers/media/video/davinci/vpbe_display.c    Manjunath Hadli    2011-06-17  1376  

:::::: The code at line 1358 was first introduced by commit
:::::: a2c25b444e99f2369b29e507568c8703186174d0 [media] davinci vpbe: V4L2 display driver for DM644X SoC

:::::: TO: Manjunath Hadli <manjunath.hadli@xxxxxx>
:::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>

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

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