Reviewed-by: Marek Olšák <marek.ol...@amd.com>
Marek
On Fri, May 18, 2018 at 7:07 PM, Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
wrote:
> We're not sharing 32_32_32 formats between different GPUs, so we
> do not have to align for vega on pre-vega cards.
>
> Fixes: e361970ed73 "radv: Add support for IMG_DATA_FORMAT_32_32_32."
> ---
> src/amd/common/ac_surface.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c
> index 9e742dc8a45..d7da9950256 100644
> --- a/src/amd/common/ac_surface.c
> +++ b/src/amd/common/ac_surface.c
> @@ -301,10 +301,10 @@ static int gfx6_compute_level(ADDR_HANDLE addrlib,
> */
> if (config->info.levels == 1 &&
> AddrSurfInfoIn->tileMode == ADDR_TM_LINEAR_ALIGNED &&
> - AddrSurfInfoIn->bpp) {
> + AddrSurfInfoIn->bpp &&
> + util_is_power_of_two_or_zero(AddrSurfInfoIn->bpp)) {
> unsigned alignment = 256 / (AddrSurfInfoIn->bpp / 8);
>
> - assert(util_is_power_of_two_or_zero(AddrSurfInfoIn->bpp));
> AddrSurfInfoIn->width = align(AddrSurfInfoIn->width,
> alignment);
> }
>
> --
> 2.17.0
>
> _______________________________________________
> 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
