<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 20, 2016 at 11:38 AM, Kristian Høgsberg <span dir="ltr"><<a href="mailto:krh@bitplanet.net" target="_blank">krh@bitplanet.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Apr 18, 2016 at 5:10 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> The first patch in this series adds a short style guide for the Vulkan<br>
> driver.  The rest adds a new emit macro and updates the entire driver to<br>
> use it and, while we're there, makes the style more consistent.<br>
><br>
> Jason Ekstrand (18):<br>
>   anv: Add a short style guide<br>
>   anv: Add a new block-based batch emit macro<br>
>   anv/cmd_buffer: Use the new emit macro for 3DPRIMITIVE commands<br>
>   anv/cmd_buffer: Use the new emit macro for PIPE_CONTROL and<br>
>     STATE_BASE_ADDRESS<br>
>   anv/cmd_buffer: Use the new emit macro for DEPTH/STENCIL_BUFFER<br>
>   anv/cmd_buffer: Use the new emit macro for 3DSTATE_CONSTANT<br>
>   anv/cmd_buffer: Use the new emit macro for compute shader dispatch<br>
>   anv/cmd_buffer: Use the new emit macro for DRAWING_RECTANGLE<br>
>   anv/cmd_buffer: Use the new emit macro for quaries<br>
>   anv/gen8_cmd_buffer: Use the new emit macro<br>
>   anv/genX_pipeline: Use the new emit macro<br>
>   anv/gen8_pipeline: Use the new emit macro<br>
>   anv/state: Use the new emit macro<br>
>   anv/device: Use the new emit macro<br>
>   anv/gen7_cmd_buffer: Use the new emit macro<br>
>   anv/gen7_pipeline: Use the new emit macro<br>
>   anv: Remove the old emit macro<br>
>   anv: s/anv_batch_emit_blk/anv_batch_emit/<br>
><br>
>  src/intel/vulkan/STYLE                |  67 +++++<br>
>  src/intel/vulkan/anv_batch_chain.c    |  17 +-<br>
>  src/intel/vulkan/anv_device.c         |   8 +-<br>
>  src/intel/vulkan/anv_private.h        |  19 +-<br>
>  src/intel/vulkan/gen7_cmd_buffer.c    | 115 ++++----<br>
>  src/intel/vulkan/gen7_pipeline.c      | 236 +++++++++--------<br>
>  src/intel/vulkan/gen8_cmd_buffer.c    | 155 ++++++-----<br>
>  src/intel/vulkan/gen8_pipeline.c      | 367 +++++++++++++-------------<br>
>  src/intel/vulkan/genX_cmd_buffer.c    | 480 +++++++++++++++++++---------------<br>
>  src/intel/vulkan/genX_pipeline.c      |  25 +-<br>
>  src/intel/vulkan/genX_pipeline_util.h |  58 ++--<br>
>  src/intel/vulkan/genX_state.c         | 155 +++++------<br>
>  12 files changed, 938 insertions(+), 764 deletions(-)<br>
>  create mode 100644 src/intel/vulkan/STYLE<br>
<br>
</div></div>That's a lot of lines!<br></blockquote><div><br></div><div>I'm just trying to make as much of the Vulkan driver blame to me as possible. :-P<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Acked-by: Kristian Høgsberg <<a href="mailto:krh@bitplanet.net">krh@bitplanet.net</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>