On Fri, Oct 25, 2019 at 02:59:42PM +0100, Chris Wilson wrote:
> Avoid angering clang and smatch by using a constant value in a '&&' test,
> by forcing that constant value into a boolean.
>
> E.g.,
> drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c:159:13: warning: use of
> logical '&&' with constant operand [-Wconstant-logical-operand]
> if (!delay && CONFIG_DRM_I915_PREEMPT_TIMEOUT) {
> ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Reported-by: kbuild test robot <l...@intel.com>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Nathan Chancellor <natechancel...@gmail.com>
> Cc: Nick Desaulniers <ndesaulni...@google.com>
> Cc: Jani Nikula <jani.nik...@intel.com>
LGTM, thank you for the fix!
Reviewed-by: Nathan Chancellor <natechancel...@gmail.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
