]> git.sesse.net Git - ffmpeg/commitdiff
vaapi_encode_h265: Remove confusing and redundant tile options
authorMark Thompson <sw@jkqxz.net>
Tue, 28 Jul 2020 22:50:20 +0000 (23:50 +0100)
committerMark Thompson <sw@jkqxz.net>
Sat, 1 Aug 2020 16:18:30 +0000 (17:18 +0100)
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.

doc/encoders.texi
libavcodec/vaapi_encode_h265.c

index 2f5457fb26d5bca4474121d941d749b55a3ae970..78b3be960527f1256cae7379f03602349a8bada9 100644 (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
index f6008778dfdfbe0c03da775efbc70967c6053c39..511218f659555128804e9e85e96b3f85e6da44ad 100644 (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 },
 };