All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patchwork <patchwork@emeril.freedesktop.org>
To: "Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>
Cc: intel-xe@lists.freedesktop.org
Subject: ✗ CI.checkpatch: warning for series starting with [CI-test,1/8] drm: Add drm_vblank_work_flush_all().
Date: Wed, 27 Mar 2024 19:25:43 -0000	[thread overview]
Message-ID: <171156754381.1060333.15872018127803336158@8e613ede5ea5> (raw)
In-Reply-To: <20240327190710.2359-1-maarten.lankhorst@linux.intel.com>

== Series Details ==

Series: series starting with [CI-test,1/8] drm: Add drm_vblank_work_flush_all().
URL   : https://patchwork.freedesktop.org/series/131706/
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
10b531c2aeb176a1a539b4a77216232f97719cec
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit c459a8b0337a1c8b9a90d71cebb5032f0254f684
Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Date:   Wed Mar 27 20:07:09 2024 +0100

    drm/xe/display: Re-use display vmas when possible
    
    i915 has this really nice, infrastructure where everything becomes
    complicated, GGTT needs eviction, etc..
    
    Lets not do that, and make the dumbest possible interface instead.
    Try to retrieve the VMA from old_plane_state, or intel_fbdev if kernel
    fb.
    
    Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
+ /mt/dim checkpatch 5e11d8eaccbd1c0a6b16bc6622dcbb78c350d640 drm-intel
c25b49ebd032 drm: Add drm_vblank_work_flush_all().
-:33: WARNING:WAITQUEUE_ACTIVE: waitqueue_active without comment
#33: FILE: drivers/gpu/drm/drm_vblank_work.c:249:
+			    waitqueue_active(&vblank->work_wait_queue),

total: 0 errors, 1 warnings, 0 checks, 41 lines checked
f448e1417cc9 drm/i915: Use vblank worker to unpin old legacy cursor fb safely
95bd1b08aea4 drm/i915: Use the same vblank worker for atomic unpin
-:107: WARNING:LONG_LINE: line length of 105 exceeds 100 columns
#107: FILE: drivers/gpu/drm/i915/display/intel_crtc.c:641:
+							 drm_crtc_accurate_vblank_count(&crtc->base) + 1,

-:110: WARNING:LONG_LINE_COMMENT: line length of 110 exceeds 100 columns
#110: FILE: drivers/gpu/drm/i915/display/intel_crtc.c:644:
+				/* Remove plane from atomic state, cleanup/free is done from vblank worker. */

total: 0 errors, 2 warnings, 0 checks, 98 lines checked
45344f253d94 drm/xe: Fix bo leak in intel_fb_bo_framebuffer_init
dc8993b3887a drm/xe/display: Preparations for preallocating dpt bo
7637975d6ba2 drm/xe: Use simple xchg to cache DPT
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

total: 0 errors, 1 warnings, 0 checks, 100 lines checked
60e1cc7e94ab drm/xe/display: Prevent overwriting original GGTT when taking over initial FB.
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
-:126: CHECK:LINE_SPACING: Please don't use multiple blank lines
#126: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:436:
+
+

-:146: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#146: 
new file mode 100644

total: 0 errors, 1 warnings, 1 checks, 222 lines checked
c459a8b0337a drm/xe/display: Re-use display vmas when possible
-:180: ERROR:CODE_INDENT: code indent should use tabs where possible
#180: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:465:
+^I            sizeof(new_plane_state->view.gtt))) {$

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



  parent reply	other threads:[~2024-03-27 19:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-27 19:07 [CI-test 1/8] drm: Add drm_vblank_work_flush_all() Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 2/8] drm/i915: Use vblank worker to unpin old legacy cursor fb safely Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 3/8] drm/i915: Use the same vblank worker for atomic unpin Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 4/8] drm/xe: Fix bo leak in intel_fb_bo_framebuffer_init Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 5/8] drm/xe/display: Preparations for preallocating dpt bo Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 6/8] drm/xe: Use simple xchg to cache DPT Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 7/8] drm/xe/display: Prevent overwriting original GGTT when taking over initial FB Maarten Lankhorst
2024-03-27 19:07 ` [CI-test 8/8] drm/xe/display: Re-use display vmas when possible Maarten Lankhorst
2024-03-27 19:25 ` ✓ CI.Patch_applied: success for series starting with [CI-test,1/8] drm: Add drm_vblank_work_flush_all() Patchwork
2024-03-27 19:25 ` Patchwork [this message]
2024-03-27 19:26 ` ✓ CI.KUnit: " Patchwork
2024-03-27 19:38 ` ✓ CI.Build: " Patchwork
2024-03-27 19:41 ` ✓ CI.Hooks: " Patchwork
2024-03-27 19:43 ` ✗ CI.checksparse: warning " Patchwork
2024-03-27 20:09 ` ✓ CI.BAT: success " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=171156754381.1060333.15872018127803336158@8e613ede5ea5 \
    --to=patchwork@emeril.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=maarten.lankhorst@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.