== Series Details == Series: series starting with [1/7] drm/i915: Introduce i915_address_space.mutex URL : https://patchwork.freedesktop.org/series/46289/ State : warning == Summary == $ dim checkpatch origin/drm-tip bfb49745fd80 drm/i915: Introduce i915_address_space.mutex 0fc3f4498130 drm/i915: Move fence register tracking to GGTT 236a17479b0d drm/i915: Convert fences to use a GGTT lock rather than struct_mutex c2383b61bd9b drm/i915: Move fence-reg interface to i915_gem_fence_reg.h eb0335ad83e2 drm/i915: Dynamically allocate the array of drm_i915_gem_fence_reg 53e04e954008 drm/i915: Pull all the reset functionality together into i915_reset.c -:1070: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #1070: new file mode 100644 -:1075: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1075: FILE: drivers/gpu/drm/i915/i915_reset.c:1: +/* -:1216: WARNING:TYPO_SPELLING: 'acknowledgement' may be misspelled - perhaps 'acknowledgment'? #1216: FILE: drivers/gpu/drm/i915/i915_reset.c:142: + /* Assert reset for at least 20 usec, and wait for acknowledgement. */ -:1793: WARNING:MEMORY_BARRIER: memory barrier without comment #1793: FILE: drivers/gpu/drm/i915/i915_reset.c:719: + smp_mb__after_atomic(); -:2161: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'W' - possible side-effects? #2161: FILE: drivers/gpu/drm/i915/i915_reset.c:1087: +#define i915_wedge_on_timeout(W, DEV, TIMEOUT) \ + for (__init_wedge((W), (DEV), (TIMEOUT), __func__); \ + (W)->i915; \ + __fini_wedge((W))) -:2172: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const #2172: FILE: drivers/gpu/drm/i915/i915_reset.c:1098: + char *error_event[] = { I915_ERROR_UEVENT "=1", NULL }; -:2173: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const #2173: FILE: drivers/gpu/drm/i915/i915_reset.c:1099: + char *reset_event[] = { I915_RESET_UEVENT "=1", NULL }; -:2174: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const #2174: FILE: drivers/gpu/drm/i915/i915_reset.c:1100: + char *reset_done_event[] = { I915_ERROR_UEVENT "=0", NULL }; -:2352: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #2352: FILE: drivers/gpu/drm/i915/i915_reset.h:1: +/* total: 0 errors, 8 warnings, 1 checks, 2806 lines checked c3b127c4ee92 drm/i915: Remove GPU reset dependence on struct_mutex -:639: WARNING:MEMORY_BARRIER: memory barrier without comment #639: FILE: drivers/gpu/drm/i915/i915_reset.c:563: + smp_store_mb(i915->gpu_error.restart, NULL); total: 0 errors, 1 warnings, 0 checks, 1265 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
