✗ CI.checkpatch: warning for drm/xe: Ensure display fb is aligned in GGTT to a multiple of 64k, through padding
Patchwork
patchwork at emeril.freedesktop.org
Mon Aug 19 15:39:41 UTC 2024
== Series Details ==
Series: drm/xe: Ensure display fb is aligned in GGTT to a multiple of 64k, through padding
URL : https://patchwork.freedesktop.org/series/137462/
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 91c54ea93a8df194a4c1a19a37a7c9c12d15e02b
Author: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
Date: Mon Aug 19 17:31:27 2024 +0200
drm/xe: Ensure display fb is aligned in GGTT to a multiple of 64k, through padding
This workaround is needed on battlemage to ensure that there is no
corruption when CCS is used.
For testing, always enable the workaround. Should be easier to see if something blows up. :)
Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
+ /mt/dim checkpatch 4574b385fd4f5016478f5ddc29e0a8b5e338b0ac drm-intel
91c54ea93a8d drm/xe: Ensure display fb is aligned in GGTT to a multiple of 64k, through padding
-:10: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#10:
For testing, always enable the workaround. Should be easier to see if something blows up. :)
-:22: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#22: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:18:
+static inline bool needs_bmg_64k_workaround(struct xe_device *xe, const struct intel_framebuffer *fb)
-:56: WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#56: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:52:
+ /* Start over with the first few pages, dpt is always aligned to a multiple
+ * of 512 pages, which means that there is enough padding here
total: 0 errors, 3 warnings, 0 checks, 137 lines checked
More information about the Intel-xe
mailing list