✗ Fi.CI.CHECKPATCH: warning for drm/i915: Bigjoiner preparations.

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Bigjoiner preparations.
Date: Wed, 21 Aug 2019 14:41:57 -0000	[thread overview]
Message-ID: <20190821144157.4278.75521@emeril.freedesktop.org> (raw)
In-Reply-To: <20190821133221.29456-1-maarten.lankhorst@linux.intel.com>

== Series Details ==

Series: drm/i915: Bigjoiner preparations.
URL   : https://patchwork.freedesktop.org/series/65543/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
bb9674943922 drm/i915/dp: Fix dsc bpp calculations.
0e2d405f7bbf drm/i915: Prepare to split crtc state in uapi and hw state
-:11: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#11: 
- crtc, *_changed flags, event, commit, state, mode_blob, (plane/connector/encoder)_mask.

-:1976: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#1976: FILE: drivers/gpu/drm/i915/display/intel_display.c:11171:
+	crtc_state->uapi.active = crtc_state->uapi.enable = true;

-:2646: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#2646: FILE: drivers/gpu/drm/i915/display/intel_display.c:16658:
+		crtc_state->hw.active = crtc_state->hw.enable =

-:3798: ERROR:CODE_INDENT: code indent should use tabs where possible
#3798: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:224:
+^I^I^I^I          new_crtc_state->uapi.event);$

-:3798: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#3798: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:224:
+		drm_crtc_arm_vblank_event(&crtc->base,
+				          new_crtc_state->uapi.event);

total: 1 errors, 1 warnings, 3 checks, 4194 lines checked
15321f86c2b3 drm/i915: Handle a few more cases for hw/sw split
4c4ad0feadf2 drm/i915: Complete sw/hw split
8bc9c0b2e0a0 drm/i915: Kill off is_planar_yuv_format
782d5f4e19cc drm/i915: Get rid of crtc_state->fb_changed
2c4f08fcf5ee drm/i915: Remove begin/finish_crtc_commit.
bb6ec153c77c drm/i915: Do not add all planes when checking scalers on glk+
c3e067461453 drm/i915: Add debugfs entries for reading out DPCD DSC and FEC.
1f51e8536ef9 drm/i915: Move FEC enable timeout wait to enable_ddi_dp
-:6: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#6: 
Even without bigjoiner I get a timeout when enabling FEC, the length of the timeout

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

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