RepoMirrors
/
ffmpeg
mirror of https://git.ffmpeg.org/ffmpeg.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

vaapi_encode_h265: Remove confusing and redundant tile options 

The tile_rows/cols options currently do a confusingly different thing to
the options of the same name on other encoders like libvpx and libaom.
There is no backward-compatibility reason to implement the log2 behaviour
as there was for libaom, so just get rid of them entirely.
This commit is contained in:
Mark Thompson 2020-07-28 23:50:20 +01:00
parent 11a2d05b3f
commit e80fe32943
2 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
doc/encoders.texi
View File

@ -3268,14 +3268,6 @@ Set the number of tiles to encode the input video with, as rows x columns.
Larger numbers allow greater parallelism in both encoding and decoding, but
may decrease coding efficiency.
@item tile_rows
Selects how many rows of tiles to encode with. For example, 4 tile rows would
be requested by setting the tile_rows option to 4.
@item tile_cols
Selects how many columns of tiles to encode with. For example, 5 tile columns
would be requested by setting the tile_cols option to 5.
@end table
@item mjpeg_vaapi

4
libavcodec/vaapi_encode_h265.c
View File

@ -1291,10 +1291,6 @@ static const AVOption vaapi_encode_h265_options[] = {
{ "tiles", "Tile rows x cols",
OFFSET(trows), AV_OPT_TYPE_IMAGE_SIZE, { .str = NULL }, 0, 0, FLAGS },
{ "tile_rows", "Number of rows for tile encoding",
OFFSET(trows), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, INT_MAX, FLAGS },
{ "tile_cols", "Number of cols for tile encoding",
OFFSET(tcols), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, INT_MAX, FLAGS },
{ NULL },
};