dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Jim Cromie <jim.cromie@gmail.com>
To: linux-kernel@vger.kernel.org, jbaron@akamai.com,
	gregkh@linuxfoundation.org, ukaszb@chromium.org
Cc: intel-gfx-trybot@lists.freedesktop.org,
	dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org,
	intel-gvt-dev@lists.freedesktop.org,
	intel-gfx@lists.freedesktop.org, daniel.vetter@ffwll.ch,
	tvrtko.ursulin@linux.intel.com, jani.nikula@intel.com,
	ville.syrjala@linux.intel.com, Jim Cromie <jim.cromie@gmail.com>,
	Andy Whitcroft <apw@canonical.com>, Joe Perches <joe@perches.com>,
	Dwaipayan Ray <dwaipayanray1@gmail.com>,
	Lukas Bulwahn <lukas.bulwahn@gmail.com>
Subject: [PATCH 15/63] checkpatch: add an exception to the do-while wrapper advice
Date: Fri, 24 Jan 2025 23:45:29 -0700	[thread overview]
Message-ID: <20250125064619.8305-16-jim.cromie@gmail.com> (raw)
In-Reply-To: <20250125064619.8305-1-jim.cromie@gmail.com>

Several new DYNDBG_CLASSMAP_* macros (added in following commits)
expand to multi-statement declarations, like module_param_named does.
They are invoked in file-scope, not in function scope, and cannot be
wrapped by a do-while, so add an exception by name for them.

