subject:"\[linuxtv\-media\:devel\-3.17\-rc6 491\/499\] drivers\/media\/usb\/em28xx\/em28xx.h\:787\:2\: warning\: 'vid' may be used uninitialized in this function"

Re: [linuxtv-media:devel-3.17-rc6 491/499] drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used uninitialized in this function

2014-09-23 Thread Frank Schäfer


Am 23.09.2014 um 04:27 schrieb kbuild test robot:
> tree:   git://linuxtv.org/media_tree.git devel-3.17-rc6
> head:   7f8de65b0dc84c19e79d7a642a5655524c57035c
> commit: f5ac7a471e156f997833f94bad2228e57122c227 [491/499] [media] em28xx: 
> remove some unnecessary fields from struct em28xx_audio_mode
> config: i386-randconfig-r0-0923 (attached as .config)
> reproduce:
>   git checkout f5ac7a471e156f997833f94bad2228e57122c227
>   # save the attached .config to linux build tree
>   make ARCH=i386 
>
> Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
> http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings
>
> All warnings:
>
>In file included from drivers/media/usb/em28xx/em28xx-core.c:35:0:
>drivers/media/usb/em28xx/em28xx-core.c: In function 'em28xx_audio_setup':
>>> drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used 
>>> uninitialized in this function [-Wmaybe-uninitialized]
>  printk(KERN_INFO "%s: "fmt,\
>  ^
>drivers/media/usb/em28xx/em28xx-core.c:507:6: note: 'vid' was declared here
>  u32 vid;
>  ^
This is a false warning.
Applying https://patchwork.linuxtv.org/patch/25918/ makes it disappear.

Regards,
Frank


> vim +/vid +787 drivers/media/usb/em28xx/em28xx.h
>
> 47677e51 drivers/media/usb/em28xx/em28xx.h   Mauro Carvalho Chehab 2014-03-05 
>  771  void em28xx_free_device(struct kref *ref);
> c8793b03 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2008-01-13 
>  772  
> 855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27 
>  773  /* Provided by em28xx-camera.c */
> 855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27 
>  774  int em28xx_detect_sensor(struct em28xx *dev);
> 855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27 
>  775  int em28xx_init_camera(struct em28xx *dev);
> 855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27 
>  776  
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  777  /* printk macros */
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  778  
> 3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08 
>  779  #define em28xx_err(fmt, arg...) do {\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  780printk(KERN_ERR fmt , ##arg); } while (0)
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  781  
> 3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08 
>  782  #define em28xx_errdev(fmt, arg...) do {\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  783printk(KERN_ERR "%s: "fmt,\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  784dev->name , ##arg); } while (0)
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  785  
> 3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08 
>  786  #define em28xx_info(fmt, arg...) do {\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
> @787printk(KERN_INFO "%s: "fmt,\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  788dev->name , ##arg); } while (0)
> 3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08 
>  789  #define em28xx_warn(fmt, arg...) do {\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  790printk(KERN_WARNING "%s: "fmt,\
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  791dev->name , ##arg); } while (0)
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  792  
> a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
>  793  #endif
>
> :: The code at line 787 was first introduced by commit
> :: a6c2ba283565dbc9f055dcb2ecba1971460bb535 [PATCH] v4l: 716: support for 
> em28xx board family
>
> :: TO: a...@osdl.org 
> :: CC: Linus Torvalds 
>
> ---
> 0-DAY kernel build testing backend  Open Source Technology Center
> http://lists.01.org/mailman/listinfo/kbuild Intel Corporation

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[linuxtv-media:devel-3.17-rc6 491/499] drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used uninitialized in this function

2014-09-22 Thread kbuild test robot

tree:   git://linuxtv.org/media_tree.git devel-3.17-rc6
head:   7f8de65b0dc84c19e79d7a642a5655524c57035c
commit: f5ac7a471e156f997833f94bad2228e57122c227 [491/499] [media] em28xx: 
remove some unnecessary fields from struct em28xx_audio_mode
config: i386-randconfig-r0-0923 (attached as .config)
reproduce:
  git checkout f5ac7a471e156f997833f94bad2228e57122c227
  # save the attached .config to linux build tree
  make ARCH=i386 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings:

   In file included from drivers/media/usb/em28xx/em28xx-core.c:35:0:
   drivers/media/usb/em28xx/em28xx-core.c: In function 'em28xx_audio_setup':
>> drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used 
>> uninitialized in this function [-Wmaybe-uninitialized]
 printk(KERN_INFO "%s: "fmt,\
 ^
   drivers/media/usb/em28xx/em28xx-core.c:507:6: note: 'vid' was declared here
 u32 vid;
 ^

vim +/vid +787 drivers/media/usb/em28xx/em28xx.h

47677e51 drivers/media/usb/em28xx/em28xx.h   Mauro Carvalho Chehab 2014-03-05  
771  void em28xx_free_device(struct kref *ref);
c8793b03 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2008-01-13  
772  
855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27  
773  /* Provided by em28xx-camera.c */
855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27  
774  int em28xx_detect_sensor(struct em28xx *dev);
855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27  
775  int em28xx_init_camera(struct em28xx *dev);
855ff38e drivers/media/usb/em28xx/em28xx.h   Frank Schaefer2013-03-27  
776  
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
777  /* printk macros */
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
778  
3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08  
779  #define em28xx_err(fmt, arg...) do {\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
780  printk(KERN_ERR fmt , ##arg); } while (0)
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
781  
3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08  
782  #define em28xx_errdev(fmt, arg...) do {\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
783  printk(KERN_ERR "%s: "fmt,\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
784  dev->name , ##arg); } while (0)
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
785  
3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08  
786  #define em28xx_info(fmt, arg...) do {\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08 
@787  printk(KERN_INFO "%s: "fmt,\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
788  dev->name , ##arg); } while (0)
3acf2809 drivers/media/video/em28xx/em28xx.h Mauro Carvalho Chehab 2005-11-08  
789  #define em28xx_warn(fmt, arg...) do {\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
790  printk(KERN_WARNING "%s: "fmt,\
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
791  dev->name , ##arg); } while (0)
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
792  
a6c2ba28 drivers/media/video/em28xx/em28xx.h Andrew Morton 2005-11-08  
793  #endif

:: The code at line 787 was first introduced by commit
:: a6c2ba283565dbc9f055dcb2ecba1971460bb535 [PATCH] v4l: 716: support for 
em28xx board family

:: TO: a...@osdl.org 
:: CC: Linus Torvalds 

---
0-DAY kernel build testing backend  Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
#
# Automatically generated file; DO NOT EDIT.
# Linux/i386 3.17.0-rc5 Kernel Configuration
#
# CONFIG_64BIT is not set
CONFIG_X86_32=y
CONFIG_X86=y
CONFIG_INSTRUCTION_DECODER=y
CONFIG_OUTPUT_FORMAT="elf32-i386"
CONFIG_ARCH_DEFCONFIG="arch/x86/configs/i386_defconfig"
CONFIG_LOCKDEP_SUPPORT=y
CONFIG_STACKTRACE_SUPPORT=y
CONFIG_HAVE_LATENCYTOP_SUPPORT=y
CONFIG_MMU=y
CONFIG_NEED_SG_DMA_LENGTH=y
CONFIG_GENERIC_ISA_DMA=y
CONFIG_GENERIC_BUG=y
CONFIG_GENERIC_HWEIGHT=y
CONFIG_ARCH_MAY_HAVE_PC_FDC=y
CONFIG_RWSEM_XCHGADD_ALGORITHM=y
CONFIG_GENERIC_CALIBRATE_DELAY=y
CONFIG_ARCH_HAS_CPU_RELAX=y
CONFIG_ARCH_HAS_CACHE_LINE_SIZE=y
CONFIG_HAVE_SETUP_PER_CPU_AREA=y
CONFIG_NEED_PER_CPU_EMBED_FIRST_CHUNK=y
CONFIG_NEED_PER_CPU_PAGE_FIRST_CHUNK=y
CONFIG_ARCH_HIBERNATION_POSSIBLE=y
CONFIG_ARCH_SUSPEND_POSSIBLE=y
CONFIG_ARCH_WANT_HUGE_PMD_SHARE=y
CONFIG_ARCH_WANT_GENERAL_HUGETLB=y
# CONFIG_ZONE_DMA32 is not set
# CONFIG_AUDIT_ARCH is not set
CONFIG_ARCH_SUPPORTS_OPTI

Re: [linuxtv-media:devel-3.17-rc6 491/499] drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used uninitialized in this function

[linuxtv-media:devel-3.17-rc6 491/499] drivers/media/usb/em28xx/em28xx.h:787:2: warning: 'vid' may be used uninitialized in this function

2 matches

Site Navigation

Mail list logo

Footer information