[FFmpeg-devel] [PATCH 1/6] vaapi_encode_h265: Remove confusing and redundant tile options
Mark Thompson
sw at jkqxz.net
Wed Jul 29 01:50:20 EEST 2020
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 | 8 --------
libavcodec/vaapi_encode_h265.c | 4 ----
2 files changed, 12 deletions(-)
diff --git a/doc/encoders.texi b/doc/encoders.texi
index ed8ef63784..c837e9455a 100644
--- a/doc/encoders.texi
+++ b/doc/encoders.texi
@@ -3206,14 +3206,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.
- at 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.
-
- at 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
diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
index f6008778df..511218f659 100644
--- a/libavcodec/vaapi_encode_h265.c
+++ b/libavcodec/vaapi_encode_h265.c
@@ -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 },
};
--
2.27.0
More information about the ffmpeg-devel
mailing list