[radeon-alex:amd-staging-drm-next 3/5] drivers/gpu/drm/ttm/ttm_tt.c:240:6: sparse: symbol 'ttm_tt_init_fields' was not declared. Should it be static?
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: 085145ebf0e9c401e18cd9f7b2dd835a20c188c9 commit: 69665e4b79150133c3b28807f212972a1d3a392a [3/5] drm/ttm: add ttm_sg_tt_init reproduce: # apt-get install sparse git checkout 69665e4b79150133c3b28807f212972a1d3a392a make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
drivers/gpu/drm/ttm/ttm_tt.c:240:6: sparse: symbol 'ttm_tt_init_fields' was not declared. Should it be static?
Please review and possibly fold the followup patch.
--- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
Fixes: 69665e4b7915 ("drm/ttm: add ttm_sg_tt_init") Signed-off-by: Fengguang Wu fengguang.wu@intel.com --- ttm_tt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c index 9711331..45c8ae1 100644 --- a/drivers/gpu/drm/ttm/ttm_tt.c +++ b/drivers/gpu/drm/ttm/ttm_tt.c @@ -237,8 +237,8 @@ void ttm_tt_destroy(struct ttm_tt *ttm) ttm->func->destroy(ttm); }
-void ttm_tt_init_fields(struct ttm_tt *ttm, struct ttm_bo_device *bdev, - unsigned long size, uint32_t page_flags) +static void ttm_tt_init_fields(struct ttm_tt *ttm, struct ttm_bo_device *bdev, + unsigned long size, uint32_t page_flags) { ttm->bdev = bdev; ttm->num_pages = (size + PAGE_SIZE - 1) >> PAGE_SHIFT;
dri-devel@lists.freedesktop.org
-
kbuild test robot