<div dir="ltr">On 2 December 2013 11:39, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
 src/mesa/drivers/dri/i965/brw_tex_layout.c    | 51 +++++++++++++++++++++------<br>
 src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 10 ++++++<br>
 2 files changed, 50 insertions(+), 11 deletions(-)<br></blockquote><div><br></div><div>Since "horizontal slice pitch" and "vertical slice pitch" are not defined in the bspec, these funtions should have documentation explaining what they mean.<br>
<br></div><div>Correct me if I'm wrong, but I think what you mean by "horizontal slice pitch" is:<br><br></div><div>- For a 3D texture, the horizontal spacing between slices at a given miplevel.<br></div><div>
- For any other texture, not used.<br><br></div><div>And by "vertical slice pitch"<br><br></div><div>- For a 3D texture, the vertical spacing between each row of slices.<br></div><div>- For any other texture, the vertical spacing between array slices.<br>
<br></div><div>With additional comments added, this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div></div></div>