Quoting Mark Thompson (2017-08-11 01:37:06)
>  
> -    priv->ctu_width     = FFALIGN(ctx->surface_width,  32) / 32;
> -    priv->ctu_height    = FFALIGN(ctx->surface_height, 32) / 32;
> +    // This seems to be an Intel driver constraint.  Despite MinCbSizeY
> +    // being 8, we are still required to encode at 16-pixel alignment and
> +    // then crop back (so 1080 lines is still encode as 1088 + cropping).
> +    priv->ctu_width     = FFALIGN(ctx->surface_width,  16) / 16;
> +    priv->ctu_height    = FFALIGN(ctx->surface_height, 16) / 16;

Is this related to the rest of the patch? Does not seem so.

Otherwise looks ok as far as I can tell.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to