✗ Fi.CI.CHECKPATCH: warning for Implicit dev_priv removal and GT compartmentalization (rev12)

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for Implicit dev_priv removal and GT compartmentalization (rev12)
Date: Wed, 19 Jun 2019 14:44:38 -0000	[thread overview]
Message-ID: <20190619144438.18133.9814@emeril.freedesktop.org> (raw)
In-Reply-To: <20190619132459.25351-1-tvrtko.ursulin@linux.intel.com>

== Series Details ==

Series: Implicit dev_priv removal and GT compartmentalization (rev12)
URL   : https://patchwork.freedesktop.org/series/62046/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
68ab239207f2 drm/i915: Convert intel_vgt_(de)balloon to uncore
53c6eabf50ab drm/i915: Introduce struct intel_gt as replacement for anonymous i915->gt
-:11: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#11: 
This is a first step towards cleaning up the separation between i915 and gt.

-:17: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#17: 
new file mode 100644

-:22: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#22: FILE: drivers/gpu/drm/i915/gt/intel_gt_types.h:1:
+/*

-:23: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#23: FILE: drivers/gpu/drm/i915/gt/intel_gt_types.h:2:
+ * SPDX-License-Identifier: MIT

total: 0 errors, 4 warnings, 0 checks, 99 lines checked
b51f9f2eb707 drm/i915: Move intel_gt initialization to a separate file
-:30: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#30: 
new file mode 100644

-:35: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#35: FILE: drivers/gpu/drm/i915/gt/intel_gt.c:1:
+/*

-:36: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#36: FILE: drivers/gpu/drm/i915/gt/intel_gt.c:2:
+ * SPDX-License-Identifier: MIT

-:56: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#56: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:1:
+/*

-:57: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#57: FILE: drivers/gpu/drm/i915/gt/intel_gt.h:2:
+ * SPDX-License-Identifier: MIT

total: 0 errors, 5 warnings, 0 checks, 85 lines checked
cd0aa8ce237f drm/i915: Store some backpointers in struct intel_gt
da6f11edebb0 drm/i915: Move intel_gt_pm_init under intel_gt_init_early
29c511511aa4 drm/i915: Make i915_check_and_clear_faults take intel_gt
83a012f8ceec drm/i915: Convert i915_gem_init_swizzling to intel_gt
37f5c1bc9946 drm/i915: Use intel_uncore_rmw in intel_gt_init_swizzling
f849957af495 drm/i915: Convert init_unused_rings to intel_gt
3f21de8ed081 drm/i915: Convert gt workarounds to intel_gt
95277e0c5efc drm/i915: Store backpointer to intel_gt in the engine
64db5ccf36a6 drm/i915: Convert intel_mocs_init_l3cc_table to intel_gt
4369360027dd drm/i915: Convert i915_ppgtt_init_hw to intel_gt
d2d4f1691dec drm/i915: Consolidate some open coded mmio rmw
ed6ca0266574 drm/i915: Convert i915_gem_init_hw to intel_gt
-:128: WARNING:AVOID_BUG: Avoid crashing the kernel - try using WARN_ON & recovery code rather than BUG() or BUG_ON()
#128: FILE: drivers/gpu/drm/i915/i915_gem.c:1311:
+	BUG_ON(!i915->kernel_context);

total: 0 errors, 1 warnings, 0 checks, 116 lines checked
beeacff41679 drm/i915: Move intel_engines_resume into common init
dc873d377a2a drm/i915: Stop using I915_READ/WRITE in intel_wopcm_init_hw
fd9eef1ec578 drm/i915: Compartmentalize i915_ggtt_probe_hw
a935bc49ad1b drm/i915: Compartmentalize i915_ggtt_init_hw
bb06d756637a drm/i915: Make ggtt invalidation work on ggtt
4cac147dda65 drm/i915: Store intel_gt backpointer in vm
6ba1c0ef131b drm/i915: Compartmentalize i915_gem_suspend/restore_gtt_mappings
180840ad1ef4 drm/i915: Convert i915_gem_flush_ggtt_writes to intel_gt
-:114: WARNING:MEMORY_BARRIER: memory barrier without comment
#114: FILE: drivers/gpu/drm/i915/gt/intel_gt.c:178:
+	wmb();

total: 0 errors, 1 warnings, 0 checks, 173 lines checked
2bbedf32690a drm/i915: Move i915_gem_chipset_flush to intel_gt
-:108: WARNING:MEMORY_BARRIER: memory barrier without comment
#108: FILE: drivers/gpu/drm/i915/gt/intel_gt.c:197:
+	wmb();

total: 0 errors, 1 warnings, 0 checks, 268 lines checked
e9e698f7f656 drm/i915: Compartmentalize timeline_init/park/fini
206f973106f8 drm/i915: Compartmentalize i915_ggtt_cleanup_hw
f1749b8874a7 drm/i915: Compartmentalize i915_gem_init_ggtt
325adf2a658b drm/i915: Store ggtt pointer in intel_gt
6b54fbe7e1cb drm/i915: Compartmentalize ring buffer creation
3b70f0b57adf drm/i915: Save trip via top-level i915 in a few more places
425fbddb3c75 drm/i915: Make timelines gt centric
b497df7f37e1 drm/i915: Rename i915_timeline to intel_timeline and move under gt
-:342: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#342: 
rename from drivers/gpu/drm/i915/i915_timeline.c

-:682: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#682: FILE: drivers/gpu/drm/i915/gt/intel_timeline.h:1:
+/*

total: 0 errors, 2 warnings, 0 checks, 1326 lines checked
c665162775c2 drm/i915: Eliminate dual personality of i915_scratch_offset

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