From 8007e8fc67e3a36ad065415356aa057f01cd83b6 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 28 Dec 2018 18:33:14 +0100 Subject: [PATCH] avfilter/af_afir: fix minp/maxp range and change default value for maxp --- doc/filters.texi | 6 +++--- libavfilter/af_afir.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 65ce25bc18..45d90f6165 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -1232,12 +1232,12 @@ Set video stream frame rate. This option is used only when @var{response} is ena @item minp Set minimal partition size used for convolution. Default is @var{16}. -Allowed range is from @var{16} to @var{65536}. +Allowed range is from @var{16} to @var{32768}. Lower values decreases latency at cost of higher CPU usage. @item maxp -Set maximal partition size used for convolution. Default is @var{65536}. -Allowed range is from @var{16} to @var{65536}. +Set maximal partition size used for convolution. Default is @var{8192}. +Allowed range is from @var{16} to @var{32768}. Lower values decreases latency at cost of higher CPU usage. @end table diff --git a/libavfilter/af_afir.c b/libavfilter/af_afir.c index 743f7d2658..ca0c8811a4 100644 --- a/libavfilter/af_afir.c +++ b/libavfilter/af_afir.c @@ -735,8 +735,8 @@ static const AVOption afir_options[] = { { "channel", "set IR channel to display frequency response", OFFSET(ir_channel), AV_OPT_TYPE_INT, {.i64=0}, 0, 1024, VF }, { "size", "set video size", OFFSET(w), AV_OPT_TYPE_IMAGE_SIZE, {.str = "hd720"}, 0, 0, VF }, { "rate", "set video rate", OFFSET(frame_rate), AV_OPT_TYPE_VIDEO_RATE, {.str = "25"}, 0, INT32_MAX, VF }, - { "minp", "set min partition size", OFFSET(minp), AV_OPT_TYPE_INT, {.i64=16}, 16, 65536, AF }, - { "maxp", "set max partition size", OFFSET(maxp), AV_OPT_TYPE_INT, {.i64=65536}, 16, 65536, AF }, + { "minp", "set min partition size", OFFSET(minp), AV_OPT_TYPE_INT, {.i64=16}, 16, 32768, AF }, + { "maxp", "set max partition size", OFFSET(maxp), AV_OPT_TYPE_INT, {.i64=8192}, 16, 32768, AF }, { NULL } };