[Mesa-dev] [PATCH] i965: Implement the actual tables for texture alignment units [v2]

Eric Anholt eric at anholt.net
Mon Nov 21 16:19:39 PST 2011


On Mon, 21 Nov 2011 15:53:09 -0800, Chad Versace <chad.versace at linux.intel.com> wrote:
> From: Kenneth Graunke <kenneth at whitecape.org>
> 
> I implemented functions for horizontal/vertical alignment units separately
> because I find it easier to read that way...especially with all the
> corner-cases.
> 
> [chad] Corrected the vertical alignment calculation by checking for
> depthstencil formats.
> 
> v2:
>    - Fix typos in intel_horizaontal_texture_alingment_unit():

v3: fix typos in "horizontal" and "alignment".  (j/k)

>      s/height/width/ and s/VALIGN/HALIGN.
>    - Remove special case for compressed formats in
>      intel_get_texture_alignment unit(). Compressed formats are already
>      handled in the halign and valign functions.
>    - Replace check ``_mesa_is_depth_format(...) ||
>      _mesa_is_depthstencil_format(...)`` with explcitit checks against
>      GL_DEPTH_COMPONENT and GL_DEPTH_STENCIL.

Reviewed-by: Eric Anholt <eic at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20111121/43dbcc3a/attachment.pgp>


More information about the mesa-dev mailing list