== Series Details == Series: refactor display structs a little bit URL : https://patchwork.freedesktop.org/series/94367/ State : warning == Summary == $ dim checkpatch origin/drm-tip f4fbd6000df1 drm/i915: move display funcs into a display struct. 76dc91485857 drm/i915/display: move cdclk info into display -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:662: WARNING:LONG_LINE: line length of 120 exceeds 100 columns #662: FILE: drivers/gpu/drm/i915/display/intel_cdclk.h:77: + to_intel_cdclk_state(intel_atomic_get_old_global_obj_state(state, &to_i915(state->base.dev)->display.cdclk.obj)) -:665: WARNING:LONG_LINE: line length of 120 exceeds 100 columns #665: FILE: drivers/gpu/drm/i915/display/intel_cdclk.h:79: + to_intel_cdclk_state(intel_atomic_get_new_global_obj_state(state, &to_i915(state->base.dev)->display.cdclk.obj)) -:711: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #711: FILE: drivers/gpu/drm/i915/display/intel_display.c:11273: + cdclk_state->logical = cdclk_state->actual = i915->display.cdclk.hw; total: 0 errors, 3 warnings, 1 checks, 782 lines checked 7b219ea05311 drm/i915: move more pll/clocks into display struct. -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:149: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #149: FILE: drivers/gpu/drm/i915/display/intel_display.c:179: + dev_priv->display.czclk_freq = vlv_get_cck_clock_hpll(dev_priv, "czclk", CCK_CZ_CLOCK_CONTROL); total: 0 errors, 1 warnings, 1 checks, 302 lines checked 38bff6053be9 drm/i915/display: move gmbus into display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:231: WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line #231: FILE: drivers/gpu/drm/i915/i915_drv.h:860: + * controller on different i2c buses. */ -:300: WARNING:LONG_LINE_COMMENT: line length of 104 exceeds 100 columns #300: FILE: drivers/gpu/drm/i915/i915_reg.h:3453: +#define GMBUS0 _MMIO(dev_priv->display.gpio_mmio_base + 0x5100) /* clock/port select */ -:309: WARNING:LONG_LINE_COMMENT: line length of 101 exceeds 100 columns #309: FILE: drivers/gpu/drm/i915/i915_reg.h:3462: +#define GMBUS1 _MMIO(dev_priv->display.gpio_mmio_base + 0x5104) /* command/status */ -:328: WARNING:LONG_LINE_COMMENT: line length of 108 exceeds 100 columns #328: FILE: drivers/gpu/drm/i915/i915_reg.h:3485: +#define GMBUS3 _MMIO(dev_priv->display.gpio_mmio_base + 0x510c) /* data buffer bytes 3-0 */ -:329: WARNING:LONG_LINE_COMMENT: line length of 113 exceeds 100 columns #329: FILE: drivers/gpu/drm/i915/i915_reg.h:3486: +#define GMBUS4 _MMIO(dev_priv->display.gpio_mmio_base + 0x5110) /* interrupt mask (Pineview+) */ total: 0 errors, 6 warnings, 0 checks, 282 lines checked bb277479cb80 drm/i915/display: move intel_dmc into display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:81: WARNING:LONG_LINE: line length of 113 exceeds 100 columns #81: FILE: drivers/gpu/drm/i915/display/intel_display_power.c:968: + DMC_PROGRAM(dev_priv->display.dmc.dmc_info[DMC_FW_MAIN].start_mmioaddr, 0)), total: 0 errors, 2 warnings, 0 checks, 322 lines checked 6ce1abe08452 drm/i915/display: move mipi_mmio_base to display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one total: 0 errors, 1 warnings, 0 checks, 401 lines checked f924c533b785 drm/i915/display: move pps_mmio_base to display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one total: 0 errors, 1 warnings, 0 checks, 36 lines checked 00d5669bd6e9 drm/i915/drrs: just use some local vars to simplify drrs code abfd2f3e84da drm/i915/display: move drrs into display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one total: 0 errors, 1 warnings, 0 checks, 146 lines checked bc448a9d147a drm/i915/display: move fbc into display struct -:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one total: 0 errors, 1 warnings, 0 checks, 420 lines checked