✗ CI.checkpatch: warning for drm/xe: Cleanup GGTT usage outside of xe_ggtt.c

Patchwork patchwork at emeril.freedesktop.org
Fri Apr 4 21:04:57 UTC 2025


== Series Details ==

Series: drm/xe: Cleanup GGTT usage outside of xe_ggtt.c
URL   : https://patchwork.freedesktop.org/series/147281/
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
99e5a866b5e13f134e606a3e29d9508d97826fb3
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 7f1ba551a1a2ab36786d6e6b680679ca9df7a78e
Author: Maarten Lankhorst <dev at lankhorst.se>
Date:   Fri Apr 4 22:50:28 2025 +0200

    drm/xe: Implement a helper for reading out a GGTT PTE at a specified offset
    
    Split the GGTT PTE readout to a separate function, this is useful for
    adding testcases in the next commit, and also cleaner than manually
    reading out GGTT.
    
    Signed-off-by: Maarten Lankhorst <dev at lankhorst.se>
+ /mt/dim checkpatch ebade66890117778809714085d5151b0050488a8 drm-intel
2b6856af3833 drm/xe: Use xe_ggtt_map_bo_unlocked for resume
ea06c84bf754 drm/xe: Add xe_ggtt_might_lock
4832cb913804 drm/xe: Add xe_ggtt_alloc
-:32: WARNING:LINE_SPACING: Missing a blank line after declarations
#32: FILE: drivers/gpu/drm/xe/xe_ggtt.c:174:
+	struct xe_ggtt *ggtt = drmm_kzalloc(&tile_to_xe(tile)->drm, sizeof(*ggtt), GFP_KERNEL);
+	if (ggtt)

total: 0 errors, 1 warnings, 0 checks, 45 lines checked
b189ab8e4f20 drm/xe/display: Remove dereferences of ggtt for tile id
88e3a657f7b6 drm/xe/ggtt: Seperate flags and address in PTE encoding
-:4: WARNING:TYPO_SPELLING: 'Seperate' may be misspelled - perhaps 'Separate'?
#4: 
Subject: [PATCH] drm/xe/ggtt: Seperate flags and address in PTE encoding
                              ^^^^^^^^

-:160: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#160: FILE: drivers/gpu/drm/xe/xe_ggtt.c:662:
+		     xe_res_next(&cur, XE_PAGE_SIZE)) {
+

total: 0 errors, 1 warnings, 1 checks, 187 lines checked
7a230e6038b7 drm/xe/display: Dont poke into GGTT internals to fill a DPT
-:54: WARNING:LINE_SPACING: Missing a blank line after declarations
#54: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:66:
+			u64 addr = xe_bo_addr(bo, src_idx * XE_PAGE_SIZE, XE_PAGE_SIZE);
+			iosys_map_wr(map, *dpt_ofs, u64, pte | addr);

total: 0 errors, 1 warnings, 0 checks, 79 lines checked
799603496c45 drm/xe/display: Convert GGTT mapping to use pte_encode_flags
13efa9ee4ec1 drm/xe: Remove pte_encode_bo callback
7f1ba551a1a2 drm/xe: Implement a helper for reading out a GGTT PTE at a specified offset




More information about the Intel-xe mailing list