broadcom/vc5: Fix BASE_LEVEL handling with txl.
authorEric Anholt <eric@anholt.net>
Wed, 22 Nov 2017 00:32:33 +0000 (16:32 -0800)
committerEric Anholt <eric@anholt.net>
Wed, 22 Nov 2017 18:56:31 +0000 (10:56 -0800)
The HW doesn't add the base level anywhere (the min/max lod clamping is
what does base level), so we need to add it manually in this case.

Fixes piglit tex-miplevel-selection *Lod 2D.

src/broadcom/compiler/nir_to_vir.c
src/gallium/drivers/vc5/vc5_uniforms.c

index 879704a..7a31e24 100644 (file)
@@ -380,9 +380,11 @@ ntq_emit_tex(struct v3d_compile *c, nir_tex_instr *instr)
                         p0_unpacked.bias_supplied = true;
                         break;
                 case nir_tex_src_lod:
-                        /* XXX: Needs base level addition */
                         coords[next_coord++] =
-                                ntq_get_src(c, instr->src[i].src, 0);
+                                vir_FADD(c,
+                                         ntq_get_src(c, instr->src[i].src, 0),
+                                         vir_uniform(c, QUNIFORM_TEXTURE_FIRST_LEVEL,
+                                                     unit));
 
                         if (instr->op != nir_texop_txf &&
                             instr->op != nir_texop_tg4) {
index 37354b8..5e18134 100644 (file)
@@ -324,6 +324,10 @@ vc5_write_uniforms(struct vc5_context *vc5, struct vc5_compiled_shader *shader,
                         break;
 
                 case QUNIFORM_TEXTURE_FIRST_LEVEL:
+                        cl_aligned_f(&uniforms,
+                                     texstate->textures[uinfo->data[i]]->u.tex.first_level);
+                        break;
+
                 case QUNIFORM_TEXTURE_BORDER_COLOR:
                         /* XXX */
                         break;