[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Infoframe precompute/check (rev4)

[Patchwork]

== Series Details ==

Series: drm/i915: Infoframe precompute/check (rev4)
URL   : https://patchwork.freedesktop.org/series/49983/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
bd2b9221d985 video/hdmi: Constify 'buffer' to the unpack functions
865f880ff261 video/hdmi: Pass buffer size to infoframe unpack functions
2b2f89f9122e video/hdmi: Constify infoframe passed to the log functions
44007fc474cd video/hdmi: Constify infoframe passed to the pack functions
869ca9812ed0 video/hdmi: Add an enum for HDMI packet types
0ec089a8d890 video/hdmi: Handle the MPEG Source infoframe
-:353: CHECK:BOOL_MEMBER: Avoid using bool structure members because of possible alignment issues - see: https://lkml.org/lkml/2017/11/21/384
#353: FILE: include/linux/hdmi.h:355:
+	bool field_repeat;

total: 0 errors, 0 warnings, 1 checks, 340 lines checked
6f188967d1cb video/hdmi: Handle the NTSC VBI infoframe
d8151ddb397c drm/i915: Use memmove() for punching the hole into infoframes
052da5736c71 drm/i915: Pass intel_encoder to infoframe functions
14810ca2d1e7 drm/i915: Add the missing HDMI gamut metadata packet stuff
-:36: WARNING:LONG_LINE: line over 100 characters
#36: FILE: drivers/gpu/drm/i915/i915_reg.h:7956:
+#define HSW_TVIDEO_DIP_GMP_DATA(trans, i)	_MMIO_TRANS2(trans, _HSW_VIDEO_DIP_GMP_DATA_A + (i) * 4)

total: 0 errors, 1 warnings, 0 checks, 55 lines checked
bc1216d6c2e5 drm/i915: Return the mask of enabled infoframes from ->inforame_enabled()
fe8d386ab1bc drm/i915: Store mask of enabled infoframes in the crtc state
551baa38db82 drm/i915: Precompute HDMI infoframes
125c7a3847d1 drm/i915: Read out HDMI infoframes
f224f3b22927 drm/i915/sdvo: Precompute HDMI infoframes
26c55f850c3d drm/i915/sdvo: Read out HDMI infoframes
c97506450a5c drm/i915: Check infoframe state in intel_pipe_config_compare()
-:70: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'name' - possible side-effects?
#70: FILE: drivers/gpu/drm/i915/intel_display.c:11575:
+#define PIPE_CONF_CHECK_INFOFRAME(name) do { \
+	if (!intel_compare_infoframe(&current_config->infoframes.name, \
+				     &pipe_config->infoframes.name)) { \
+		pipe_config_infoframe_err(dev_priv, adjust, __stringify(name), \
+					  &current_config->infoframes.name, \
+					  &pipe_config->infoframes.name); \
+		ret = false; \
+	} \
+} while (0)

total: 0 errors, 0 warnings, 1 checks, 67 lines checked
8fdf44951d06 drm/i915: Include infoframes in the crtc state dump