[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/6] drm: Store the calculated vrefresh in the user mode

Patchwork patchwork at emeril.freedesktop.org
Thu Jun 28 19:57:44 UTC 2018


== Series Details ==

Series: series starting with [1/6] drm: Store the calculated vrefresh in the user mode
URL   : https://patchwork.freedesktop.org/series/45610/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
2a77f40fd19a drm: Store the calculated vrefresh in the user mode
ef213bd1f5e6 drm: Set mode->vrefresh before mode validation
5c5a934387d6 drm/i915/sdvo: Fix multi function encoder stuff
-:86: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#86: FILE: drivers/gpu/drm/i915/intel_sdvo.c:769:
+	if (IS_LVDS(intel_sdvo_connector) &&
 	   (intel_sdvo->sdvo_lvds_fixed_mode->hdisplay != width ||

total: 0 errors, 0 warnings, 1 checks, 271 lines checked
e191e1a7ccd5 drm/i915/sdvo: Utilize intel_panel for fixed_mode
-:54: WARNING:LONG_LINE: line over 100 characters
#54: FILE: drivers/gpu/drm/i915/intel_sdvo.c:1146:
+							     intel_sdvo_connector->base.panel.fixed_mode))

total: 0 errors, 1 warnings, 0 checks, 95 lines checked
3b94f6dc43fa drm/i915: Make sure panel fixed_mode has vrefresh populated
3fa2a42678d1 drm/i915: Filter out modes that don't match the fixed mode vrefresh
-:14: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit d93fa1b47b8f ("Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."")'
#14: 
that was reverted in commit d93fa1b47b8f ("Revert "drm/i915/edp:

-:90: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 16)
#90: FILE: drivers/gpu/drm/i915/intel_sdvo.c:1652:
 		if (mode->vdisplay > fixed_mode->vdisplay)
[...]
+		if (mode->vrefresh != fixed_mode->vrefresh)

total: 1 errors, 1 warnings, 0 checks, 41 lines checked



More information about the Intel-gfx mailing list