<div dir="ltr"><div class="gmail_extra">..<br><div class="gmail_quote">On Fri, Jul 14, 2017 at 3:50 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</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 Wed 12 Jul 2017, Jason Ekstrand wrote:<br>
> From: Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>><br>
><br>
> v2: move is_aux into if block. (Jason)<br>
> Use else block instead of goto (Jason)<br>
><br>
> v3: Fix up logic for is_aux (Ben)<br>
> Fix up size calculations and add FIXME (Ben)<br>
><br>
> v4 (Jason Ekstrand):<br>
> Use the aux_pitch in the image instead of calculating it<br>
><br>
> Signed-off-by: Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>><br>
> Acked-by: Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a>><br>
> Reviewed-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
> ---<br>
>  src/intel/isl/isl.h                      |  6 ++++<br>
>  src/mesa/drivers/dri/i965/<wbr>intel_screen.c | 55 ++++++++++++++++++++----------<wbr>--<br>
>  2 files changed, 40 insertions(+), 21 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
> index d81df31..7dd3618 100644<br>
> --- a/src/intel/isl/isl.h<br>
> +++ b/src/intel/isl/isl.h<br>
> @@ -1519,6 +1519,12 @@ isl_tiling_to_i915_tiling(enum isl_tiling tiling);<br>
>  const struct isl_drm_modifier_info * ATTRIBUTE_CONST<br>
>  isl_drm_modifier_get_info(<wbr>uint64_t modifier);<br>
><br>
> +static inline bool<br>
> +isl_drm_modifier_has_aux(<wbr>uint64_t modifier)<br>
> +{<br>
> +   return isl_drm_modifier_get_info(<wbr>modifier)->aux_usage != ISL_AUX_USAGE_NONE;<br>
> +}<br>
<br>
</div></div>Thanks for this improvement in v3.<br>
Reviewed-by: Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
<br>
If I missed any patches, let me know.<br>
</blockquote></div><br></div><div class="gmail_extra">I sent another patch that removes an assert.  It's pretty trivial.<br></div></div>