✗ CI.checkpatch: warning for drm/i915/display: conversion to struct intel_display

[Patchwork]

== Series Details ==

Series: drm/i915/display: conversion to struct intel_display
URL   : https://patchwork.freedesktop.org/series/137655/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
9fe5037901cabbcdf27a6fe0dfb047ca1474d363
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 08f5e72c164a50e87503d334ac0124e86097369d
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Thu Aug 22 19:04:55 2024 +0300

    drm/i915/display: convert params to struct intel_display
    
    Going forward, struct intel_display shall replace struct
    drm_i915_private as the main display device data pointer type. Convert
    intel_display_params.[ch] and intel_display_debugfs_params.[ch] to
    struct intel_display.
    
    Some stragglers are left behind where needed.
    
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 882c26c7017bdcc4eca493c7bf1ffb034d40be02 drm-intel
bbdf455fdd96 drm/i915/vblank: use drm_crtc_vblank_crtc() instead of open-coding
3df99bee84d9 drm/i915/vblank: fix context imbalance warnings
7331964bf9c5 drm/i915/vblank: convert to struct intel_display
-:204: WARNING:LONG_LINE: line length of 124 exceeds 100 columns
#204: FILE: drivers/gpu/drm/i915/display/intel_vblank.c:398:
+		position = (intel_de_read_fw(display, PIPEFRAMEPIXEL(display, pipe)) & PIPE_PIXEL_MASK) >> PIPE_PIXEL_SHIFT;

total: 0 errors, 1 warnings, 0 checks, 291 lines checked
d3392a12520d drm/i915/vrr: convert to struct intel_display
-:296: WARNING:LONG_LINE: line length of 106 exceeds 100 columns
#296: FILE: drivers/gpu/drm/i915/display/intel_vrr.c:413:
+							 TRANS_VRR_FLIPLINE(display, cpu_transcoder)) + 1;

total: 0 errors, 1 warnings, 0 checks, 289 lines checked
90eb850fc46e drm/i915/tv: convert to struct intel_display
-:349: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#349: FILE: drivers/gpu/drm/i915/display/intel_tv.c:1606:
+	save_tv_dac = tv_dac = intel_de_read(display, TV_DAC);

-:350: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#350: FILE: drivers/gpu/drm/i915/display/intel_tv.c:1607:
+	save_tv_ctl = tv_ctl = intel_de_read(display, TV_CTL);

total: 0 errors, 0 warnings, 2 checks, 514 lines checked
784630f00ffd drm/i915/sprite: convert to struct intel_display
08f5e72c164a drm/i915/display: convert params to struct intel_display
-:91: WARNING:MACRO_ARG_UNUSED: Argument 'T' is not used in function-like macro
#91: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:183:
+#define PRINT(T, x, ...) _param_print(p, display->drm->driver->name, #x, display->params.x);

-:91: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#91: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:183:
+#define PRINT(T, x, ...) _param_print(p, display->drm->driver->name, #x, display->params.x);

total: 0 errors, 2 warnings, 0 checks, 101 lines checked