etnaviv: fix nearest_linear / linear_nearest filtering on GC3000
authorJonathan Marek <jonathan@marek.ca>
Wed, 19 Jun 2019 15:43:54 +0000 (11:43 -0400)
committerJonathan Marek <jonathan@marek.ca>
Sun, 14 Jul 2019 14:34:17 +0000 (10:34 -0400)
The MIN filter is never used when not using mipmaps. This fixes that.

Interestingly, only GC3000 needs this (GC2000 works without this fix).

Signed-off-by: Jonathan Marek <jonathan@marek.ca>
Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
src/gallium/drivers/etnaviv/etnaviv_texture_state.c
src/gallium/drivers/etnaviv/etnaviv_texture_state.h

index ffb3ffdcc846473965d728e8cf2830132caea849..c3c68740f971629c10ab2fc7fe25fdf3bdd4f8e4 100644 (file)
@@ -77,6 +77,12 @@ etna_create_sampler_state_state(struct pipe_context *pipe,
       cs->min_lod = cs->max_lod = etna_float_to_fixp55(ss->min_lod);
    }
 
+   /* if max_lod is 0, MIN filter will never be used (GC3000)
+    * when min filter is different from mag filter, we need HW to compute LOD
+    * the workaround is to set max_lod to at least 1
+    */
+   cs->max_lod_min = (ss->min_img_filter != ss->mag_img_filter) ? 1 : 0;
+
    return cs;
 }
 
@@ -284,10 +290,12 @@ etna_emit_texture_state(struct etna_context *ctx)
             ss = etna_sampler_state(ctx->sampler[x]);
             sv = etna_sampler_view(ctx->sampler_view[x]);
 
+            unsigned max_lod = MAX2(MIN2(ss->max_lod, sv->max_lod), ss->max_lod_min);
+
             /* min and max lod is determined both by the sampler and the view */
             /*020C0*/ EMIT_STATE(TE_SAMPLER_LOD_CONFIG(x),
                                  ss->TE_SAMPLER_LOD_CONFIG |
-                                 VIVS_TE_SAMPLER_LOD_CONFIG_MAX(MIN2(ss->max_lod, sv->max_lod)) |
+                                 VIVS_TE_SAMPLER_LOD_CONFIG_MAX(max_lod) |
                                  VIVS_TE_SAMPLER_LOD_CONFIG_MIN(MAX2(ss->min_lod, sv->min_lod)));
          }
       }
index 87841477be9a25d4a732ac83888efd5aa0e74a54..a426d4b6c1e19997666228df53fc8ec88e11e098 100644 (file)
@@ -43,7 +43,7 @@ struct etna_sampler_state {
    uint32_t TE_SAMPLER_CONFIG0;
    uint32_t TE_SAMPLER_CONFIG1;
    uint32_t TE_SAMPLER_LOD_CONFIG;
-   unsigned min_lod, max_lod;
+   unsigned min_lod, max_lod, max_lod_min;
 };
 
 static inline struct etna_sampler_state *