From 942456f64688059bb8b110544ecc5aaf9cfd7e49 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Wed, 27 Jun 2018 15:42:10 -0700 Subject: [PATCH] v3d: Skip printing sub-id or pad fields in CLIF dumping. The parser doesn't expect them, so our fields would end up mismatched. They're not really useful in console output, either. --- src/broadcom/cle/v3d_decoder.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/broadcom/cle/v3d_decoder.c b/src/broadcom/cle/v3d_decoder.c index ed39ec48fc5..7eb963fcf18 100644 --- a/src/broadcom/cle/v3d_decoder.c +++ b/src/broadcom/cle/v3d_decoder.c @@ -960,6 +960,14 @@ v3d_print_group(struct clif_dump *clif, struct v3d_group *group, v3d_field_iterator_init(&iter, group, p); while (v3d_field_iterator_next(clif, &iter)) { + /* Clif parsing uses the packet name, and expects no + * sub-id. + */ + if (strcmp(iter.field->name, "sub-id") == 0 || + strcmp(iter.field->name, "unused") == 0 || + strcmp(iter.field->name, "Pad") == 0) + continue; + fprintf(clif->out, " %s: %s\n", iter.name, iter.value); if (iter.struct_desc) { uint64_t struct_offset = offset + iter.offset; -- 2.30.2