[Intel-xe] ✗ CI.checkpatch: warning for drm/xe: Remove GEN[0-9]*_ prefixes

Patchwork patchwork at emeril.freedesktop.org
Mon Nov 20 10:21:16 UTC 2023


== Series Details ==

Series: drm/xe: Remove GEN[0-9]*_ prefixes
URL   : https://patchwork.freedesktop.org/series/126558/
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
63c2b6b160bca2df6efc7bc4cea6f442097d7854
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit cc32ec062ffea4de2fd4eeeb21e80022891261ba
Author: Lucas De Marchi <lucas.demarchi at intel.com>
Date:   Thu Nov 16 14:23:16 2023 -0800

    drm/xe: Remove GEN[0-9]*_ prefixes
    
    After noticing in logs there there were still mentions to GEN6
    registers, it was clear commit 860bc3f9b76d ("drm/xe: Drop gen afixes
    from registers") didn't take care of all the afixes. Some were added
    later, but there are also constans and strings still using that.
    Continue the cleanup removing the remaining ones.
    
    To keep it consistent with code nearby, a few other changes are made:
    
            - Remove prefix in INTEL_LEGACY_64B_CONTEXT
            - Remove GEN8_CTX_L3LLC_COHERENT since it's unused
            - Rename GEN9_FREQ_SCALER to GT_FREQUENCY_SCALER
    
    Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
    Reviewed-by: Matt Roper <matthew.d.roper at intel.com>
+ /mt/dim checkpatch b25aa17bffa88c86fec716e40bdb3848eea17b23 drm-intel
cc32ec062 drm/xe: Remove GEN[0-9]*_ prefixes
-:6: WARNING:REPEATED_WORD: Possible repeated word: 'there'
#6: 
After noticing in logs there there were still mentions to GEN6

-:7: WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '860bc3f9b76d', maybe rebased or not pulled?
#7: 
registers, it was clear commit 860bc3f9b76d ("drm/xe: Drop gen afixes

-:296: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#296: FILE: drivers/gpu/drm/xe/xe_mocs.c:152:
+#define MOCS_ENTRIES \
 	/* Entries 0 and 1 are defined per-platform */ \
 	/* Base - L3 + LLC */ \
 	MOCS_ENTRY(2, \

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




More information about the Intel-xe mailing list