Tom Rix
2023-Mar-03 13:27 UTC
[Nouveau] [PATCH] drm/nouveau/fifo: set gf100_fifo_nonstall_block_dump storage-class-specifier to static
gcc with W=1 reports drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c:451:1: error: no previous prototype for ?gf100_fifo_nonstall_block? [-Werror=missing-prototypes] 451 | gf100_fifo_nonstall_block(struct nvkm_event *event, int type, int index) | ^~~~~~~~~~~~~~~~~~~~~~~~~ gf100_fifo_nonstall_block is only used in gf100.c, so it should be static Signed-off-by: Tom Rix <trix at redhat.com> --- drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c index 5bb65258c36d..6c94451d0faa 100644 --- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c +++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c @@ -447,7 +447,7 @@ gf100_fifo_nonstall_allow(struct nvkm_event *event, int type, int index) spin_unlock_irqrestore(&fifo->lock, flags); } -void +static void gf100_fifo_nonstall_block(struct nvkm_event *event, int type, int index) { struct nvkm_fifo *fifo = container_of(event, typeof(*fifo), nonstall.event); -- 2.27.0
Karol Herbst
2023-Mar-03 14:28 UTC
[Nouveau] [PATCH] drm/nouveau/fifo: set gf100_fifo_nonstall_block_dump storage-class-specifier to static
Reviewed-by: Karol Herbst <kherbst at redhat.com> will push in a moment On Fri, Mar 3, 2023 at 2:27 PM Tom Rix <trix at redhat.com> wrote:> > gcc with W=1 reports > drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c:451:1: error: > no previous prototype for ?gf100_fifo_nonstall_block? [-Werror=missing-prototypes] > 451 | gf100_fifo_nonstall_block(struct nvkm_event *event, int type, int index) > | ^~~~~~~~~~~~~~~~~~~~~~~~~ > > gf100_fifo_nonstall_block is only used in gf100.c, so it should be static > > Signed-off-by: Tom Rix <trix at redhat.com> > --- > drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c > index 5bb65258c36d..6c94451d0faa 100644 > --- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c > +++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c > @@ -447,7 +447,7 @@ gf100_fifo_nonstall_allow(struct nvkm_event *event, int type, int index) > spin_unlock_irqrestore(&fifo->lock, flags); > } > > -void > +static void > gf100_fifo_nonstall_block(struct nvkm_event *event, int type, int index) > { > struct nvkm_fifo *fifo = container_of(event, typeof(*fifo), nonstall.event); > -- > 2.27.0 >
Seemingly Similar Threads
- [PATCH 1/3] nouveau: Do not BUG_ON(!spin_is_locked()) on UP
- [PATCH] drm/nouveau/fifo: set nvkm_engn_cgrp_get storage-class-specifier to static
- [PATCH] drm/nouveau/nvfw/acr: set wpr_generic_header_dump storage-class-specifier to static
- [PATCH] drm/nouveau/acr/ga102: set variable ga102_gsps storage-class-specifier to static
- [RFC] Documentation: nouveau: Introduce some nouveau documentation