Mesa (master): etnaviv: etnaviv_fence: Simplify the return code logic

Christian Gmeiner austriancoder at kemper.freedesktop.org
Sat Apr 22 15:49:45 UTC 2017


Module: Mesa
Branch: master
Commit: 53e39f6df47fcfc19f36e425c0f880a9b1a2a7d6
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=53e39f6df47fcfc19f36e425c0f880a9b1a2a7d6

Author: Fabio Estevam <festevam at gmail.com>
Date:   Mon Apr 17 19:36:40 2017 -0300

etnaviv: etnaviv_fence: Simplify the return code logic

The return code can be simplified by using the logical not operator.

Signed-off-by: Fabio Estevam <festevam at gmail.com>
Reviewed-by: Christian Gmeiner <christian.gmeiner at gmail.com>

---

 src/gallium/drivers/etnaviv/etnaviv_fence.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/gallium/drivers/etnaviv/etnaviv_fence.c b/src/gallium/drivers/etnaviv/etnaviv_fence.c
index 65402aaa3b..d82708eacb 100644
--- a/src/gallium/drivers/etnaviv/etnaviv_fence.c
+++ b/src/gallium/drivers/etnaviv/etnaviv_fence.c
@@ -65,10 +65,8 @@ static boolean
 etna_screen_fence_finish(struct pipe_screen *pscreen, struct pipe_context *ctx,
                          struct pipe_fence_handle *fence, uint64_t timeout)
 {
-   if (fence->fence_fd != -1) {
-      int ret = sync_wait(fence->fence_fd, timeout / 1000000);
-      return ret == 0;
-   }
+   if (fence->fence_fd != -1)
+	return !sync_wait(fence->fence_fd, timeout / 1000000);
 
    if (etna_pipe_wait_ns(fence->screen->pipe, fence->timestamp, timeout))
       return false;




More information about the mesa-commit mailing list