commit 8eb246d7f6137c6d1783c187d37e415e61211d2d
author Vinson Lee <vlee@freedesktop.org> Wed Oct 14 16:25:31 2020 -0700
committer Vinson Lee <vlee@freedesktop.org> Fri Oct 16 15:25:37 2020 -0700
tree cb87c8aa702b23c4df7e914918e796623fae1355
parent 5e27e0432274a29e75f9267269a2b367eda33cbf

v3dv: Remove unsigned comparison to zero.

index is of type uint32_t.

Fix defect reported by Coverity Scan.

Macro compares unsigned to 0 (NO_EFFECT)
unsigned_compare: This greater-than-or-equal-to-zero comparison of
an unsigned value is always true. index >= 0U.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7153>

src/broadcom/vulkan/v3dv_descriptor_set.c

diff --git a/src/broadcom/vulkan/v3dv_descriptor_set.c b/src/broadcom/vulkan/v3dv_descriptor_set.c
index 41754bd..89657ce 100644
--- a/src/broadcom/vulkan/v3dv_descriptor_set.c
+++ b/src/broadcom/vulkan/v3dv_descriptor_set.c
@@ -88,7 +88,7 @@
                                    uint32_t index,
                                    uint32_t *dynamic_offset)
 {
-   assert(index >= 0 && index < map->num_desc);
+   assert(index < map->num_desc);
 
    uint32_t set_number = map->set[index];
    assert((descriptor_state->valid & 1 << set_number));