== Series Details == Series: Enable fastset for mbus_join state change URL : https://patchwork.freedesktop.org/series/130480/ State : warning == Summary == Error: dim checkpatch failed 5d2ec76ef8f2 drm/i915: Update mbus in intel_dbuf_mbus_update and do it properly -:9: WARNING:TYPO_SPELLING: 'swithing' may be misspelled - perhaps 'switching'? #9: or increasing amount of pipes(typical scenario is swithing between ^^^^^^^^ -:12: WARNING:TYPO_SPELLING: 'swithing' may be misspelled - perhaps 'switching'? #12: As of BSpec 49213 if we are swithing from multiple to single display ^^^^^^^^ -:38: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'old_dbuf_state->active_pipes == 0' #38: FILE: drivers/gpu/drm/i915/display/skl_watermark.c:3587: + if ((hweight8(new_dbuf_state->active_pipes) <= hweight8(old_dbuf_state->active_pipes)) + || (old_dbuf_state->active_pipes == 0)) -:39: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #39: FILE: drivers/gpu/drm/i915/display/skl_watermark.c:3588: + if ((hweight8(new_dbuf_state->active_pipes) <= hweight8(old_dbuf_state->active_pipes)) + || (old_dbuf_state->active_pipes == 0)) total: 0 errors, 2 warnings, 2 checks, 28 lines checked 763e4c15d36d drm/i915: Implement vblank synchronized MBUS join changes -:24: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #24: v4: - Place "single display to multiple displays scenario" MBUS/DBOX programming -:201: WARNING:LINE_SPACING: Missing a blank line after declarations #201: FILE: drivers/gpu/drm/i915/display/skl_watermark.c:3653: + int i; + intel_dbuf_mbus_update(state); -:212: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'crtc->pipe != pipe' #212: FILE: drivers/gpu/drm/i915/display/skl_watermark.c:3664: + if (!new_crtc_state->hw.active || (crtc->pipe != pipe)) total: 0 errors, 2 warnings, 1 checks, 211 lines checked
