== 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(¤t_config->infoframes.name, \ + &pipe_config->infoframes.name)) { \ + pipe_config_infoframe_err(dev_priv, adjust, __stringify(name), \ + ¤t_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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx