<div dir="ltr">On 25 September 2013 10:18, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</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">On 07/22/2013 03:54 PM, Frank Henigman wrote:> Extend the fast texture upload from BGRA X-tiled to include RGBA,<div class="im">
> diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.c b/src/mesa/drivers/dri/i965/intel_tex_subimage.c<br>
> index 05e684c..6df7de1 100644<br>
> --- a/src/mesa/drivers/dri/i965/<u></u>intel_tex_subimage.c<br>
> +++ b/src/mesa/drivers/dri/i965/<u></u>intel_tex_subimage.c<br>
> @@ -43,6 +43,30 @@<br>
><br>
>   #define FILE_DEBUG_FLAG DEBUG_TEXTURE<br>
><br>
> +#define ALIGN_DOWN(a, b) ( (a) / (b) * (b) )<br>
> +#define ALIGN_UP(a, b) ALIGN_DOWN( (a)+(b)-1, b )<br>
<br></div>
There already exists a macro in Mesa to align-up, named ALIGN. Let's eliminate<br>
duplicate definitions by aliasing ALIGN_UP to ALIGN.<br>
<br>
#define ALIGN_UP(a, b) ALIGN(a, b)</blockquote><div><br></div><div>There's also a macro to do ALIGN_DOWN, which is called ROUND_DOWN_TO().<br><br></div><div>Note that the existing ALIGN() and ROUND_DOWN_TO() macros only work when b is a power of two.<br>
</div></div></div></div>