cc: Andy Whitcroft <apw@canonical.com>
cc: Joe Perches <joe@perches.com>
cc: Dwaipayan Ray <dwaipayanray1@gmail.com>
cc: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
---
 scripts/checkpatch.pl | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 9eed3683ad76..4a93b2ede8cd 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -5963,6 +5963,7 @@ sub process {
 				MODULE_PARM_DESC|
 				DECLARE_PER_CPU|
 				DEFINE_PER_CPU|
+				DYNDBG_CLASSMAP|
 				__typeof__\(|
 				union|
 				struct|
-- 
2.48.1


  parent reply	other threads:[~2025-01-25  6:47 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-25  6:45 [PATCH 00/63] Fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y Jim Cromie
2025-01-25  6:45 ` [PATCH 01/63] docs/dyndbg: update examples \012 to \n Jim Cromie
2025-02-25 14:30   ` Louis Chauvet
2025-03-16 13:50     ` jim.cromie
2025-01-25  6:45 ` [PATCH 02/63] test-dyndbg: fixup CLASSMAP usage error Jim Cromie
2025-02-25 11:22   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 03/63] dyndbg: reword "class unknown," to "class:_UNKNOWN_" Jim Cromie
2025-02-25 11:22   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 04/63] dyndbg: make ddebug_class_param union members same size Jim Cromie
2025-02-25 11:24   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 05/63] dyndbg: replace classmap list with a vector Jim Cromie
2025-02-25 14:08   ` Louis Chauvet
2025-03-16 14:04     ` jim.cromie
2025-01-25  6:45 ` [PATCH 06/63] dyndbg: ddebug_apply_class_bitmap - add module arg, select on it Jim Cromie
2025-02-25 14:09   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 07/63] dyndbg: split param_set_dyndbg_classes to _module & wrapper fns Jim Cromie
2025-02-25 14:09   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 08/63] dyndbg: drop NUM_TYPE_ARRAY Jim Cromie
2025-02-25 14:09   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 09/63] dyndbg: reduce verbose/debug clutter Jim Cromie
2025-02-25 14:10   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 10/63] dyndbg: silence debugs with no-change updates Jim Cromie
2025-02-25 14:10   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 11/63] dyndbg: tighten ddebug_class_name() 1st arg type Jim Cromie
2025-02-25 14:11   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 12/63] dyndbg: tighten fn-sig of ddebug_apply_class_bitmap Jim Cromie
2025-02-25 14:12   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 13/63] dyndbg: reduce verbose=3 messages in ddebug_add_module Jim Cromie
2025-02-25 14:12   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 14/63] dyndbg-API: remove DD_CLASS_TYPE_(DISJOINT|LEVEL)_NAMES and code Jim Cromie
2025-01-25  6:45 ` Jim Cromie [this message]
2025-02-25 14:12   ` [PATCH 15/63] checkpatch: add an exception to the do-while wrapper advice Louis Chauvet
2025-01-25  6:45 ` [PATCH 16/63] dyndbg-API: replace DECLARE_DYNDBG_CLASSMAP Jim Cromie
2025-02-25 14:16   ` Louis Chauvet
2025-03-16 19:46     ` jim.cromie
2025-03-24 15:07       ` Louis Chauvet
2025-03-25 16:03         ` jim.cromie
2025-01-25  6:45 ` [PATCH 17/63] dyndbg: check DYNDBG_CLASSMAP_DEFINE args at compile-time Jim Cromie
2025-02-25 14:17   ` Louis Chauvet
2025-03-16 20:46     ` jim.cromie
2025-01-25  6:45 ` [PATCH 18/63] dyndbg: add/use for_subvec() to reduce boilerplate Jim Cromie
2025-02-25 14:18   ` Louis Chauvet
2025-03-16 20:50     ` jim.cromie
2025-01-25  6:45 ` [PATCH 19/63] dyndbg: make proper substructs in _ddebug_info Jim Cromie
2025-02-25 14:19   ` Louis Chauvet
2025-03-16 20:58     ` jim.cromie
2025-01-25  6:45 ` [PATCH 20/63] dyndbg: drop premature optimization in ddebug_add_module Jim Cromie
2025-02-25 14:26   ` Louis Chauvet
2025-01-25  6:45 ` [PATCH 21/63] dyndbg: allow ddebug_add_module to fail Jim Cromie
2025-02-25 14:26   ` Louis Chauvet
2025-03-16 21:04     ` jim.cromie
2025-01-25  6:45 ` [PATCH 22/63] dyndbg: rework ddebug_attach_*module_classes() Jim Cromie
2025-01-25  6:45 ` [PATCH 23/63] dyndbg: fail modprobe on ddebug_class_range_overlap() Jim Cromie
2025-02-25 14:27   ` Louis Chauvet
2025-03-16 21:07     ` jim.cromie
2025-01-25  6:45 ` [PATCH 24/63] dyndbg: hoist the range-overlap checks Jim Cromie
2025-01-25  6:45 ` [PATCH 25/63] ddebug: cleanup-range-overlap fails Jim Cromie
2025-01-25  6:45 ` [PATCH 26/63] dyndbg-test: change do_prints testpoint to accept a loopct Jim Cromie
2025-01-25  6:45 ` [PATCH 27/63] selftests-dyndbg: add tools/testing/selftests/dynamic_debug/* Jim Cromie
2025-01-25  6:45 ` [PATCH 28/63] dyndbg-API: promote DYNDBG_CLASSMAP_PARAM to API Jim Cromie
2025-02-25 14:29   ` Louis Chauvet
2025-03-16 21:14     ` jim.cromie
2025-03-24 22:58       ` jim.cromie
2025-01-25  6:45 ` [PATCH 29/63] dyndbg-doc: add classmap info to howto Jim Cromie
2025-01-25  6:45 ` [PATCH 30/63] dyndbg: treat comma as a token separator Jim Cromie
2025-01-25  6:45 ` [PATCH 31/63] selftests-dyndbg: add comma_terminator_tests Jim Cromie
2025-01-25  6:45 ` [PATCH 32/63] dyndbg: split multi-query strings with % Jim Cromie
2025-01-25  6:45 ` [PATCH 33/63] selftests-dyndbg: test_percent_splitting Jim Cromie
2025-01-25  6:45 ` [PATCH 34/63] docs/dyndbg: explain new delimiters: comma, percent Jim Cromie
2025-01-25  6:45 ` [PATCH 35/63] selftests-dyndbg: add test_mod_submod Jim Cromie
2025-01-25  6:45 ` [PATCH 36/63] docs/dyndbg: explain flags parse 1st Jim Cromie
2025-01-25  6:45 ` [PATCH 37/63] dyndbg: change __dynamic_func_call_cls* macros into expressions Jim Cromie
2025-01-25  6:45 ` [PATCH 38/63] dyndbg: drop "protection" of class'd pr_debugs from legacy queries Jim Cromie
2025-01-25  6:45 ` [PATCH 39/63] drm: use correct ccflags-y spelling Jim Cromie
2025-01-25  6:45 ` [PATCH 40/63] checkpatch: dont warn about unused macro arg on empty body Jim Cromie
2025-01-25  6:45 ` [PATCH 41/63] drm-dyndbg: adapt drm core to use dyndbg classmaps-v2 Jim Cromie
2025-01-25  6:45 ` [PATCH 42/63] drm-dyndbg: adapt DRM to invoke DYNDBG_CLASSMAP_PARAM Jim Cromie
2025-01-25  6:45 ` [PATCH 43/63] drm-print: fix config-dependent unused variable Jim Cromie
2025-01-25  6:45 ` [PATCH 44/63] drm-dyndbg: DRM_CLASSMAP_USE in amdgpu driver Jim Cromie
2025-01-25  6:45 ` [PATCH 45/63] drm-dyndbg: DRM_CLASSMAP_USE in i915 driver Jim Cromie
2025-01-25  6:46 ` [PATCH 46/63] drm-dyndbg: DRM_CLASSMAP_USE in drm_crtc_helper Jim Cromie
2025-01-25  6:46 ` [PATCH 47/63] drm-dyndbg: DRM_CLASSMAP_USE in drm_dp_helper Jim Cromie
2025-01-25  6:46 ` [PATCH 48/63] drm-dyndbg: DRM_CLASSMAP_USE in nouveau Jim Cromie
2025-01-25  6:46 ` [PATCH 49/63] drm-dyndbg: add DRM_CLASSMAP_USE to Xe driver Jim Cromie
2025-01-25  6:46 ` [PATCH 50/63] drm-dyndbg: add DRM_CLASSMAP_USE to virtio_gpu Jim Cromie
2025-01-25  6:46 ` [PATCH 51/63] drm-dyndbg: add DRM_CLASSMAP_USE to simpledrm Jim Cromie
2025-01-25  6:46 ` [PATCH 52/63] drm-dyndbg: add DRM_CLASSMAP_USE to bochs Jim Cromie
2025-01-25  6:46 ` [PATCH 53/63] drm-dyndbg: add DRM_CLASSMAP_USE to etnaviv Jim Cromie
2025-01-25  6:46 ` [PATCH 54/63] drm-dyndbg: add DRM_CLASSMAP_USE to gma500 driver Jim Cromie
2025-01-25  6:46 ` [PATCH 55/63] drm-dyndbg: add DRM_CLASSMAP_USE to radeon Jim Cromie
2025-01-25  6:46 ` [PATCH 56/63] drm-dyndbg: add DRM_CLASSMAP_USE to vmwgfx driver Jim Cromie
2025-01-25  6:46 ` [PATCH 57/63] drm-dyndbg: add DRM_CLASSMAP_USE to vkms driver Jim Cromie
2025-01-25  6:46 ` [PATCH 58/63] drm-dyndbg: add DRM_CLASSMAP_USE to udl driver Jim Cromie
2025-01-25  6:46 ` [PATCH 59/63] drm-dyndbg: add DRM_CLASSMAP_USE to mgag200 driver Jim Cromie
2025-01-25  6:46 ` [PATCH 60/63] drm-dyndbg: add DRM_CLASSMAP_USE to the gud driver Jim Cromie
2025-01-25  6:46 ` [PATCH 61/63] drm-dyndbg: add DRM_CLASSMAP_USE to the qxl driver Jim Cromie
2025-01-25  6:46 ` [PATCH 62/63] drm-dyndbg: add DRM_CLASSMAP_USE to the drm_gem_shmem_helper driver Jim Cromie
2025-01-25  6:46 ` [PATCH 63/63] drm: restore CONFIG_DRM_USE_DYNAMIC_DEBUG un-BROKEN Jim Cromie
2025-01-25  6:46 ` [PATCH 64/64] This series fixes dyndbg (dynamic debug) classmap support for DRM Jim Cromie
2025-02-20  8:31 ` [PATCH 00/63] Fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y Greg KH
2025-02-20  9:45   ` Simona Vetter
2025-02-28 16:24     ` Louis Chauvet
2025-03-12 16:26       ` jim.cromie

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=20250125064619.8305-16-jim.cromie@gmail.com \
    --to=jim.cromie@gmail.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=apw@canonical.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=dwaipayanray1@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=intel-gfx-trybot@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-gvt-dev@lists.freedesktop.org \
    --cc=jani.nikula@intel.com \
    --cc=jbaron@akamai.com \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lukas.bulwahn@gmail.com \
    --cc=tvrtko.ursulin@linux.intel.com \
    --cc=ukaszb@chromium.org \
    --cc=ville.syrjala@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).