== Series Details == Series: drm/i915: stop storing the media fuse URL : https://patchwork.freedesktop.org/series/58387/ State : warning == Summary == $ dim checkpatch origin/drm-tip de5dfcc93c76 drm/i915: stop storing the media fuse -:23: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'first' - possible side-effects? #23: FILE: drivers/gpu/drm/i915/i915_drv.h:2443: +#define ENGINE_INSTANCES_MASK(dev_priv, first, count) \ + ((INTEL_INFO(dev_priv)->engine_mask & \ + GENMASK(first + count - 1, first)) >> first) -:23: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'first' may be better as '(first)' to avoid precedence issues #23: FILE: drivers/gpu/drm/i915/i915_drv.h:2443: +#define ENGINE_INSTANCES_MASK(dev_priv, first, count) \ + ((INTEL_INFO(dev_priv)->engine_mask & \ + GENMASK(first + count - 1, first)) >> first) -:23: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'count' may be better as '(count)' to avoid precedence issues #23: FILE: drivers/gpu/drm/i915/i915_drv.h:2443: +#define ENGINE_INSTANCES_MASK(dev_priv, first, count) \ + ((INTEL_INFO(dev_priv)->engine_mask & \ + GENMASK(first + count - 1, first)) >> first) -:26: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #26: FILE: drivers/gpu/drm/i915/i915_drv.h:2446: +#define VDBOX_MASK(dev_priv) ENGINE_INSTANCES_MASK(dev_priv, VCS0, I915_MAX_VCS); -:27: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #27: FILE: drivers/gpu/drm/i915/i915_drv.h:2447: +#define VEBOX_MASK(dev_priv) ENGINE_INSTANCES_MASK(dev_priv, VECS0, I915_MAX_VECS); total: 0 errors, 2 warnings, 3 checks, 64 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
