[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/7] drm/i915: Drop one more usless master_transcoder assignment

Patchwork patchwork at emeril.freedesktop.org
Mon Oct 19 21:56:02 UTC 2020


== Series Details ==

Series: series starting with [1/7] drm/i915: Drop one more usless master_transcoder assignment
URL   : https://patchwork.freedesktop.org/series/82848/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
5e9c0a3f80bf drm/i915: Drop one more usless master_transcoder assignment
40e3f078c22a drm/i915: Introduce with_intel_display_power_if_enabled()
-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:330:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'domain' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:330:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/display/intel_display_power.h:330:
+#define with_intel_display_power_if_enabled(i915, domain, wf) \
+	for ((wf) = intel_display_power_get_if_enabled((i915), (domain)); (wf); \
+	     intel_display_power_put_async((i915), (domain), (wf)), (wf) = 0)

total: 0 errors, 0 warnings, 3 checks, 10 lines checked
ba8826254488 drm/i915: Adjust intel_dsc_power_domain() calling convention
-:279: WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#279: FILE: drivers/gpu/drm/i915/display/intel_vdsc.c:1133:
+					  intel_dsc_power_domain(crtc, old_crtc_state->cpu_transcoder));

total: 0 errors, 1 warnings, 0 checks, 245 lines checked
23aaa8770260 drm/i915: Extract hsw_panel_transcoders()
0de1b95e83f6 drm/i915: Pimp HSW+ transcoder state readout
-:135: CHECK:SPACING: No space is necessary after a cast
#135: FILE: drivers/gpu/drm/i915/display/intel_display.c:11111:
+	cpu_transcoder = (enum transcoder) crtc->pipe;

total: 0 errors, 0 warnings, 1 checks, 187 lines checked
d1ffd8987bbb drm/i915: cpu_transcoder readout for bigjoiner
-:51: CHECK:SPACING: No space is necessary after a cast
#51: FILE: drivers/gpu/drm/i915/display/intel_display.c:11073:
+		power_domain = intel_dsc_power_domain(crtc, (enum transcoder) pipe);

-:87: CHECK:SPACING: No space is necessary after a cast
#87: FILE: drivers/gpu/drm/i915/display/intel_display.c:11163:
+		cpu_transcoder = (enum transcoder) crtc->pipe - 1;

total: 0 errors, 0 warnings, 2 checks, 75 lines checked
d1051eed84f3 drm/i915: FIXMEs for bigjoiner readout
-:10: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:38: WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#38: FILE: drivers/gpu/drm/i915/display/intel_display.c:5859:
+#if 0

total: 0 errors, 2 warnings, 0 checks, 45 lines checked




More information about the Intel-gfx mailing list