Re: [PATCH 3/3] [media] em28xx: don't store usb_device at struct em28xx

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

 



Hi Mauro,

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on next-20161208]
[cannot apply to v4.9-rc8]
[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/em28xx-don-t-change-the-device-s-name/20161209-035446
base:   git://linuxtv.org/media_tree.git master
config: i386-randconfig-i1-201649 (attached as .config)
compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/media/usb/em28xx/em28xx-input.c: In function 'em28xx_register_snapshot_button':
>> drivers/media/usb/em28xx/em28xx-input.c:577:19: error: 'struct em28xx' has no member named 'udev'
     usb_make_path(dev->udev, dev->snapshot_button_path,
                      ^
   In file included from include/linux/byteorder/little_endian.h:4:0,
                    from arch/x86/include/uapi/asm/byteorder.h:4,
                    from include/asm-generic/bitops/le.h:5,
                    from arch/x86/include/asm/bitops.h:504,
                    from include/linux/bitops.h:36,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/timer.h:4,
                    from include/linux/workqueue.h:8,
                    from drivers/media/usb/em28xx/em28xx.h:32,
                    from drivers/media/usb/em28xx/em28xx-input.c:24:
   drivers/media/usb/em28xx/em28xx-input.c:589:40: error: 'struct em28xx' has no member named 'udev'
     input_dev->id.vendor = le16_to_cpu(dev->udev->descriptor.idVendor);
                                           ^
   include/uapi/linux/byteorder/little_endian.h:35:51: note: in definition of macro '__le16_to_cpu'
    #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                      ^
   drivers/media/usb/em28xx/em28xx-input.c:589:25: note: in expansion of macro 'le16_to_cpu'
     input_dev->id.vendor = le16_to_cpu(dev->udev->descriptor.idVendor);
                            ^
   drivers/media/usb/em28xx/em28xx-input.c:590:41: error: 'struct em28xx' has no member named 'udev'
     input_dev->id.product = le16_to_cpu(dev->udev->descriptor.idProduct);
                                            ^
   include/uapi/linux/byteorder/little_endian.h:35:51: note: in definition of macro '__le16_to_cpu'
    #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                      ^
   drivers/media/usb/em28xx/em28xx-input.c:590:26: note: in expansion of macro 'le16_to_cpu'
     input_dev->id.product = le16_to_cpu(dev->udev->descriptor.idProduct);
                             ^
   drivers/media/usb/em28xx/em28xx-input.c: In function 'em28xx_ir_init':
   drivers/media/usb/em28xx/em28xx-input.c:802:19: error: 'struct em28xx' has no member named 'udev'
     usb_make_path(dev->udev, ir->phys, sizeof(ir->phys));
                      ^
   In file included from include/linux/byteorder/little_endian.h:4:0,
                    from arch/x86/include/uapi/asm/byteorder.h:4,
                    from include/asm-generic/bitops/le.h:5,
                    from arch/x86/include/asm/bitops.h:504,
                    from include/linux/bitops.h:36,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/timer.h:4,
                    from include/linux/workqueue.h:8,
                    from drivers/media/usb/em28xx/em28xx.h:32,
                    from drivers/media/usb/em28xx/em28xx-input.c:24:
   drivers/media/usb/em28xx/em28xx-input.c:809:39: error: 'struct em28xx' has no member named 'udev'
     rc->input_id.vendor = le16_to_cpu(dev->udev->descriptor.idVendor);
                                          ^
   include/uapi/linux/byteorder/little_endian.h:35:51: note: in definition of macro '__le16_to_cpu'
    #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                      ^
   drivers/media/usb/em28xx/em28xx-input.c:809:24: note: in expansion of macro 'le16_to_cpu'
     rc->input_id.vendor = le16_to_cpu(dev->udev->descriptor.idVendor);
                           ^
   drivers/media/usb/em28xx/em28xx-input.c:810:40: error: 'struct em28xx' has no member named 'udev'
     rc->input_id.product = le16_to_cpu(dev->udev->descriptor.idProduct);
                                           ^
   include/uapi/linux/byteorder/little_endian.h:35:51: note: in definition of macro '__le16_to_cpu'
    #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                      ^
   drivers/media/usb/em28xx/em28xx-input.c:810:25: note: in expansion of macro 'le16_to_cpu'
     rc->input_id.product = le16_to_cpu(dev->udev->descriptor.idProduct);
                            ^

vim +577 drivers/media/usb/em28xx/em28xx-input.c

769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  561  	}
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  562  	/* Schedule next poll */
f52226099 drivers/media/usb/em28xx/em28xx-input.c   Frank Schaefer        2013-12-01  563  	schedule_delayed_work(&dev->buttons_query_work,
0ff950a73 drivers/media/usb/em28xx/em28xx-input.c   Frank Schaefer        2013-12-14  564  			      msecs_to_jiffies(dev->button_polling_interval));
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  565  }
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  566  
f52226099 drivers/media/usb/em28xx/em28xx-input.c   Frank Schaefer        2013-12-01  567  static int em28xx_register_snapshot_button(struct em28xx *dev)
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  568  {
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  569  	struct input_dev *input_dev;
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  570  	int err;
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  571  
42d0e2158 drivers/media/usb/em28xx/em28xx-input.c   Mauro Carvalho Chehab 2016-12-08  572  	dev_info(&dev->intf->dev, "Registering snapshot button...\n");
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  573  	input_dev = input_allocate_device();
da4a73394 drivers/media/usb/em28xx/em28xx-input.c   Joe Perches           2013-10-23  574  	if (!input_dev)
f52226099 drivers/media/usb/em28xx/em28xx-input.c   Frank Schaefer        2013-12-01  575  		return -ENOMEM;
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  576  
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26 @577  	usb_make_path(dev->udev, dev->snapshot_button_path,
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  578  		      sizeof(dev->snapshot_button_path));
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  579  	strlcat(dev->snapshot_button_path, "/sbutton",
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  580  		sizeof(dev->snapshot_button_path));
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  581  
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  582  	input_dev->name = "em28xx snapshot button";
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  583  	input_dev->phys = dev->snapshot_button_path;
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  584  	input_dev->evbit[0] = BIT_MASK(EV_KEY) | BIT_MASK(EV_REP);
769af2146 drivers/media/video/em28xx/em28xx-input.c Ezequiel García       2012-03-26  585  	set_bit(EM28XX_SNAPSHOT_KEY, input_dev->keybit);

:::::: The code at line 577 was first introduced by commit
:::::: 769af2146a93c27c8834dbca54c02cd67468036d [media] em28xx: Change scope of em28xx-input local functions to static

:::::: TO: Ezequiel García <elezegarcia@xxxxxxxxx>
:::::: 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