[FFmpeg-devel] [PATCH 2/2] decklink: Add support for output of Active Format Description (AFD)

[Vittorio Giovara]

On Wed, Aug 22, 2018 at 9:53 PM, Devin Heitmueller <
dheitmueller at ltnglobal.com> wrote:

> Implement support for including AFD in decklink output.  This
> includes making sure the AFD data is preserved when going from
> an AVFrame to a V210 packet (needed for 10-bit support).
>
> Updated to reflect feedback from Marton Balint <cus at passwd.hu>,
> Carl Eugen Hoyos <ceffmpeg at gmail.com> and Aaron Levinson
> <alevinsn_dev at levland.net>.
>
> Signed-off-by: Devin Heitmueller <dheitmueller at ltnglobal.com>
> ---
>  libavcodec/avcodec.h         |  6 +++++
>  libavcodec/v210enc.c         |  8 +++++++
>  libavdevice/decklink_enc.cpp | 54 ++++++++++++++++++++++++++++++
> ++++++++++++--
>  3 files changed, 66 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 31e50d5a94..192e15746d 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -1362,6 +1362,12 @@ enum AVPacketSideDataType {
>      AV_PKT_DATA_ENCRYPTION_INFO,
>
>      /**
> +     * Active Format Description data consisting of a single byte as
> specified
> +     * in ETSI TS 101 154 using AVActiveFormatDescription enum.
> +     */
> +    AV_PKT_DATA_AFD,
> +
> +    /**
>       * The number of side data types.
>       * This is not part of the public API/ABI in the sense that it may
>       * change when new side data types are added.
>

I think you should add an entry in ff_decode_frame_props() so that pkt side
data can propagate to frame side data
-- 
Vittorio