✗ Fi.CI.CHECKPATCH: warning for drm/i915: Add support for asynchronous display power disabling (rev5)

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Imre Deak <imre.deak@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Add support for asynchronous display power disabling (rev5)
Date: Mon, 13 May 2019 19:43:21 -0000	[thread overview]
Message-ID: <20190513194321.3913.63246@emeril.freedesktop.org> (raw)
In-Reply-To: <20190509173446.31095-1-imre.deak@intel.com>

== Series Details ==

Series: drm/i915: Add support for asynchronous display power disabling (rev5)
URL   : https://patchwork.freedesktop.org/series/60242/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
7638893cffd3 drm/i915: Add support for tracking wakerefs w/o power-on guarantee
-:45: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'T' may be better as '(T)' to avoid precedence issues
#45: FILE: drivers/gpu/drm/i915/i915_utils.h:105:
+#define struct_member(T, member) (((T *)0)->member)

-:45: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'member' may be better as '(member)' to avoid precedence issues
#45: FILE: drivers/gpu/drm/i915/i915_utils.h:105:
+#define struct_member(T, member) (((T *)0)->member)

total: 0 errors, 0 warnings, 2 checks, 341 lines checked
7650efc33704 drm/i915: Force printing wakeref tacking during pm_cleanup
bc549fc35e2b drm/i915: Verify power domains state during suspend in all cases
a5cac9598b02 drm/i915: Add support for asynchronous display power disabling
-:626: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#626: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:85:
+}
+static inline void

total: 0 errors, 0 warnings, 1 checks, 519 lines checked
9e224e536182 drm/i915: Disable power asynchronously during DP AUX transfers
16c16ea57356 drm/i915: WARN for eDP encoders in intel_dp_detect_dpcd()
d0a3f6ba151b drm/i915: Remove the unneeded AUX power ref from intel_dp_detect()
-:176: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#176: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:94:
+#define with_intel_display_power(i915, domain, wf) \
+	for ((wf) = intel_display_power_get((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:176: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'domain' - possible side-effects?
#176: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:94:
+#define with_intel_display_power(i915, domain, wf) \
+	for ((wf) = intel_display_power_get((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:176: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects?
#176: FILE: drivers/gpu/drm/i915/intel_runtime_pm.h:94:
+#define with_intel_display_power(i915, domain, wf) \
+	for ((wf) = intel_display_power_get((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

total: 0 errors, 0 warnings, 3 checks, 119 lines checked
067c55ce5456 drm/i915: Remove the unneeded AUX power ref from intel_dp_hpd_pulse()
-:12: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#12: 
commit 1c767b339b39 ("drm/i915: take display port power domain in DP HPD handler")

total: 0 errors, 1 warnings, 0 checks, 46 lines checked
97e49c443919 drm/i915: Replace use of PLLS power domain with DISPLAY_CORE domain
30503f7a72e7 drm/i915: Avoid taking the PPS lock for non-eDP/VLV/CHV
fc767820cb42 drm/i915: Assert that TypeC ports are not used for eDP

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