nouveau: remove always false argument in nouveau_fence_new()
authorEmil Velikov <emil.l.velikov@gmail.com>
Mon, 16 Jan 2017 16:25:19 +0000 (16:25 +0000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Wed, 18 Jan 2017 16:01:15 +0000 (16:01 +0000)
No point in having the extra argument considering that it's effectively
unused since the function was introduced.

Cc: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/gallium/drivers/nouveau/nouveau_fence.c
src/gallium/drivers/nouveau/nouveau_fence.h
src/gallium/drivers/nouveau/nv30/nv30_screen.c
src/gallium/drivers/nouveau/nv50/nv50_screen.c
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c

index 691553ae7e4ffcf22a186a1000c36c5b54d9488f..6c299cbc6a5a6c5913ddbdf4c7e32eb8c79fb647 100644 (file)
@@ -30,8 +30,7 @@
 #endif
 
 bool
-nouveau_fence_new(struct nouveau_screen *screen, struct nouveau_fence **fence,
-                  bool emit)
+nouveau_fence_new(struct nouveau_screen *screen, struct nouveau_fence **fence)
 {
    *fence = CALLOC_STRUCT(nouveau_fence);
    if (!*fence)
@@ -41,9 +40,6 @@ nouveau_fence_new(struct nouveau_screen *screen, struct nouveau_fence **fence,
    (*fence)->ref = 1;
    LIST_INITHEAD(&(*fence)->work);
 
-   if (emit)
-      nouveau_fence_emit(*fence);
-
    return true;
 }
 
@@ -242,7 +238,7 @@ nouveau_fence_next(struct nouveau_screen *screen)
 
    nouveau_fence_ref(NULL, &screen->fence.current);
 
-   nouveau_fence_new(screen, &screen->fence.current, false);
+   nouveau_fence_new(screen, &screen->fence.current);
 }
 
 void
index f10016da826559921556533d00f0ef9252164e2c..e14572bce8f972f406b57148e85a09146543006e 100644 (file)
@@ -32,8 +32,7 @@ struct nouveau_fence {
 void nouveau_fence_emit(struct nouveau_fence *);
 void nouveau_fence_del(struct nouveau_fence *);
 
-bool nouveau_fence_new(struct nouveau_screen *, struct nouveau_fence **,
-                       bool emit);
+bool nouveau_fence_new(struct nouveau_screen *, struct nouveau_fence **);
 bool nouveau_fence_work(struct nouveau_fence *, void (*)(void *), void *);
 void nouveau_fence_update(struct nouveau_screen *, bool flushed);
 void nouveau_fence_next(struct nouveau_screen *);
index 7dfc0fce73d2deac6be7c04283fea9ad944ee8be..56bde4185b52492e0d9d1556d356367669193e60 100644 (file)
@@ -735,6 +735,6 @@ nv30_screen_create(struct nouveau_device *dev)
 
    nouveau_pushbuf_kick(push, push->channel);
 
-   nouveau_fence_new(&screen->base, &screen->base.fence.current, false);
+   nouveau_fence_new(&screen->base, &screen->base.fence.current);
    return &screen->base;
 }
index f766aadb06c09301aade862e48a6e7b0ff53b460..fcff85d9ed14e4ea19d82372254e892f3e6f6da9 100644 (file)
@@ -1017,7 +1017,7 @@ nv50_screen_create(struct nouveau_device *dev)
       goto fail;
    }
 
-   nouveau_fence_new(&screen->base, &screen->base.fence.current, false);
+   nouveau_fence_new(&screen->base, &screen->base.fence.current);
 
    return &screen->base;
 
index 1ef481b4e4d6da2f697c13beceab4da5c0fcc022..ffbe733060cf3d6eca62ae538aa75f4e37d81548 100644 (file)
@@ -1233,7 +1233,7 @@ nvc0_screen_create(struct nouveau_device *dev)
    screen->default_tsc = CALLOC_STRUCT(nv50_tsc_entry);
    screen->default_tsc->tsc[0] = G80_TSC_0_SRGB_CONVERSION;
 
-   nouveau_fence_new(&screen->base, &screen->base.fence.current, false);
+   nouveau_fence_new(&screen->base, &screen->base.fence.current);
 
    return &screen->base;