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-i915-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 Corporation

Attachment: .config.gz
Description: Binary data

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to