✗ CI.checkpatch: warning for Future-proof for multi-tile + multi-GT cases

[Patchwork]

== Series Details ==

Series: Future-proof for multi-tile + multi-GT cases
URL   : https://patchwork.freedesktop.org/series/150192/
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
202708c00696422fd217223bb679a353a5936e23
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 71afbb33c96c3dbba93ef3c53a6e5e11701645f0
Author: Matt Roper <matthew.d.roper at intel.com>
Date:   Thu Jun 12 17:14:44 2025 -0700

    drm/xe: Don't compare GT ID to GT count when determining valid GTs
    
    On current platforms with multiple GTs, all of the GT IDs are
    consecutive; as a result we know that the GT IDs range from 0 to
    gt_count-1 and can determine if a GT ID is valid by comparing against
    the count.  The consecutive nature of GT IDs may not hold true on future
    platforms if/when we have platforms that are both multi-tile and have
    multiple GTs within each tile.  Once such platforms exist, it's quite
    possible that we could wind up with something like a GT list composed of
    IDs 0, 2, and 3 with no GT 1 (which would be a 2-tile platform with
    media only on the second tile).
    
    To future-proof the code we should stop comparing against the GT count
    to determine whether a GT ID is valid or not.  Instead we should do an
    actual lookup of the ID to determine whether the GT exists.  This also
    means that our GT loop macro should not end at the GT count, but should
    rather examine the entire space up to (# of tiles) * (max GT per tile)
    to ensure it doesn't stop prematurely.
    
    Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
+ /mt/dim checkpatch 93593ca3b525390c11682f32f6d803032acd56b2 drm-intel
3127852a154a drm/xe: Export xe_step_name for kunit tests
-:9: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#9: 
  ERROR: modpost: "xe_step_name" [drivers/gpu/drm/xe/tests/xe_test.ko] undefined!

total: 0 errors, 1 warnings, 0 checks, 11 lines checked
b8e42ba265d0 drm/xe: Track maximum GTs per tile on a per-platform basis
af6a50d81323 drm/xe/tests/pci: Ensure all platforms have a valid GT/tile count
-:30: ERROR:OPEN_BRACE: that open brace { should be on the previous line
#30: FILE: drivers/gpu/drm/xe/tests/xe_pci.c:66:
+void xe_call_for_each_platform(xe_platform_fn xe_fn)
+{

total: 1 errors, 0 warnings, 0 checks, 166 lines checked
37516d7f9349 drm/xe: Assign GT IDs properly on multi-tile + multi-GT platforms
71afbb33c96c drm/xe: Don't compare GT ID to GT count when determining valid GTs
-:40: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#40: FILE: drivers/gpu/drm/xe/xe_device.h:131:
+	for ((id__) = 0; (id__) < (xe__)->info.tile_count * (xe__)->info.max_gt_per_tile; (id__)++) \

total: 0 errors, 1 warnings, 0 checks, 50 lines checked