OSDN Git Service

etnaviv: remove number of pixel pipes validation
authorChristian Gmeiner <christian.gmeiner@gmail.com>
Wed, 8 Feb 2017 12:14:05 +0000 (13:14 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Wed, 1 Mar 2017 13:34:41 +0000 (13:34 +0000)
This validation was added before the etnaviv drm driver landed in
the linux kernel. Due some pre-merge API changes we had to fix-up
this value but with a mainline kernel this is not a problem anymore.

Lets remove that validation which also gets rid of problem caught
by Coverity, reported to me by imirkin.

Cc: "17.0" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
(cherry picked from commit e8d600710cf7c1a646a36947ee7752c89860bf11)

src/gallium/drivers/etnaviv/etnaviv_screen.c

index 918b78d..28db9b9 100644 (file)
@@ -576,16 +576,6 @@ etna_get_specs(struct etna_screen *screen)
       DBG("could not get ETNA_GPU_PIXEL_PIPES");
       goto fail;
    }
-   if (val < 1 && val > ETNA_MAX_PIXELPIPES) {
-      if (val == 0) {
-         fprintf(stderr, "Warning: zero pixel pipes (update kernel?)\n");
-         val = 1;
-      } else {
-         fprintf(stderr, "Error: bad pixel pipes value %u\n",
-                 (unsigned int)val);
-         goto fail;
-      }
-   }
    screen->specs.pixel_pipes = val;
 
    if (etna_gpu_get_param(screen->gpu, ETNA_GPU_NUM_CONSTANTS, &val)) {