<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 16, 2016 at 10:39 AM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat 11 Jun 2016, Jason Ekstrand wrote:<br>
> ---<br>
>  src/intel/isl/isl_surface_state.c | 28 ++++++++--------------------<br>
>  1 file changed, 8 insertions(+), 20 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl_surface_state.c b/src/intel/isl/isl_surface_state.c<br>
> index 50570aa..1e94e60 100644<br>
> --- a/src/intel/isl/isl_surface_state.c<br>
> +++ b/src/intel/isl/isl_surface_state.c<br>
> @@ -110,9 +110,8 @@ get_surftype(enum isl_surf_dim dim, isl_surf_usage_flags_t usage)<br>
</span>>  /*<br>
<span class="">>   * Get the values to pack into RENDER_SUFFACE_STATE.SurfaceHorizontalAlignment<br>
>   * and SurfaceVerticalAlignment.<br>
>   */<br>
> -static void<br>
> -get_halign_valign(const struct isl_surf *surf,<br>
> -                  uint32_t *halign, uint32_t *valign)<br>
> +static struct isl_extent3d<br>
> +get_image_alignment(const struct isl_surf *surf)<br>
<br>
<br>
</span>The function comment is incorrect post-patch. It should say something to<br>
the tune of "Returns indices into isl_to_gen_halign, isl_to_gen_valign".<br>
Specifically, the function comment needs to clarify (with as few words<br>
as possible) that the units of the returned extent is neither samples,<br>
pixels, nor elements, but something entirely different--array indices--<br>
because it's not really an extent at all.<br>
</blockquote></div><br></div><div class="gmail_extra">Right.  It's the "logical" halign/valign values not the actual hardware enums.<br></div></div>