== Series Details ==
Series: series starting with [1/5] drm/i915: Switch
intel_legacy_cursor_update() to intel_ types
URL : https://patchwork.freedesktop.org/series/67337/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
6e83c3bdcc40 drm/i915: Switch intel_legacy_cursor_update() to intel_ types
3cdcb4af301c drm/i915: Prepare the connector/encoder mask eadout for hw vs.
uapi state split
4d9d48f0a291 drm/i915: Prepare the mode readout for hw vs. uapi state split
cd2aa84c322f drm/i915: Add intel_atomic_crtc_state_for_each_plane_state()
-:31: WARNING:LONG_LINE: line over 100 characters
#31: FILE: drivers/gpu/drm/i915/display/intel_atomic.c:451:
+ return
to_intel_plane_state(state->base.planes[drm_plane_index(&plane->base)].state);
-:67: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in
parentheses
#67: FILE: drivers/gpu/drm/i915/display/intel_display.h:443:
+#define intel_for_each_plane_mask(plane, dev, plane_mask) \
+ list_for_each_entry((plane), &(dev)->mode_config.plane_list, base.head)
\
+ for_each_if ((plane_mask) & drm_plane_mask(&(plane)->base))
-:67: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'plane' - possible
side-effects?
#67: FILE: drivers/gpu/drm/i915/display/intel_display.h:443:
+#define intel_for_each_plane_mask(plane, dev, plane_mask) \
+ list_for_each_entry((plane), &(dev)->mode_config.plane_list, base.head)
\
+ for_each_if ((plane_mask) & drm_plane_mask(&(plane)->base))
-:69: WARNING:SPACING: space prohibited between function name and open
parenthesis '('
#69: FILE: drivers/gpu/drm/i915/display/intel_display.h:445:
+ for_each_if ((plane_mask) & drm_plane_mask(&(plane)->base))
-:71: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in
parentheses
#71: FILE: drivers/gpu/drm/i915/display/intel_display.h:447:
+#define intel_atomic_crtc_state_for_each_plane_state(plane, plane_state,
crtc_state, state) \
+ intel_for_each_plane_mask((plane), (state)->base.dev,
(crtc_state)->base.plane_mask) \
+ for_each_if (((plane_state) = \
+ __intel_atomic_get_current_plane_state((state),
(plane))))
-:71: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'plane' - possible
side-effects?
#71: FILE: drivers/gpu/drm/i915/display/intel_display.h:447:
+#define intel_atomic_crtc_state_for_each_plane_state(plane, plane_state,
crtc_state, state) \
+ intel_for_each_plane_mask((plane), (state)->base.dev,
(crtc_state)->base.plane_mask) \
+ for_each_if (((plane_state) = \
+ __intel_atomic_get_current_plane_state((state),
(plane))))
-:71: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'state' - possible
side-effects?
#71: FILE: drivers/gpu/drm/i915/display/intel_display.h:447:
+#define intel_atomic_crtc_state_for_each_plane_state(plane, plane_state,
crtc_state, state) \
+ intel_for_each_plane_mask((plane), (state)->base.dev,
(crtc_state)->base.plane_mask) \
+ for_each_if (((plane_state) = \
+ __intel_atomic_get_current_plane_state((state),
(plane))))
-:73: WARNING:SPACING: space prohibited between function name and open
parenthesis '('
#73: FILE: drivers/gpu/drm/i915/display/intel_display.h:449:
+ for_each_if (((plane_state) = \
total: 2 errors, 3 warnings, 3 checks, 255 lines checked
7009d869fbe6 drm/i915: Add intel_atomic_add_affected_planes()
-:47: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#47: FILE: drivers/gpu/drm/i915/display/intel_atomic.c:469:
+
+}
total: 0 errors, 0 warnings, 1 checks, 49 lines checked
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
