== Series Details == Series: drm/i915: Add support for asynchronous display power disabling (rev4) URL : https://patchwork.freedesktop.org/series/60242/ State : warning == Summary == $ dim checkpatch origin/drm-tip cf4ea8e124c3 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 79f1a80345ce drm/i915: Force printing wakeref tacking during pm_cleanup 8fddd30bab27 drm/i915: Verify power domains state during suspend in all cases ee3d9b40ce98 drm/i915: Add support for asynchronous display power disabling b782fd807447 drm/i915: Disable power asynchronously during DP AUX transfers e050454a5a0a drm/i915: WARN for eDP encoders in intel_dp_detect_dpcd() 5acca0e6cc51 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:79: +#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:79: +#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:79: +#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 c7f8f0661059 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 5ed418d5c21c drm/i915: Replace use of PLLS power domain with DISPLAY_CORE domain 8be8c1a2a3ec drm/i915: Avoid taking the PPS lock for non-eDP/VLV/CHV 00c05bcbe976 drm/i915: Assert that TypeC ports are not used for eDP _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx