Looks good to me :) The series is:

Reviewed-by: Plamena Manolova <plamena.manol...@intel.com>

On Wed, Jun 14, 2017 at 2:19 AM, Jason Ekstrand <ja...@jlekstrand.net>
wrote:

> ---
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> index f24b0b5..7991f81 100644
> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> @@ -1684,13 +1684,8 @@ intel_miptree_alloc_non_msrt_mcs(struct
> brw_context *brw,
>     const uint32_t alloc_flags =
>        is_lossless_compressed ? 0 : BO_ALLOC_FOR_RENDER;
>
> -   /* ISL has stricter set of alignment rules then the drm allocator.
> -    * Therefore one can pass the ISL dimensions in terms of bytes instead
> of
> -    * trying to recalculate based on different format block sizes.
> -    */
> -   buf->bo = brw_bo_alloc_tiled_2d(brw->bufmgr, "ccs-miptree",
> -                                   buf->pitch, buf->size / buf->pitch,
> -                                   1, I915_TILING_Y, &buf->pitch,
> alloc_flags);
> +   buf->bo = brw_bo_alloc_tiled(brw->bufmgr, "ccs-miptree", buf->size,
> +                                I915_TILING_Y, buf->pitch, alloc_flags);
>     if (!buf->bo) {
>        free(buf);
>        free(aux_state);
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to