doh!! sent the wrong version... going to sent the right one now...

On Fri, 2016-02-12 at 09:32 +0800, kbuild test robot wrote:
> Hi Rodrigo,
> 
> [auto build test WARNING on drm-intel/for-linux-next]
> [also build test WARNING on v4.5-rc3 next-20160211]
> [if your patch is applied to the wrong git tree, please drop us a 
> note to help improving the system]
> 
> url:    https://github.com/0day-ci/linux/commits/Rodrigo-Vivi/drm-i91
> 5-Avoid-vblank-counter-for-gen9/20160212-090608
> base:   git://anongit.freedesktop.org/drm-intel for-linux-next
> config: x86_64-randconfig-x000-201606 (attached as .config)
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from include/uapi/linux/stddef.h:1:0,
>                     from include/linux/stddef.h:4,
>                     from include/uapi/linux/posix_types.h:4,
>                     from include/uapi/linux/types.h:13,
>                     from include/linux/types.h:5,
>                     from include/linux/sysrq.h:18,
>                     from drivers/gpu/drm/i915/i915_irq.c:31:
>    drivers/gpu/drm/i915/i915_irq.c: In function 'intel_irq_init':
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: comparison of 
> constant '9' with boolean expression is always false [-Wbool-compare]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:147:28: note: in definition of macro 
> '__trace_if'
>      if (__builtin_constant_p((cond)) ? !!(cond) :   \
>                                ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: logical not is 
> only applied to the left hand side of comparison [-Wlogical-not
> -parentheses]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:147:28: note: in definition of macro 
> '__trace_if'
>      if (__builtin_constant_p((cond)) ? !!(cond) :   \
>                                ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: comparison of 
> constant '9' with boolean expression is always false [-Wbool-compare]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:147:40: note: in definition of macro 
> '__trace_if'
>      if (__builtin_constant_p((cond)) ? !!(cond) :   \
>                                            ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: logical not is 
> only applied to the left hand side of comparison [-Wlogical-not
> -parentheses]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:147:40: note: in definition of macro 
> '__trace_if'
>      if (__builtin_constant_p((cond)) ? !!(cond) :   \
>                                            ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: comparison of 
> constant '9' with boolean expression is always false [-Wbool-compare]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:158:16: note: in definition of macro 
> '__trace_if'
>       ______r = !!(cond);     \
>                    ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
>    drivers/gpu/drm/i915/i915_irq.c:4578:55: warning: logical not is 
> only applied to the left hand side of comparison [-Wlogical-not
> -parentheses]
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>                                                           ^
>    include/linux/compiler.h:158:16: note: in definition of macro 
> '__trace_if'
>       ______r = !!(cond);     \
>                    ^
> > > drivers/gpu/drm/i915/i915_irq.c:4578:2: note: in expansion of 
> > > macro 'if'
>      if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)->gen >= 9)
>      ^
> 
> vim +/if +4578 drivers/gpu/drm/i915/i915_irq.c
> 
>   4562                } else if (IS_GEN2(dev_priv)) {
>   4563                        dev->max_vblank_count = 0;
>   4564                        dev->driver->get_vblank_counter = 
> i8xx_get_vblank_counter;
>   4565                } else if (IS_G4X(dev_priv) || 
> INTEL_INFO(dev_priv)->gen >= 5) {
>   4566                        dev->max_vblank_count = 0xffffffff; /* 
> full 32 bit counter */
>   4567                        dev->driver->get_vblank_counter = 
> g4x_get_vblank_counter;
>   4568                } else {
>   4569                        dev->driver->get_vblank_counter = 
> i915_get_vblank_counter;
>   4570                        dev->max_vblank_count = 0xffffff; /* 
> only 24 bits of frame count */
>   4571                }
>   4572        
>   4573                /*
>   4574                 * Opt out of the vblank disable timer on 
> everything except gen2.
>   4575                 * Gen2 doesn't have a hardware frame counter 
> and so depends on
>   4576                 * vblank interrupts to produce sane vblank 
> seuquence numbers.
>   4577                 */
> > 4578                if (!IS_GEN2(dev_priv) && !INTEL_INFO(dev_priv)
> > ->gen >= 9)
>   4579                        dev->vblank_disable_immediate = true;
>   4580        
>   4581                dev->driver->get_vblank_timestamp = 
> i915_get_vblank_timestamp;
>   4582                dev->driver->get_scanout_position = 
> i915_get_crtc_scanoutpos;
>   4583        
>   4584                if (IS_CHERRYVIEW(dev_priv)) {
>   4585                        dev->driver->irq_handler = 
> cherryview_irq_handler;
>   4586                        dev->driver->irq_preinstall = 
> cherryview_irq_preinstall;
> 
> ---
> 0-DAY kernel test infrastructure                Open Source 
> Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Cor
> poration
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to