[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915: Prepare to split crtc state in uapi and hw state
Patchwork
patchwork at emeril.freedesktop.org
Wed Sep 25 16:47:23 UTC 2019
== Series Details ==
Series: series starting with [1/4] drm/i915: Prepare to split crtc state in uapi and hw state
URL : https://patchwork.freedesktop.org/series/67227/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
4cc35a6ea9c7 drm/i915: Prepare to split crtc state in uapi and hw state
-:2130: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#2130: FILE: drivers/gpu/drm/i915/display/intel_display.c:11264:
+ crtc_state->uapi.active = crtc_state->uapi.enable = true;
-:2837: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#2837: FILE: drivers/gpu/drm/i915/display/intel_display.c:16735:
+ crtc_state->hw.active = crtc_state->hw.enable =
-:4001: ERROR:CODE_INDENT: code indent should use tabs where possible
#4001: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:211:
+^I^I^I^I new_crtc_state->uapi.event);$
-:4001: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#4001: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:211:
+ drm_crtc_arm_vblank_event(&crtc->base,
+ new_crtc_state->uapi.event);
total: 1 errors, 0 warnings, 3 checks, 4371 lines checked
b6094973d8a5 drm/i915: Handle a few more cases for hw/sw split
864203251a89 drm/i915: Complete sw/hw split, v2.
-:15: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#15:
- Clear crtc_state->hw on disable, instead of using clear_intel_crtc_state().
total: 0 errors, 1 warnings, 0 checks, 204 lines checked
fd18dc2e2495 drm/i915: Remove begin/finish_crtc_commit, v2.
-:125: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#125: FILE: drivers/gpu/drm/i915/display/intel_display.c:13747:
+ I915_WRITE(PIPESRC(crtc->pipe),
+ ((new_crtc_state->pipe_src_w - 1) << 16) |
total: 0 errors, 0 warnings, 1 checks, 238 lines checked
More information about the Intel-gfx
mailing list