<div dir="ltr">Makes me think that in the future we should start splitting out midgard and bifrost specific things in to their own files for the decoder.<div>Alas that isn't today so this patch series is <span class="gmail-il">Reviewed</span>-<span class="gmail-il">by</span>: Ryan Houdek <<a href="mailto:Sonicadvance1@gmail.com">Sonicadvance1@gmail.com</a>></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, May 18, 2019 at 2:53 PM Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io">alyssa@rosenzweig.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This patch series updates pandecode to work with Bifrost traces, wiring<br>
in disassembly and eliminating #ifdef BIFROST shenanigans. Paired with a<br>
current panwrap, this enables tracing Bifrost (Dvalin) and then decoding<br>
reliably with upstream pandecode.<br>
<br>
Alyssa Rosenzweig (5):<br>
  panfrost/decode: Disassemble Bifrost shaders<br>
  panfrost: Hoist blend constant into Midgard-specific struct<br>
  panfrost/decode: Decode blend constant<br>
  panfrost: Cleanup panfrost_job comments<br>
  panfrost: Fix Bifrost-specific padding<br>
<br>
 .../drivers/panfrost/include/panfrost-job.h   | 45 ++++++++++---------<br>
 src/gallium/drivers/panfrost/meson.build      |  3 +-<br>
 src/gallium/drivers/panfrost/pan_blending.c   |  9 ++--<br>
 src/gallium/drivers/panfrost/pan_blending.h   |  4 +-<br>
 src/gallium/drivers/panfrost/pan_context.c    |  9 ++--<br>
 src/gallium/drivers/panfrost/pan_context.h    |  1 +<br>
 .../drivers/panfrost/pan_pretty_print.c       |  2 -<br>
 .../drivers/panfrost/pandecode/decode.c       | 32 +++++++++----<br>
 8 files changed, 66 insertions(+), 39 deletions(-)<br>
<br>
-- <br>
2.20.1<br>
<br>
</blockquote></div>