[jbarnes:upstream-sync-points 2/2] drivers/gpu/drm/i915/i915_sync.c:43:12: sparse: symbol 'fence_lock' was not declared. Should it be static?
tree: git://people.freedesktop.org/~jbarnes/linux upstream-sync-points head: 97b242f63adaf11c3edf16b0282243cc4274eac8 commit: 97b242f63adaf11c3edf16b0282243cc4274eac8 [2/2] drm/i915: Android sync points for i915 v3 reproduce: # apt-get install sparse git checkout 97b242f63adaf11c3edf16b0282243cc4274eac8 make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
drivers/gpu/drm/i915/i915_sync.c:43:12: sparse: symbol 'fence_lock' was not declared. Should it be static?
Please review and possibly fold the followup patch.
--- 0-DAY kernel test infrastructure Open Source Technology Center http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
drivers/gpu/drm/i915/i915_sync.c:43:12: sparse: symbol 'fence_lock' was not declared. Should it be static?
Signed-off-by: Fengguang Wu fengguang.wu@intel.com --- i915_sync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_sync.c b/drivers/gpu/drm/i915/i915_sync.c index 069c069..b614eea 100644 --- a/drivers/gpu/drm/i915/i915_sync.c +++ b/drivers/gpu/drm/i915/i915_sync.c @@ -40,7 +40,7 @@ #include "../../../staging/android/sync.h"
/* Nothing really to protect here... */ -spinlock_t fence_lock; +static spinlock_t fence_lock;
/* * i915 fences on sync timelines
dri-devel@lists.freedesktop.org
-
kbuild test robot