panfrost: Pay attention to framebuffer dimension sign
authorAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Wed, 21 Aug 2019 16:21:19 +0000 (09:21 -0700)
committerAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Wed, 21 Aug 2019 17:38:31 +0000 (10:38 -0700)
These are unsigned so the clamp-positive is redundant.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
src/gallium/drivers/panfrost/pan_context.c

index 63929890c02bdfbcccbd3920105237bfb0f87b2e..6c26e7a8a450ca61f4aa78b48cb45235376f07aa 100644 (file)
@@ -1242,24 +1242,17 @@ panfrost_emit_for_draw(struct panfrost_context *ctx, bool with_vertex_data)
          * handle the negatives if we don't */
 
         if (miny > maxy) {
-                int temp = miny;
+                unsigned temp = miny;
                 miny = maxy;
                 maxy = temp;
         }
 
         if (minx > maxx) {
-                int temp = minx;
+                unsigned temp = minx;
                 minx = maxx;
                 maxx = temp;
         }
 
-        /* Clamp everything positive, just in case */
-
-        maxx = MAX2(0, maxx);
-        maxy = MAX2(0, maxy);
-        minx = MAX2(0, minx);
-        miny = MAX2(0, miny);
-
         /* Clamp to the framebuffer size as a last check */
 
         minx = MIN2(ctx->pipe_framebuffer.width, minx);