== Series Details == Series: drm/i915/pps: Some PPS cleanups (rev2) URL : https://patchwork.freedesktop.org/series/141029/ State : warning == Summary == Error: dim checkpatch failed ff7d1dd11395 drm/i915/pps: Store the power cycle delay without the +1 fc8b358de349 drm/i915/pps: Decouple pps delays from VBT struct definition f31f677dab2f drm/i915/pps: Rename intel_pps_delay members -:69: WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line #69: FILE: drivers/gpu/drm/i915/display/intel_pps.c:679: + * and then make panel wait for power_cycle if needed. */ total: 0 errors, 1 warnings, 0 checks, 209 lines checked 127aa6788808 drm/i915/lvds: Use struct intel_pps_delays for LVDS power sequencing 8b6c82f420f9 drm/i915/pps: Spell out the eDP spec power sequencing delays a bit more clearly 7a77e96187ed drm/i915/pps: Extract msecs_to_pps_units() -:55: WARNING:LONG_LINE_COMMENT: line length of 102 exceeds 100 columns #55: FILE: drivers/gpu/drm/i915/display/intel_pps.c:1522: + spec->backlight_off = msecs_to_pps_units(50); /* no limit for T9, make it symmetric with T8 */ total: 0 errors, 1 warnings, 0 checks, 47 lines checked d267e5a36046 drm/i915/pps: Extract pps_units_to_msecs() -:36: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'field' may be better as '(field)' to avoid precedence issues #36: FILE: drivers/gpu/drm/i915/display/intel_pps.c:1563: +#define get_delay(field) pps_units_to_msecs(final->field) total: 0 errors, 0 warnings, 1 checks, 20 lines checked 9be03212247a drm/i915/pps: Eliminate pointless get_delay() macro
