public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Patchwork <patchwork@emeril.freedesktop.org>
To: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for RPM Encapsulation (rev2)
Date: Thu, 13 Jun 2019 23:37:51 -0000	[thread overview]
Message-ID: <20190613233751.17469.20207@emeril.freedesktop.org> (raw)
In-Reply-To: <20190613232156.34940-1-daniele.ceraolospurio@intel.com>

== Series Details ==

Series: RPM Encapsulation (rev2)
URL   : https://patchwork.freedesktop.org/series/61976/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
997c1d60a119 drm/i915: prefer i915_runtime_pm in intel_runtime function
4017cb4a3083 drm/i915: Remove rpm asserts that use i915
d63294884cad drm/i915: make enable/disable rpm assert function use the rpm structure
768ac1cf35a5 drm/i915: move and rename i915_runtime_pm
-:455: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment
#455: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:65:
+		spinlock_t lock;

total: 0 errors, 0 warnings, 1 checks, 519 lines checked
745e10797905 drm/i915: move a few more functions to accept the rpm structure
5a54e53a457b drm/i915: update rpm_get/put to use the rpm structure
319d7944d790 drm/i915: update with_intel_runtime_pm to use the rpm structure
-:445: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'rpm' - possible side-effects?
#445: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:182:
+#define with_intel_runtime_pm(rpm, wf) \
+	for ((wf) = intel_runtime_pm_get(rpm); (wf); \
+	     intel_runtime_pm_put((rpm), (wf)), (wf) = 0)

-:445: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects?
#445: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:182:
+#define with_intel_runtime_pm(rpm, wf) \
+	for ((wf) = intel_runtime_pm_get(rpm); (wf); \
+	     intel_runtime_pm_put((rpm), (wf)), (wf) = 0)

-:452: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'rpm' - possible side-effects?
#452: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:186:
+#define with_intel_runtime_pm_if_in_use(rpm, wf) \
+	for ((wf) = intel_runtime_pm_get_if_in_use(rpm); (wf); \
+	     intel_runtime_pm_put((rpm), (wf)), (wf) = 0)

-:452: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects?
#452: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:186:
+#define with_intel_runtime_pm_if_in_use(rpm, wf) \
+	for ((wf) = intel_runtime_pm_get_if_in_use(rpm); (wf); \
+	     intel_runtime_pm_put((rpm), (wf)), (wf) = 0)

total: 0 errors, 0 warnings, 4 checks, 396 lines checked
8882c98e415d drm/i915: make intel_wakeref work on the rpm struct

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  parent reply	other threads:[~2019-06-13 23:37 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-13 23:21 [PATCH v3 0/8] RPM encapsulation Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 1/8] drm/i915: prefer i915_runtime_pm in intel_runtime function Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 2/8] drm/i915: Remove rpm asserts that use i915 Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 3/8] drm/i915: make enable/disable rpm assert function use the rpm structure Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 4/8] drm/i915: move and rename i915_runtime_pm Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 5/8] drm/i915: move a few more functions to accept the rpm structure Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 6/8] drm/i915: update rpm_get/put to use " Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 7/8] drm/i915: update with_intel_runtime_pm " Daniele Ceraolo Spurio
2019-06-13 23:21 ` [PATCH v3 8/8] drm/i915: make intel_wakeref work on the rpm struct Daniele Ceraolo Spurio
2019-06-13 23:37 ` Patchwork [this message]
2019-06-13 23:43 ` ✗ Fi.CI.SPARSE: warning for RPM Encapsulation (rev2) Patchwork
2019-06-14 13:21 ` [PATCH v3 0/8] RPM encapsulation Imre Deak
2019-06-14 14:59   ` Chris Wilson
2019-06-14 13:36 ` ✓ Fi.CI.BAT: success for RPM Encapsulation (rev2) Patchwork
2019-06-15 10:04 ` ✓ Fi.CI.IGT: " 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=20190613233751.17469.20207@emeril.freedesktop.org \
    --to=patchwork@emeril.freedesktop.org \
    --cc=daniele.ceraolospurio@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /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