nouveau/mme: fix print inst for case MME_FERMI_OP_MERGE
authorMauro Rossi <issor.oruam@gmail.com>
Sat, 14 Jan 2023 07:18:32 +0000 (08:18 +0100)
committerMarge Bot <emma+marge@anholt.net>
Fri, 4 Aug 2023 21:32:04 +0000 (21:32 +0000)
By applying the same coding style of the other case statements,
this patch fixes the following building errors:

../src/nouveau/mme/mme_fermi.c:209:10: error: expected expression
         uint32_t src_bit  = inst->bitfield.src_bit;
         ^
../src/nouveau/mme/mme_fermi.c:216:39: error: use of undeclared identifier 'src_bit'; did you mean 'dst_bit'?
         fprintf(fp, " (%u, %u, %u)", src_bit, size, dst_bit);
                                      ^~~~~~~
                                      dst_bit
../src/nouveau/mme/mme_fermi.c:211:19: note: 'dst_bit' declared here
         uint32_t dst_bit  = inst->bitfield.dst_bit;
                  ^
2 errors generated.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24326>

src/nouveau/mme/mme_fermi.c

index 32a22af..eedb5d1 100644 (file)
@@ -206,14 +206,12 @@ mme_fermi_print_inst(FILE *fp, unsigned indent,
          print_imm(fp, inst);
          break;
       case MME_FERMI_OP_MERGE:
-         uint32_t src_bit  = inst->bitfield.src_bit;
-         uint32_t size     = inst->bitfield.size;
-         uint32_t dst_bit  = inst->bitfield.dst_bit;
-
          fprintf(fp, "%s", mme_fermi_op_to_str(inst->op));
          print_reg(fp, inst->src[0]);
          print_reg(fp, inst->src[1]);
-         fprintf(fp, " (%u, %u, %u)", src_bit, size, dst_bit);
+         fprintf(fp, " (%u, %u, %u)", inst->bitfield.src_bit,
+                                      inst->bitfield.size,
+                                      inst->bitfield.dst_bit);
          break;
       case MME_FERMI_OP_BFE_LSL_IMM:
          fprintf(fp, "%s", mme_fermi_op_to_str(inst->op));