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 Bigjoiner atomic preparations.
Date: Tue, 02 Jul 2019 21:30:19 -0000 [thread overview]
Message-ID: <20190702213019.28691.71783@emeril.freedesktop.org> (raw)
In-Reply-To: <20190702194205.13366-1-maarten.lankhorst@linux.intel.com>
== Series Details ==
Series: Bigjoiner atomic preparations.
URL : https://patchwork.freedesktop.org/series/63094/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
b9e17bf6bac7 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.
-:1968: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#1968: FILE: drivers/gpu/drm/i915/display/intel_display.c:11139:
+ crtc_state->uapi.active = crtc_state->uapi.enable = true;
-:2625: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#2625: FILE: drivers/gpu/drm/i915/display/intel_display.c:16640:
+ crtc_state->hw.active = crtc_state->hw.enable =
-:3731: ERROR:CODE_INDENT: code indent should use tabs where possible
#3731: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:223:
+^I^I^I^I new_crtc_state->uapi.event);$
-:3731: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#3731: FILE: drivers/gpu/drm/i915/display/intel_sprite.c:223:
+ drm_crtc_arm_vblank_event(&crtc->base,
+ new_crtc_state->uapi.event);
total: 1 errors, 1 warnings, 3 checks, 4166 lines checked
fe8910ead298 drm/i915: Handle a few more cases for hw/sw split
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 0 checks, 118 lines checked
d31a4fc4501d drm/i915: Complete sw/hw split
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 0 checks, 201 lines checked
9516b4dffdfa drm/i915/dp: Allow big joiner modes in intel_dp_mode_valid()
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 0 checks, 22 lines checked
1909a2665a65 drm/i915/dp: Validate modes that can only be handled in a bigjoiner configuration
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 0 checks, 307 lines checked
deb270f134b8 drm/i915: Try to make bigjoiner work in atomic check.
-:54: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#54: FILE: drivers/gpu/drm/i915/display/intel_display.c:12237:
+ crtc_state->nv12_planes = crtc_state->c8_planes = crtc_state->update_planes = 0;
-:74: WARNING:LONG_LINE: line over 100 characters
#74: FILE: drivers/gpu/drm/i915/display/intel_display.c:13624:
+ struct intel_crtc_state *old_crtc_state, *new_crtc_state, *slave_crtc_state, *master_crtc_state;
total: 0 errors, 1 warnings, 1 checks, 230 lines checked
f41590956879 drm/i915: Allow vdsc functions to be called without encoder.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2019-07-02 21:30 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-02 19:41 [RFC 0/7] Bigjoiner atomic preparations Maarten Lankhorst
2019-07-02 19:41 ` [RFC 1/7] drm/i915: Prepare to split crtc state in uapi and hw state Maarten Lankhorst
2019-07-02 19:42 ` [RFC 2/7] drm/i915: Handle a few more cases for hw/sw split Maarten Lankhorst
2019-07-02 19:42 ` [RFC 3/7] drm/i915: Complete sw/hw split Maarten Lankhorst
2019-07-02 19:42 ` [RFC 4/7] drm/i915/dp: Allow big joiner modes in intel_dp_mode_valid() Maarten Lankhorst
2019-07-03 0:13 ` Manasi Navare
2019-07-02 19:42 ` [RFC 5/7] drm/i915/dp: Validate modes that can only be handled in a bigjoiner configuration Maarten Lankhorst
2019-07-02 19:42 ` [RFC 6/7] drm/i915: Try to make bigjoiner work in atomic check Maarten Lankhorst
2019-07-03 0:32 ` Manasi Navare
2019-07-03 8:02 ` Maarten Lankhorst
2019-07-02 19:42 ` [RFC 7/7] drm/i915: Allow vdsc functions to be called without encoder Maarten Lankhorst
2019-07-10 22:35 ` Manasi Navare
2019-07-30 9:08 ` Maarten Lankhorst
2019-08-21 19:54 ` Manasi Navare
2019-07-02 21:30 ` Patchwork [this message]
2019-07-02 21:33 ` ✗ Fi.CI.SPARSE: warning for Bigjoiner atomic preparations Patchwork
2019-07-02 21:59 ` ✓ Fi.CI.BAT: success " Patchwork
2019-07-03 22:04 ` ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190702213019.28691.71783@emeril.freedesktop.org \
--to=patchwork@emeril.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=maarten.lankhorst@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox