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 [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
