== Series Details == Series: series starting with [1/3] drm/i915: Add struct to hold IP version URL : https://patchwork.freedesktop.org/series/96038/ State : warning == Summary == $ dim checkpatch origin/drm-tip 41a0609f8b33 drm/i915: Add struct to hold IP version -:44: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #44: FILE: drivers/gpu/drm/i915/i915_drv.h:1331: +#define GRAPHICS_VER_FULL(i915) IP_VER(INTEL_INFO(i915)->graphics.ver, \ + INTEL_INFO(i915)->graphics.rel) -:53: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #53: FILE: drivers/gpu/drm/i915/i915_drv.h:1337: +#define MEDIA_VER_FULL(i915) IP_VER(INTEL_INFO(i915)->media.arch, \ + INTEL_INFO(i915)->media.rel) total: 0 errors, 0 warnings, 2 checks, 139 lines checked bd6fdf51759f drm/i915: Track media IP stepping separated from GT -:34: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #34: FILE: drivers/gpu/drm/i915/i915_drv.h:1362: +#define IS_MEDIA_STEP(__i915, since, until) \ + (drm_WARN_ON(&(__i915)->drm, INTEL_MEDIA_STEP(__i915) == STEP_NONE), \ + INTEL_MEDIA_STEP(__i915) >= (since) && INTEL_MEDIA_STEP(__i915) < (until)) total: 0 errors, 0 warnings, 1 checks, 133 lines checked 92310e925ccb drm/i915: Rename GT_STEP to GRAPHICS_STEP -:206: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #206: FILE: drivers/gpu/drm/i915/i915_drv.h:1358: +#define IS_GRAPHICS_STEP(__i915, since, until) \ + (drm_WARN_ON(&(__i915)->drm, INTEL_GRAPHICS_STEP(__i915) == STEP_NONE), \ + INTEL_GRAPHICS_STEP(__i915) >= (since) && INTEL_GRAPHICS_STEP(__i915) < (until)) -:217: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #217: FILE: drivers/gpu/drm/i915/i915_drv.h:1538: +#define IS_SKL_GRAPHICS_STEP(p, since, until) (IS_SKYLAKE(p) && IS_GRAPHICS_STEP(p, since, until)) -:221: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects? #221: FILE: drivers/gpu/drm/i915/i915_drv.h:1540: +#define IS_KBL_GRAPHICS_STEP(dev_priv, since, until) \ + (IS_KABYLAKE(dev_priv) && IS_GRAPHICS_STEP(dev_priv, since, until)) -:228: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #228: FILE: drivers/gpu/drm/i915/i915_drv.h:1545: +#define IS_JSL_EHL_GRAPHICS_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_GRAPHICS_STEP(p, since, until)) -:238: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #238: FILE: drivers/gpu/drm/i915/i915_drv.h:1554: +#define IS_TGL_UY_GRAPHICS_STEP(__i915, since, until) \ ((IS_TGL_U(__i915) || IS_TGL_Y(__i915)) && \ + IS_GRAPHICS_STEP(__i915, since, until)) -:244: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #244: FILE: drivers/gpu/drm/i915/i915_drv.h:1558: +#define IS_TGL_GRAPHICS_STEP(__i915, since, until) \ (IS_TIGERLAKE(__i915) && !(IS_TGL_U(__i915) || IS_TGL_Y(__i915)) && \ + IS_GRAPHICS_STEP(__i915, since, until)) -:254: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #254: FILE: drivers/gpu/drm/i915/i915_drv.h:1565: +#define IS_DG1_GRAPHICS_STEP(p, since, until) \ + (IS_DG1(p) && IS_GRAPHICS_STEP(p, since, until)) -:264: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #264: FILE: drivers/gpu/drm/i915/i915_drv.h:1574: +#define IS_ADLS_GRAPHICS_STEP(__i915, since, until) \ (IS_ALDERLAKE_S(__i915) && \ + IS_GRAPHICS_STEP(__i915, since, until)) -:274: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #274: FILE: drivers/gpu/drm/i915/i915_drv.h:1582: +#define IS_ADLP_GRAPHICS_STEP(__i915, since, until) \ (IS_ALDERLAKE_P(__i915) && \ + IS_GRAPHICS_STEP(__i915, since, until)) -:281: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #281: FILE: drivers/gpu/drm/i915/i915_drv.h:1586: +#define IS_XEHPSDV_GRAPHICS_STEP(__i915, since, until) \ + (IS_XEHPSDV(__i915) && IS_GRAPHICS_STEP(__i915, since, until)) -:291: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i915' - possible side-effects? #291: FILE: drivers/gpu/drm/i915/i915_drv.h:1603: +#define IS_DG2_GRAPHICS_STEP(__i915, variant, since, until) \ (IS_SUBPLATFORM(__i915, INTEL_DG2, INTEL_SUBPLATFORM_##variant) && \ + IS_GRAPHICS_STEP(__i915, since, until)) total: 0 errors, 0 warnings, 11 checks, 309 lines checked
