[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Proper dbuf global state (rev3)

[Patchwork]

== Series Details ==

Series: drm/i915: Proper dbuf global state (rev3)
URL   : https://patchwork.freedesktop.org/series/73421/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
3643e15727b1 drm/i915: Handle some leftover s/intel_crtc/crtc/
c2f2161d6505 drm/i915: Remove garbage WARNs
2786de40d65c drm/i915: Add missing commas to dbuf tables
73430ed2f955 drm/i915: Use a sentinel to terminate the dbuf slice arrays
047f693c4cfa drm/i915: Make skl_compute_dbuf_slices() behave consistently for all platforms
391eab3b022c drm/i915: Polish some dbuf debugs
a684f63e9f43 drm/i915: Unify the low level dbuf code
e3028c82388d drm/i915: Introduce proper dbuf state
-:175: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#175: FILE: drivers/gpu/drm/i915/display/intel_display.c:18381:
+	dev_priv->active_pipes = cdclk_state->active_pipes =

-:623: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'state' - possible side-effects?
#623: FILE: drivers/gpu/drm/i915/intel_pm.h:77:
+#define intel_atomic_get_old_dbuf_state(state) \
+	to_intel_dbuf_state(intel_atomic_get_old_global_obj_state(state, &to_i915(state->base.dev)->dbuf.obj))

-:624: WARNING:LONG_LINE: line over 100 characters
#624: FILE: drivers/gpu/drm/i915/intel_pm.h:78:
+	to_intel_dbuf_state(intel_atomic_get_old_global_obj_state(state, &to_i915(state->base.dev)->dbuf.obj))

-:625: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'state' - possible side-effects?
#625: FILE: drivers/gpu/drm/i915/intel_pm.h:79:
+#define intel_atomic_get_new_dbuf_state(state) \
+	to_intel_dbuf_state(intel_atomic_get_new_global_obj_state(state, &to_i915(state->base.dev)->dbuf.obj))

-:626: WARNING:LONG_LINE: line over 100 characters
#626: FILE: drivers/gpu/drm/i915/intel_pm.h:80:
+	to_intel_dbuf_state(intel_atomic_get_new_global_obj_state(state, &to_i915(state->base.dev)->dbuf.obj))

total: 0 errors, 2 warnings, 3 checks, 555 lines checked
071b62e2fe20 drm/i915: Nuke skl_ddb_get_hw_state()
2b355bb11548 drm/i915: Move the dbuf pre/post plane update
35835c92bd7f drm/i915: Clean up dbuf debugs during .atomic_check()
7f9190b0dc0b drm/i915: Extract intel_crtc_ddb_weight()
45c49c9f3b73 drm/i915: Pass the crtc to skl_compute_dbuf_slices()
3f877cd02847 drm/i915: Introduce intel_dbuf_slice_size()
373ddcca596a drm/i915: Introduce skl_ddb_entry_for_slices()
aa3deb4ba4f5 drm/i915: Move pipe ddb entries into the dbuf state
80543eb34ce4 drm/i915: Extract intel_crtc_dbuf_weights()
-:137: WARNING:LINE_SPACING: Missing a blank line after declarations
#137: FILE: drivers/gpu/drm/i915/intel_pm.c:3960:
+				   struct skl_ddb_entry *alloc, /* out */
+				   int *num_active /* out */)

total: 0 errors, 1 warnings, 0 checks, 176 lines checked
5d4ec45f7e49 drm/i915: Encapsulate dbuf state handling harder
1fc35297be7f drm/i915: Do a bit more initial readout for dbuf
9cba9c99fb78 drm/i915: Check slice mask for holes