From patchwork Mon Feb 5 09:54:53 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: mrdegier@gmail.com X-Patchwork-Id: 7509 Delivered-To: ffmpegpatchwork@gmail.com Received: by 10.2.150.1 with SMTP id c1csp2604797jai; Mon, 5 Feb 2018 02:02:02 -0800 (PST) X-Google-Smtp-Source: AH8x2258KRknpFJi9GFP6TFMXfDZ+idPNRDynjYi3/pkFRLwaDyIf3umAxTs+U3CQfaUUXqqVwok X-Received: by 10.223.171.161 with SMTP id s30mr23896953wrc.160.1517824922007; Mon, 05 Feb 2018 02:02:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1517824921; cv=none; d=google.com; s=arc-20160816; b=nowHmrblgz+pvU4AaD1vYRAr6G/CUSHk00u6ZRw/LM50Z3ICmgK5Tq3ZO0HTUmMjcm RV/Y+k55k9LvbKDdnyRK9ovhG0ur2bMRh7lvztQ6gHhw6yT/O7E0e7HbZIy4ItCDYOe2 ie9t6NUUlicCRPKeIvkW7qv/tTJrE/ZnLEnGPlxWnd24jChP8YtsFM9jn3t6Xi1ABiFt 5fzNJgTk3RC9At1Vs8s5Z64d1XE0Rr2ug+RfK5qTjxqsUislC8N59sstWBgNQNpY1UYs +bDnupvXpVewm7Eg6fnnOhnvNeW3Vtx+5MsKI0OVJlicuLwJarBQUh12rV/MyOA9tcKP KfgA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:cc:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:references:in-reply-to:message-id:date :to:from:dkim-signature:delivered-to:arc-authentication-results; bh=AHeZo+lwUWo2pkmxMtwtEYn94UziQStyO8JGcpAMtlg=; b=0UjVGdXG9btfZIO7oh9L4h0Atj8k2rgtfkC3Z5YqHMVtcCRfz5eARGPGhZuRQZ43+q B3YkqtbMVumbJ7ag/t1n850+C81a6DXTzS5ia+PwULk9XnB49wDGQl7zWkY8aR4xeVEn cJS9fYk31pCGL/coDJrK5nGpB3f9itvegLD8OxVx7drRARAEGrRwfoPi6USNgtEJ7nd7 Bl+qG8j5p7gZmUts46AWuXQ36xLJn4HB5H4xxCXFXC+iCnpVHpco4F8LXEFUQ4dq0PKL a07kGWJLVBR9acnDfjFR7+5kAjw7ss8EhyB9/xWwGONOz/P07AcMxCZ/0CHFpquLQvf0 RYvg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20161025 header.b=Hh0S0aCg; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id j10si5540282wmc.102.2018.02.05.02.02.01; Mon, 05 Feb 2018 02:02:01 -0800 (PST) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20161025 header.b=Hh0S0aCg; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 22F28680754; Mon, 5 Feb 2018 12:01:52 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8ABD86802AC for ; Mon, 5 Feb 2018 12:01:45 +0200 (EET) Received: by mail-wm0-f65.google.com with SMTP id 143so24802514wma.5 for ; Mon, 05 Feb 2018 02:01:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=Wa1tkToL0j3CK1/w9f3l1Yp0dSx4Lk5l209zpfq+XIs=; b=Hh0S0aCgyqt752ihi43yOK2Urg4qHmF8gOstkDh6uJNp3BhIh0AGr4anO7oSDYOwUL a2OV6Hx/qew12GU+gcfylWhgMimLLGx/wN3Y43FThUjlwHjNElHz2EZ1/LbVKNvnkD6v GI8Y2V5jtsCS+01lKdJgl/VePCoFqc7RS/X1VPqykJJg6+oWURBPW7R3AJABxGDMxw1l NpfcEGAaNLFMEOGbfAdXgcfQjyLhOlUv/+H7x0YS9rANIEdITNb5cv1LJA3RvTRKbBTw yBs5XFGBEbzBzzdM6Up8eZjJpNOlfRRm8/o9LPL8wamFBJE5ZPYhoddZ0uANdqNeKkFU TKbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=Wa1tkToL0j3CK1/w9f3l1Yp0dSx4Lk5l209zpfq+XIs=; b=BAoj4enlSURnz6zROmKGyzmaY2MOpYkMIPoK8v4iDnqBCEEl+oGGCyDbQVakkKy0O1 itKxiwj784dgZJFBM1kkbffOHDeXIWB889SPSeatlen+ZHI/vuxLg1zFvNgQ82rZ2ceW gbcuzAnk8qbJRbeKc0unAwF948jcqJgRDQG34BWhB/0kDyCpJ1ddbxblabge09bVpFnN OmIGfFJx/w+LHFzPf1ocSd+KfzBTAIXF9GGwzBX5I/IFtbfucMpKnY+7YIfuNwA5XVAI lrmXTflS54FnsBUfgjK0/ZMgHhTc5+xRMNTUgK8fNXMptigjK4Poc/FDQUvHC5YDVUB9 M0+g== X-Gm-Message-State: AKwxytfzguKWFOcPC75mHj4IRRqaWrSXaBIC8hvITmS3HEA1eEDifVUo 14F+W0p77ZUO5XU76m6CnLjxxor3 X-Received: by 10.80.147.72 with SMTP id n8mr77962464eda.189.1517824517165; Mon, 05 Feb 2018 01:55:17 -0800 (PST) Received: from yoyodyne.local.net ([80.82.202.196]) by smtp.gmail.com with ESMTPSA id k12sm6361803edl.86.2018.02.05.01.55.16 (version=TLS1 cipher=AES128-SHA bits=128/128); Mon, 05 Feb 2018 01:55:16 -0800 (PST) From: Menno de Gier To: ffmpeg-devel@ffmpeg.org Date: Mon, 5 Feb 2018 10:54:53 +0100 Message-Id: <20180205095453.9869-1-mrdegier@gmail.com> X-Mailer: git-send-email 2.16.1 In-Reply-To: <20180126103057.13172-1-mrdegier@gmail.com> References: <20180126103057.13172-1-mrdegier@gmail.com> Subject: [FFmpeg-devel] [PATCH v4] avcodec/libopus: support disabling phase inversion. X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Menno MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Menno Signed-off-by: Menno --- doc/encoders.texi | 5 +++++ libavcodec/libopusdec.c | 33 +++++++++++++++++++++++++++++++++ libavcodec/libopusenc.c | 14 ++++++++++++++ 3 files changed, 52 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 6a410a8cb6..c5dfc646d9 100644 --- a/doc/encoders.texi +++ b/doc/encoders.texi @@ -981,6 +981,11 @@ Other values include 0 for mono and stereo, 1 for surround sound with masking and LFE bandwidth optimizations, and 255 for independent streams with an unspecified channel layout. +@item apply_phase_inv (N.A.) (requires libopus >= 1.2) +If set to 0, disables the use of phase inversion for intensity stereo, +improving the quality of mono downmixes, but slightly reducing normal stereo +quality. The default is 1 (phase inversion enabled). + @end table @anchor{libshine} diff --git a/libavcodec/libopusdec.c b/libavcodec/libopusdec.c index 4f7f4755c2..3d2ee5b61b 100644 --- a/libavcodec/libopusdec.c +++ b/libavcodec/libopusdec.c @@ -25,6 +25,7 @@ #include "libavutil/internal.h" #include "libavutil/intreadwrite.h" #include "libavutil/ffmath.h" +#include "libavutil/opt.h" #include "avcodec.h" #include "internal.h" @@ -33,11 +34,15 @@ #include "libopus.h" struct libopus_context { + AVClass *class; OpusMSDecoder *dec; int pre_skip; #ifndef OPUS_SET_GAIN union { int i; double d; } gain; #endif +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + int apply_phase_inv; +#endif }; #define OPUS_HEAD_SIZE 19 @@ -136,6 +141,15 @@ static av_cold int libopus_decode_init(AVCodecContext *avc) } #endif +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + ret = opus_multistream_decoder_ctl(opus->dec, + OPUS_SET_PHASE_INVERSION_DISABLED(!opus->apply_phase_inv)); + if (ret != OPUS_OK) + av_log(avc, AV_LOG_WARNING, + "Unable to set phase inversion: %s\n", + opus_strerror(ret)); +#endif + /* Decoder delay (in samples) at 48kHz */ avc->delay = avc->internal->skip_samples = opus->pre_skip; @@ -209,6 +223,24 @@ static void libopus_flush(AVCodecContext *avc) avc->internal->skip_samples = opus->pre_skip; } + +#define OFFSET(x) offsetof(struct libopus_context, x) +#define FLAGS AV_OPT_FLAG_AUDIO_PARAM | AV_OPT_FLAG_DECODING_PARAM +static const AVOption libopusdec_options[] = { +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + { "apply_phase_inv", "Apply intensity stereo phase inversion", OFFSET(apply_phase_inv), AV_OPT_TYPE_BOOL, { .i64 = 1 }, 0, 1, FLAGS }, +#endif + { NULL }, +}; + +static const AVClass libopusdec_class = { + .class_name = "libopusdec", + .item_name = av_default_item_name, + .option = libopusdec_options, + .version = LIBAVUTIL_VERSION_INT, +}; + + AVCodec ff_libopus_decoder = { .name = "libopus", .long_name = NULL_IF_CONFIG_SMALL("libopus Opus"), @@ -223,5 +255,6 @@ AVCodec ff_libopus_decoder = { .sample_fmts = (const enum AVSampleFormat[]){ AV_SAMPLE_FMT_FLT, AV_SAMPLE_FMT_S16, AV_SAMPLE_FMT_NONE }, + .priv_class = &libopusdec_class, .wrapper_name = "libopus", }; diff --git a/libavcodec/libopusenc.c b/libavcodec/libopusenc.c index b449497d15..4ae81b0bb2 100644 --- a/libavcodec/libopusenc.c +++ b/libavcodec/libopusenc.c @@ -39,6 +39,9 @@ typedef struct LibopusEncOpts { int packet_size; int max_bandwidth; int mapping_family; +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + int apply_phase_inv; +#endif } LibopusEncOpts; typedef struct LibopusEncContext { @@ -154,6 +157,14 @@ static int libopus_configure_encoder(AVCodecContext *avctx, OpusMSEncoder *enc, "Unable to set maximum bandwidth: %s\n", opus_strerror(ret)); } +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + ret = opus_multistream_encoder_ctl(enc, + OPUS_SET_PHASE_INVERSION_DISABLED(!opts->apply_phase_inv)); + if (ret != OPUS_OK) + av_log(avctx, AV_LOG_WARNING, + "Unable to set phase inversion: %s\n", + opus_strerror(ret)); +#endif return OPUS_OK; } @@ -530,6 +541,9 @@ static const AVOption libopus_options[] = { { "on", "Use variable bit rate", 0, AV_OPT_TYPE_CONST, { .i64 = 1 }, 0, 0, FLAGS, "vbr" }, { "constrained", "Use constrained VBR", 0, AV_OPT_TYPE_CONST, { .i64 = 2 }, 0, 0, FLAGS, "vbr" }, { "mapping_family", "Channel Mapping Family", OFFSET(mapping_family), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, 255, FLAGS, "mapping_family" }, +#ifdef OPUS_SET_PHASE_INVERSION_DISABLED_REQUEST + { "apply_phase_inv", "Apply intensity stereo phase inversion", OFFSET(apply_phase_inv), AV_OPT_TYPE_BOOL, { .i64 = 1 }, 0, 1, FLAGS }, +#endif { NULL }, };