[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Some checkpatch fixes for i915_reg.h (rev2)

2018-06-18 Thread Patchwork
== Series Details ==

Series: Some checkpatch fixes for i915_reg.h (rev2)
URL   : https://patchwork.freedesktop.org/series/44662/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
e7a983f07b24 drm/i915/i915_reg.h: fix the checkpatch SPACING issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch SPACING issues

-:53: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#53: FILE: drivers/gpu/drm/i915/i915_reg.h:144:
+#define _PIPE(pipe, a, b) ((a) + (pipe) * ((b) - (a)))

-:58: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#58: FILE: drivers/gpu/drm/i915/i915_reg.h:148:
+#define _TRANS(tran, a, b) ((a) + (tran) * ((b) - (a)))

-:61: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#61: FILE: drivers/gpu/drm/i915/i915_reg.h:150:
+#define _PORT(port, a, b) ((a) + (port) * ((b) - (a)))

-:66: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#66: FILE: drivers/gpu/drm/i915/i915_reg.h:154:
+#define _PLL(pll, a, b) ((a) + (pll) * ((b) - (a)))

-:1068: WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1068: FILE: drivers/gpu/drm/i915/i915_reg.h:2416:
+#define   DERRMR_PIPEB_SCANLINE ^I(1 << 8)$

-:1636: WARNING:LONG_LINE: line over 100 characters
#1636: FILE: drivers/gpu/drm/i915/i915_reg.h:3136:
+#define   PHY_STATUS_SPLINE_LDO(phy, ch, spline)(1 << (8 - (6 * (phy) 
+ 3 * (ch) + (spline

-:1848: WARNING:LONG_LINE_COMMENT: line over 100 characters
#1848: FILE: drivers/gpu/drm/i915/i915_reg.h:3651:
+#define   IMPROMOEN(1 << 10) /* promo is immediate or delayed 
until next idle interval (only for timeout method above) */

-:3189: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#3189: FILE: drivers/gpu/drm/i915/i915_reg.h:6832:
+#define _ID(id, a, b) ((a) + (id) * ((b) - (a)))

total: 0 errors, 4 warnings, 5 checks, 4619 lines checked
a6966a6827eb drm/i915/i915_reg.h: fix the checkpatch SPACE_BEFORE_TAB issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch SPACE_BEFORE_TAB

total: 0 errors, 1 warnings, 0 checks, 32 lines checked
77f23b3d193f drm/i915/i915_reg.h: fix the checkpatch MACRO_ARG_PRECEDENCE issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch MACRO_ARG_PRECEDENCE

total: 0 errors, 1 warnings, 0 checks, 138 lines checked

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Some checkpatch fixes for i915_reg.h

2018-06-12 Thread Patchwork
== Series Details ==

Series: Some checkpatch fixes for i915_reg.h
URL   : https://patchwork.freedesktop.org/series/44662/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
fa21130f6a88 drm/i915/i915_reg.h: fix the checkpatch SPACING issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch SPACING issues

-:50: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#50: FILE: drivers/gpu/drm/i915/i915_reg.h:144:
+#define _PIPE(pipe, a, b) ((a) + (pipe) * ((b) - (a)))

-:55: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#55: FILE: drivers/gpu/drm/i915/i915_reg.h:148:
+#define _TRANS(tran, a, b) ((a) + (tran) * ((b) - (a)))

-:58: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#58: FILE: drivers/gpu/drm/i915/i915_reg.h:150:
+#define _PORT(port, a, b) ((a) + (port) * ((b) - (a)))

-:63: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#63: FILE: drivers/gpu/drm/i915/i915_reg.h:154:
+#define _PLL(pll, a, b) ((a) + (pll) * ((b) - (a)))

-:1065: WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1065: FILE: drivers/gpu/drm/i915/i915_reg.h:2415:
+#define   DERRMR_PIPEB_SCANLINE ^I(1 << 8)$

-:1633: WARNING:LONG_LINE: line over 100 characters
#1633: FILE: drivers/gpu/drm/i915/i915_reg.h:3130:
+#define   PHY_STATUS_SPLINE_LDO(phy, ch, spline)(1 << (8 - (6 * (phy) 
+ 3 * (ch) + (spline

-:1845: WARNING:LONG_LINE_COMMENT: line over 100 characters
#1845: FILE: drivers/gpu/drm/i915/i915_reg.h:3645:
+#define   IMPROMOEN(1 << 10) /* promo is immediate or delayed 
until next idle interval (only for timeout method above) */

-:3186: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'a' - possible side-effects?
#3186: FILE: drivers/gpu/drm/i915/i915_reg.h:6826:
+#define _ID(id, a, b) ((a) + (id) * ((b) - (a)))

total: 0 errors, 4 warnings, 5 checks, 4617 lines checked
349125104d4c drm/i915/i915_reg.h: fix the checkpatch SPACE_BEFORE_TAB issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch SPACE_BEFORE_TAB

total: 0 errors, 1 warnings, 0 checks, 32 lines checked
a1e3b46722be drm/i915/i915_reg.h: fix the checkpatch MACRO_ARG_PRECEDENCE issues
-:4: WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not 
the tool that found it
#4: 
Subject: [PATCH] drm/i915/i915_reg.h: fix the checkpatch MACRO_ARG_PRECEDENCE

total: 0 errors, 1 warnings, 0 checks, 138 lines checked

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx