[Nouveau] [PATCH] drm/nouveau/hwmon: fix use of integer as a pointer

2019-12-27 Thread Ben Dooks
The special_groups[] in nouveau_hwmon_init() is an array of pointers, so use NULL instead of '0' as the initialiser. Fixes the following sparse warning: drivers/gpu/drm/nouveau/nouveau_hwmon.c:744:29: warning: Using plain integer as NULL pointer Signed-off-by: Ben Dooks --- Cc: Ben Skeggs Cc

[Nouveau] [PATCH] drm/nouveau/core/client: make nvkm_uclient_sclass static

2019-12-27 Thread Ben Dooks
The nvkm_uclient_sclass object is not used outside the file it is defined in, so make it static to avoid the following sparse warning: drivers/gpu/drm/nouveau/nvkm/core/client.c:64:1: warning: symbol 'nvkm_uclient_sclass' was not declared. Should it be static? Signed-off-by: Ben Dooks --- Cc

[Nouveau] [PATCH] drm/nouveau/disp/gv100: make gv100_disp_core_mthd_{base, sor} static

2019-12-27 Thread Ben Dooks
? drivers/gpu/drm/nouveau/nvkm/engine/disp/coregv100.c:43:1: warning: symbol 'gv100_disp_core_mthd_sor' was not declared. Should it be static? Signed-off-by: Ben Dooks --- Bad divisor in main::vcs_assign: 0 Cc: Ben Skeggs Cc: David Airlie Cc: Daniel Vetter Cc: dri-de...@lists.freedesktop.org Cc

[Nouveau] [PATCH 2/5] drm/nouveau/fifo/tu102: make tu102_fifo_runlist static

2019-12-27 Thread Ben Dooks
The tu102_fifo_runlist is not exorted, so make it static to avoid the following warning: drivers/gpu/drm/nouveau/nvkm/engine/fifo/tu102.c:47:1: warning: symbol 'tu102_fifo_runlist' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/engine/fifo

[Nouveau] [PATCH 3/5] drm/nouveau/fifo/gv100: make gv100_fifo_runlist static

2019-12-27 Thread Ben Dooks
Do not export gv100_fifo_runlist as it isn't not used to remove the following morning: drivers/gpu/drm/nouveau/nvkm/engine/fifo/gv100.c:56:1: warning: symbol 'gv100_fifo_runlist' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/engine/fifo/gv100

[Nouveau] [PATCH 2/3] drm/nouveau/kms/nv50-: make unexported items static

2019-12-27 Thread Ben Dooks
: symbol 'headc57d_olut_set' was not declared. Should it be static? drivers/gpu/drm/nouveau/dispnv50/headc57d.c:155:1: warning: symbol 'headc57d_olut' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/dispnv50/headc57d.c | 6 +++--- 1 file changed, 3

[Nouveau] [PATCH 5/5] drm/nouveau/disp/gv100: make gv100_disp_wimm static

2019-12-27 Thread Ben Dooks
The gv100_disp_wimm is not declared, so make it static to avoid the following warning: drivers/gpu/drm/nouveau/nvkm/engine/disp/wimmgv100.c:39:1: warning: symbol 'gv100_disp_wimm' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/engine/disp

[Nouveau] [PATCH 1/3] drm/nouveau/kms/nv50-: make base917c_format static

2019-12-27 Thread Ben Dooks
The base917c_format isn't exported, so make it static to avoid the following warning: drivers/gpu/drm/nouveau/dispnv50/base917c.c:26:1: warning: symbol 'base917c_format' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/dispnv50/base917c.c | 2 +- 1

[Nouveau] [PATCH] drm/nouveau/falcon: make unexported objects static

2019-12-27 Thread Ben Dooks
/msgqueue_0148cdec.c:241:1: warning: symbol 'msgqueue_0148cdec_func' was not declared. Should it be static? Signed-off-by: Ben Dooks --- Cc: Ben Skeggs Cc: David Airlie Cc: Daniel Vetter Cc: dri-de...@lists.freedesktop.org Cc: nouveau@lists.freedesktop.org Cc: linux-ker...@vger.kernel.org

[Nouveau] [PATCH 3/3] drm/nouveau/kms/nv50-: include n50_display.h for nv50_display_create

2019-12-27 Thread Ben Dooks
it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 2 ++ drivers/gpu/drm/nouveau/nv50_display.h | 3 --- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c index b46be8a091e9

[Nouveau] [PATCH 4/5] drm/nouveau/disp/gv100: make undeclared symbols static

2019-12-27 Thread Ben Dooks
? drivers/gpu/drm/nouveau/nvkm/engine/disp/wndwgv100.c:140:1: warning: symbol 'gv100_disp_wndw' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/engine/disp/wndwgv100.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu

[Nouveau] [PATCH 1/5] drm/nouveau/gr/gf100-: make undeclared symbols static

2019-12-27 Thread Ben Dooks
/gf100.c:760:1: warning: symbol 'gf100_gr_fecs_stop_ctxsw' was not declared. Should it be static? drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c:2071:1: warning: symbol 'gf100_gr_ctor_fw_legacy' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm

[Nouveau] [PATCH] drm/nouveau/acr: make wpr_generic_header_dump static

2023-05-04 Thread Ben Dooks
Make wpr_generic_header_dump static to avoid the following sparse warning: drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c:49:1: warning: symbol 'wpr_generic_header_dump' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c | 2 +- 1 file changed

[Nouveau] [PATCH] drm/nouveau/acr/ga102: make ga102_gsps static (not exported)

2023-05-04 Thread Ben Dooks
The ga102_gsps is not exported, so make it static to avoid the following sparse warning: drivers/gpu/drm/nouveau/nvkm/subdev/gsp/ga102.c:49:1: warning: symbol 'ga102_gsps' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/subdev/gsp/ga102.c | 2

[Nouveau] [PATCH] drm/nouveau/mc/ga100: make ga100_mc_device static

2023-05-04 Thread Ben Dooks
Make ga100_mc_device static as it isn't exported, to fix the following sparse warning: drivers/gpu/drm/nouveau/nvkm/subdev/mc/ga100.c:51:1: warning: symbol 'ga100_mc_device' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm/subdev/mc/ga100.c | 2

[Nouveau] [PATCH] drm/nouveau/fifo: make gf100_fifo_nonstall_block static

2023-05-04 Thread Ben Dooks
Make gf100_fifo_nonstall_block as it isn't exported, to silence the following sparse warning: drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c:451:1: warning: symbol 'gf100_fifo_nonstall_block' was not declared. Should it be static? Signed-off-by: Ben Dooks --- drivers/gpu/drm/nouveau/nvkm