✗ CI.checkpatch: warning for drm/i915/cmtg: Disable the CMTG (rev6)

[Patchwork]

== Series Details ==

Series: drm/i915/cmtg: Disable the CMTG (rev6)
URL   : https://patchwork.freedesktop.org/series/142946/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit d9ed368585eb0757e78dc0e18815aa83d74df5f9
Author: Gustavo Sousa <gustavo.sousa at intel.com>
Date:   Tue Jan 21 17:27:35 2025 -0300

    drm/i915/cmtg: Disable the CMTG
    
    The CMTG is a timing generator that runs in parallel with transcoders
    timing generators and can be used as a reference for synchronization.
    
    On PTL (display Xe3_LPD), we have observed that we are inheriting from
    GOP a display configuration with the CMTG enabled. Because our driver
    doesn't currently implement any CMTG sequences, the CMTG ends up still
    enabled after our driver takes over.
    
    We need to make sure that the CMTG is not enabled if we are not going to
    use it. For that, let's add a partial implementation in our driver that
    only cares about disabling the CMTG if it was found enabled during
    initial hardware readout. In the future, we can also implement sequences
    for enabling CMTG if that becomes a needed feature.
    
    For now, we only deal with cases when it is possible to disable the CMTG
    without requiring a modeset, which includes Xe3_LPD. For earlier display
    versions, we add a warning if we find the CMTG enabled and we can't
    disable it without a proper modeset.
    
    v2:
     - DG2 does not have the CMTG. Update HAS_CMTG() accordingly.
     - Update logic to force disabling of CMTG only for initial commit.
    v3:
     - Add missing changes for v2 that were staged but not committed.
    v4:
     - Avoid if/else duplication in intel_cmtg_dump_state() by using "n/a"
       for CMTG B enabled/disabled string for platforms without it. (Jani)
     - Prefer intel_cmtg_readout_hw_state() over intel_cmtg_readout_state().
       (Jani)
     - Use display struct instead of i915 as first parameter for
       TRANS_DDI_FUNC_CTL2(). (Jani)
     - Fewer continuation lines in variable declaration/initialization for
       better readability. (Jani)
     - Coding style improvements. (Jani)
     - Use drm_dbg_kms() instead of drm_info() for logging the disabling
       of the CMTG.
     - Make struct intel_cmtg_state entirely private to intel_cmtg.c.
    v5:
     - Do the disable sequence as part of the sanitization step after
       hardware readout instead of initial modeset commit. (Jani)
     - Adapt to commit 15133582465f ("drm/i915/display: convert global state
       to struct intel_display") by using a display struct instead of i915
       as argument for intel_atomic_global_obj_init().
    v6:
     - Do not track CMTG state as a global state. (Ville)
     - Simplify the driver logic by only disabling the CMTG only on cases
       when a modeset is not required. (Ville)
    
    Cc: Jani Nikula <jani.nikula at intel.com>
    Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
    Signed-off-by: Gustavo Sousa <gustavo.sousa at intel.com>
+ /mt/dim checkpatch 00b4340572541e1dcbacd7bd3efd360713553835 drm-intel
d9ed368585eb drm/i915/cmtg: Disable the CMTG
-:74: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#74: 
new file mode 100644

-:155: WARNING:LONG_LINE: line length of 110 exceeds 100 columns
#155: FILE: drivers/gpu/drm/i915/display/intel_cmtg.c:77:
+		    intel_cmtg_has_cmtg_b(display) ? str_enabled_disabled(cmtg_config->cmtg_b_enable) : "n/a",

-:311: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__display' - possible side-effects?
#311: FILE: drivers/gpu/drm/i915/display/intel_display_device.h:149:
+#define HAS_CMTG(__display)		(!(__display)->platform.dg2 && DISPLAY_VER(__display) >= 13)

total: 0 errors, 2 warnings, 1 checks, 255 lines checked