== Series Details == Series: drm/i915: Move PCH modeset code into its own file URL : https://patchwork.freedesktop.org/series/95863/ State : warning == Summary == $ dim checkpatch origin/drm-tip ae28f635f0fc drm/i915: Move PCH refclok stuff into its own file -:802: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #802: new file mode 100644 -:1002: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1002: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:196: + udelay(24); -:1069: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1069: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:263: + udelay(24); -:1105: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1105: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:299: + udelay(32); -:1117: ERROR:SPACING: space prohibited after that open parenthesis '(' #1117: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:311: + [BEND_IDX( 50)] = 0x3B23, -:1118: ERROR:SPACING: space prohibited after that open parenthesis '(' #1118: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:312: + [BEND_IDX( 45)] = 0x3B23, -:1119: ERROR:SPACING: space prohibited after that open parenthesis '(' #1119: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:313: + [BEND_IDX( 40)] = 0x3C23, -:1120: ERROR:SPACING: space prohibited after that open parenthesis '(' #1120: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:314: + [BEND_IDX( 35)] = 0x3C23, -:1121: ERROR:SPACING: space prohibited after that open parenthesis '(' #1121: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:315: + [BEND_IDX( 30)] = 0x3D23, -:1122: ERROR:SPACING: space prohibited after that open parenthesis '(' #1122: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:316: + [BEND_IDX( 25)] = 0x3D23, -:1123: ERROR:SPACING: space prohibited after that open parenthesis '(' #1123: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:317: + [BEND_IDX( 20)] = 0x3E23, -:1124: ERROR:SPACING: space prohibited after that open parenthesis '(' #1124: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:318: + [BEND_IDX( 15)] = 0x3E23, -:1125: ERROR:SPACING: space prohibited after that open parenthesis '(' #1125: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:319: + [BEND_IDX( 10)] = 0x3F23, -:1126: ERROR:SPACING: space prohibited after that open parenthesis '(' #1126: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:320: + [BEND_IDX( 5)] = 0x3F23, -:1127: ERROR:SPACING: space prohibited after that open parenthesis '(' #1127: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:321: + [BEND_IDX( 0)] = 0x0025, -:1128: ERROR:SPACING: space prohibited after that open parenthesis '(' #1128: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:322: + [BEND_IDX( -5)] = 0x0025, -:1394: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1394: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:588: + udelay(200); -:1413: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1413: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:607: + udelay(200); -:1424: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1424: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:618: + udelay(200); -:1438: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #1438: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:632: + udelay(200); -:1442: WARNING:AVOID_BUG: Avoid crashing the kernel - try using WARN_ON & recovery code rather than BUG() or BUG_ON() #1442: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.c:636: + BUG_ON(val != final); total: 12 errors, 2 warnings, 7 checks, 1411 lines checked 0a9f7351d945 drm/i915: Move PCH modeset code to its own file -:453: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #453: new file mode 100644 -:764: CHECK:SPACING: No space is necessary after a cast #764: FILE: drivers/gpu/drm/i915/display/intel_pch_display.c:307: + assert_fdi_tx_enabled(dev_priv, (enum pipe) cpu_transcoder); total: 0 errors, 1 warnings, 1 checks, 796 lines checked 03d86e18ea3e drm/i915: Clean up the {ilk, lpt}_pch_enable() calling convention 32638e909748 drm/i915: Move LPT PCH readout code a92a0a2e979c drm/i915: Extract ilk_pch_get_config() -:192: CHECK:SPACING: No space is necessary after a cast #192: FILE: drivers/gpu/drm/i915/display/intel_pch_display.c:346: + pll_id = (enum intel_dpll_id) pipe; total: 0 errors, 0 warnings, 1 checks, 188 lines checked 03365a324a75 drm/i915: Move iCLKIP readout to the pch code b9dd6f1b71d4 drm/i915: Introduce ilk_pch_disable() and ilk_pch_post_disable() bc37d7855c26 drm/i915: Move intel_ddi_fdi_post_disable() to fdi code 4d8f6f29f43b drm/i915: Introduce lpt_pch_disable()