[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Minor revid/stepping and workaround cleanup (rev5)
== Series Details == Series: Minor revid/stepping and workaround cleanup (rev5) URL : https://patchwork.freedesktop.org/series/92299/ State : warning == Summary == $ dim checkpatch origin/drm-tip 486210f705f0 drm/i915/step: s/_revid_tbl/_revids 5a7f5fb477f9 drm/i915: Make pre-production detection use direct revid comparison 595f9e313f2d drm/i915/skl: Use revid->stepping tables -:58: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #58: FILE: drivers/gpu/drm/i915/i915_drv.h:1518: +#define IS_SKL_GT_STEP(p, since, until) (IS_SKYLAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 80 lines checked 57b7fcc6d24f drm/i915/kbl: Drop pre-production revision from stepping table 55db7b1abf0b drm/i915/bxt: Use revid->stepping tables 16a24e1a17f6 drm/i915/glk: Use revid->stepping tables 7609fedec7c6 drm/i915/icl: Use revid->stepping tables -:120: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #120: FILE: drivers/gpu/drm/i915/i915_drv.h:1532: +#define IS_ICL_GT_STEP(p, since, until) \ + (IS_ICELAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 87 lines checked 2eb0bd7f91f4 drm/i915/jsl_ehl: Use revid->stepping tables -:56: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #56: FILE: drivers/gpu/drm/i915/i915_drv.h:1535: +#define IS_JSL_EHL_GT_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_GT_STEP(p, since, until)) -:58: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #58: FILE: drivers/gpu/drm/i915/i915_drv.h:1537: +#define IS_JSL_EHL_DISPLAY_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 51 lines checked d459b42fb71d drm/i915/rkl: Use revid->stepping tables -:49: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #49: FILE: drivers/gpu/drm/i915/i915_drv.h:1552: +#define IS_RKL_DISPLAY_STEP(p, since, until) \ + (IS_ROCKETLAKE(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 51 lines checked c7a0a64af9c3 drm/i915/dg1: Use revid->stepping tables -:129: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #129: FILE: drivers/gpu/drm/i915/i915_drv.h:1546: +#define IS_DG1_GT_STEP(p, since, until) \ + (IS_DG1(p) && IS_GT_STEP(p, since, until)) -:131: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #131: FILE: drivers/gpu/drm/i915/i915_drv.h:1548: +#define IS_DG1_DISPLAY_STEP(p, since, until) \ + (IS_DG1(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 118 lines checked 4616f4431b2a drm/i915/cnl: Drop all workarounds 8aac230553bb drm/i915/icl: Drop workarounds that only apply to pre-production steppings ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Minor revid/stepping and workaround cleanup (rev4)
== Series Details == Series: Minor revid/stepping and workaround cleanup (rev4) URL : https://patchwork.freedesktop.org/series/92299/ State : warning == Summary == $ dim checkpatch origin/drm-tip 62d758aadfe3 drm/i915/step: s/_revid_tbl/_revids 76bd03989163 drm/i915: Make pre-production detection use direct revid comparison 5c84247d6ac9 drm/i915/skl: Use revid->stepping tables -:57: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #57: FILE: drivers/gpu/drm/i915/i915_drv.h:1518: +#define IS_SKL_GT_STEP(p, since, until) (IS_SKYLAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 80 lines checked 1c898561ba39 drm/i915/kbl: Drop pre-production revision from stepping table 27320b6e947b drm/i915/bxt: Use revid->stepping tables bfc4af69ce3b drm/i915/glk: Use revid->stepping tables ea04a99db33e drm/i915/icl: Use revid->stepping tables -:116: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #116: FILE: drivers/gpu/drm/i915/i915_drv.h:1532: +#define IS_ICL_GT_STEP(p, since, until) \ + (IS_ICELAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 87 lines checked 95aec3b31d70 drm/i915/jsl_ehl: Use revid->stepping tables -:56: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #56: FILE: drivers/gpu/drm/i915/i915_drv.h:1535: +#define IS_JSL_EHL_GT_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_GT_STEP(p, since, until)) -:58: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #58: FILE: drivers/gpu/drm/i915/i915_drv.h:1537: +#define IS_JSL_EHL_DISPLAY_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 51 lines checked d515112313b0 drm/i915/rkl: Use revid->stepping tables -:49: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #49: FILE: drivers/gpu/drm/i915/i915_drv.h:1552: +#define IS_RKL_DISPLAY_STEP(p, since, until) \ + (IS_ROCKETLAKE(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 51 lines checked 28dea784c5d3 drm/i915/dg1: Use revid->stepping tables -:129: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #129: FILE: drivers/gpu/drm/i915/i915_drv.h:1546: +#define IS_DG1_GT_STEP(p, since, until) \ + (IS_DG1(p) && IS_GT_STEP(p, since, until)) -:131: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #131: FILE: drivers/gpu/drm/i915/i915_drv.h:1548: +#define IS_DG1_DISPLAY_STEP(p, since, until) \ + (IS_DG1(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 118 lines checked a66bc53c20b3 drm/i915/cnl: Drop all workarounds d62d0ec1b3aa drm/i915/icl: Drop workarounds that only apply to pre-production steppings ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Minor revid/stepping and workaround cleanup (rev2)
== Series Details == Series: Minor revid/stepping and workaround cleanup (rev2) URL : https://patchwork.freedesktop.org/series/92299/ State : warning == Summary == $ dim checkpatch origin/drm-tip 602f2fe49dc0 drm/i915/step: s/_revid_tbl/_revids 023cab2a2100 drm/i915: Make pre-production detection use direct revid comparison f260bb283918 drm/i915/skl: Use revid->stepping tables -:56: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #56: FILE: drivers/gpu/drm/i915/i915_drv.h:1518: +#define IS_SKL_GT_STEP(p, since, until) (IS_SKYLAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 80 lines checked 02447a87c0cc drm/i915/kbl: Drop pre-production revision from stepping table 2ee48fd0d6f6 drm/i915/bxt: Use revid->stepping tables c506016c4648 drm/i915/glk: Use revid->stepping tables db965751e531 drm/i915/icl: Use revid->stepping tables -:116: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #116: FILE: drivers/gpu/drm/i915/i915_drv.h:1532: +#define IS_ICL_GT_STEP(p, since, until) \ + (IS_ICELAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 87 lines checked fc2b6164fa50 drm/i915/jsl_ehl: Use revid->stepping tables -:51: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #51: FILE: drivers/gpu/drm/i915/i915_drv.h:1535: +#define IS_JSL_EHL_GT_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_GT_STEP(p, since, until)) -:53: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #53: FILE: drivers/gpu/drm/i915/i915_drv.h:1537: +#define IS_JSL_EHL_DISPLAY_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 51 lines checked c53e1e6ba585 drm/i915/rkl: Use revid->stepping tables -:48: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #48: FILE: drivers/gpu/drm/i915/i915_drv.h:1552: +#define IS_RKL_DISPLAY_STEP(p, since, until) \ + (IS_ROCKETLAKE(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 51 lines checked 36d6af0814ee drm/i915/dg1: Use revid->stepping tables -:124: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #124: FILE: drivers/gpu/drm/i915/i915_drv.h:1546: +#define IS_DG1_GT_STEP(p, since, until) \ + (IS_DG1(p) && IS_GT_STEP(p, since, until)) -:126: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #126: FILE: drivers/gpu/drm/i915/i915_drv.h:1548: +#define IS_DG1_DISPLAY_STEP(p, since, until) \ + (IS_DG1(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 118 lines checked febbb9864d60 drm/i915/cnl: Drop all workarounds 4fbd5e59cb10 drm/i915/icl: Drop workarounds that only apply to pre-production steppings ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Minor revid/stepping and workaround cleanup
== Series Details == Series: Minor revid/stepping and workaround cleanup URL : https://patchwork.freedesktop.org/series/92299/ State : warning == Summary == $ dim checkpatch origin/drm-tip 76c5a033ed04 drm/i915: Make pre-production detection use direct revid comparison c88cee5325fe drm/i915/skl: Use revid->stepping tables -:54: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #54: FILE: drivers/gpu/drm/i915/i915_drv.h:1450: +#define IS_SKL_GT_STEP(p, since, until) (IS_SKYLAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 85 lines checked 4f8fb265f095 drm/i915/icl: Use revid->stepping tables -:93: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #93: FILE: drivers/gpu/drm/i915/i915_drv.h:1457: +#define IS_ICL_GT_STEP(p, since, until) \ + (IS_ICELAKE(p) && IS_GT_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 94 lines checked 46752176291f drm/i915/jsl_ehl: Use revid->stepping tables -:51: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #51: FILE: drivers/gpu/drm/i915/i915_drv.h:1460: +#define IS_JSL_EHL_GT_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_GT_STEP(p, since, until)) -:53: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #53: FILE: drivers/gpu/drm/i915/i915_drv.h:1462: +#define IS_JSL_EHL_DISPLAY_STEP(p, since, until) \ + (IS_JSL_EHL(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 51 lines checked 54dfa2be0428 drm/i915/rkl: Use revid->stepping tables -:48: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #48: FILE: drivers/gpu/drm/i915/i915_drv.h:1477: +#define IS_RKL_DISPLAY_STEP(p, since, until) \ + (IS_ROCKETLAKE(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 1 checks, 51 lines checked 9e913fff5691 drm/i915/dg1: Use revid->stepping tables -:124: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #124: FILE: drivers/gpu/drm/i915/i915_drv.h:1471: +#define IS_DG1_GT_STEP(p, since, until) \ + (IS_DG1(p) && IS_GT_STEP(p, since, until)) -:126: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #126: FILE: drivers/gpu/drm/i915/i915_drv.h:1473: +#define IS_DG1_DISPLAY_STEP(p, since, until) \ + (IS_DG1(p) && IS_DISPLAY_STEP(p, since, until)) total: 0 errors, 0 warnings, 2 checks, 118 lines checked c78151696795 drm/i915/cnl: Drop all workarounds ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx