From patchwork Fri Mar 31 08:00:21 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steve Lhomme X-Patchwork-Id: 3217 Delivered-To: ffmpegpatchwork@gmail.com Received: by 10.103.44.195 with SMTP id s186csp1924731vss; Fri, 31 Mar 2017 01:00:44 -0700 (PDT) X-Received: by 10.28.88.2 with SMTP id m2mr2014009wmb.12.1490947244701; Fri, 31 Mar 2017 01:00:44 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id n20si7053961wra.196.2017.03.31.01.00.43; Fri, 31 Mar 2017 01:00:44 -0700 (PDT) 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; 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 6A779689999; Fri, 31 Mar 2017 11:00:40 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wr0-f193.google.com (mail-wr0-f193.google.com [209.85.128.193]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9458968991C for ; Fri, 31 Mar 2017 11:00:34 +0300 (EEST) Received: by mail-wr0-f193.google.com with SMTP id u18so18304444wrc.0 for ; Fri, 31 Mar 2017 01:00:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id; bh=pWopybYTFXZIsiYFKOR94K2/TR/6fNT4nutRhtDg3M0=; b=j4i+roOh/gKVJlmFFntg5C0fT8F09z3jwI6m28DWnSnfB2RonP2HuX4diAi10F3KF8 q+Cqp2CLm/SUQ02FN548E49PweR58LBjeaTQ2x0n/GFuqdGGG2AmEtLObFACwjV8QD+Y dr0BpV9VOKd8qJH7aSbJVkf5Nshdnw5ZAOlvyGyxNe3pTL68KX9PN6C3RCzM4n6+zqBn DJskiXWCLeht6hF2Jl2/x6kEW7BJTKeLJzTRxH2ecD63ttuyXq9rKcdWJq9h7qBm3bNc chrsaKQ3j6iEnFG+wyJtQk5jvvQP7+9agRgArxnvkImITx/ScQJ8Q5nve9i6zTWihTqH U+QA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=pWopybYTFXZIsiYFKOR94K2/TR/6fNT4nutRhtDg3M0=; b=nNHGljzD+El2Vra+FTHTxW0h3xt+pKz8Fx/BXvL2JjN/Hcw3eE+vam6Ufk7ESJonNO DEkYJyLcKe305U924JNTTf4Mziacwax4Uh+X6Q052d2USVyiR2D+TnrCCq5hoXuwNVD+ wcEXVnVCrSCohUkyBNToYooqF4YzD9i1oeByv1eYfYG5lloXJyU5QafkhA4erYAIYmgO bfo21HN3TJOIYTFNPlqZyG4kLDWXgHPQPV9hFwsvLRpcEnOcubgQflvZnc4yytT5xjEp 6tzaL89GjrqBL6LzgZrvVzF5D3fL/pHB1Or0X3UKXVGwYoHjxQLpL3nDo+zNto9L9lOC VYDw== X-Gm-Message-State: AFeK/H16xJVd7dYm2xnzfELLpucE3/bWsgdQe6RHp4tgvWxqZ3874uKH7oiL1Mm/1/peFw== X-Received: by 10.223.177.219 with SMTP id r27mr1653153wra.194.1490947234117; Fri, 31 Mar 2017 01:00:34 -0700 (PDT) Received: from localhost.localdomain (2a01cb0c000ea9002158cfa09c18f50d.ipv6.abo.wanadoo.fr. [2a01:cb0c:e:a900:2158:cfa0:9c18:f50d]) by smtp.gmail.com with ESMTPSA id v14sm1873222wmv.24.2017.03.31.01.00.32 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 31 Mar 2017 01:00:33 -0700 (PDT) From: Steve Lhomme To: ffmpeg-devel@ffmpeg.org Date: Fri, 31 Mar 2017 10:00:21 +0200 Message-Id: <20170331080023.10432-1-robux4@gmail.com> X-Mailer: git-send-email 2.11.1 Subject: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata 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 MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. --- update the previous patch: - rename CEA 861.3 to CTA-861.3 - use MaxCLL and MaxFALL names that are more commonly found - use unsigned integer rather than rational numbers as (supposedly) in the specs - provide the structure size on output of av_content_light_metadata_alloc() --- libavutil/frame.c | 1 + libavutil/frame.h | 6 ++++++ libavutil/mastering_display_metadata.c | 23 ++++++++++++++++++++ libavutil/mastering_display_metadata.h | 39 ++++++++++++++++++++++++++++++++++ libavutil/version.h | 2 +- 5 files changed, 70 insertions(+), 1 deletion(-) diff --git a/libavutil/frame.c b/libavutil/frame.c index 8811dcdcfe..dd12c5549c 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -772,6 +772,7 @@ const char *av_frame_side_data_name(enum AVFrameSideDataType type) case AV_FRAME_DATA_SKIP_SAMPLES: return "Skip samples"; case AV_FRAME_DATA_AUDIO_SERVICE_TYPE: return "Audio service type"; case AV_FRAME_DATA_MASTERING_DISPLAY_METADATA: return "Mastering display metadata"; + case AV_FRAME_DATA_CONTENT_LIGHT_LEVEL: return "Content light level metadata"; case AV_FRAME_DATA_GOP_TIMECODE: return "GOP timecode"; } return NULL; diff --git a/libavutil/frame.h b/libavutil/frame.h index 7cb78a1a44..1ecb6a087a 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -127,6 +127,12 @@ enum AVFrameSideDataType { * libavutil/spherical.h. */ AV_FRAME_DATA_SPHERICAL, + + /** + * Content light level (based on CTA-861.3). This payload containts data in + * the form of the AVContentLightMetadata struct. + */ + AV_FRAME_DATA_CONTENT_LIGHT_LEVEL, }; enum AVActiveFormatDescription { diff --git a/libavutil/mastering_display_metadata.c b/libavutil/mastering_display_metadata.c index e1683e55c7..6069347617 100644 --- a/libavutil/mastering_display_metadata.c +++ b/libavutil/mastering_display_metadata.c @@ -41,3 +41,26 @@ AVMasteringDisplayMetadata *av_mastering_display_metadata_create_side_data(AVFra return (AVMasteringDisplayMetadata *)side_data->data; } + +AVContentLightMetadata *av_content_light_metadata_alloc(size_t *size) +{ + AVContentLightMetadata *metadata = av_mallocz(sizeof(AVContentLightMetadata)); + + if (size) + *size = sizeof(*metadata); + + return metadata; +} + +AVContentLightMetadata *av_content_light_metadata_create_side_data(AVFrame *frame) +{ + AVFrameSideData *side_data = av_frame_new_side_data(frame, + AV_FRAME_DATA_CONTENT_LIGHT_LEVEL, + sizeof(AVContentLightMetadata)); + if (!side_data) + return NULL; + + memset(side_data->data, 0, sizeof(AVContentLightMetadata)); + + return (AVContentLightMetadata *)side_data->data; +} diff --git a/libavutil/mastering_display_metadata.h b/libavutil/mastering_display_metadata.h index 936533fec4..847b0b62c6 100644 --- a/libavutil/mastering_display_metadata.h +++ b/libavutil/mastering_display_metadata.h @@ -86,4 +86,43 @@ AVMasteringDisplayMetadata *av_mastering_display_metadata_alloc(void); */ AVMasteringDisplayMetadata *av_mastering_display_metadata_create_side_data(AVFrame *frame); +/** + * Content light level needed by to transmit HDR over HDMI (CTA-861.3). + * + * To be used as payload of a AVFrameSideData or AVPacketSideData with the + * appropriate type. + * + * @note The struct should be allocated with av_content_light_metadata_alloc() + * and its size is not a part of the public ABI. + */ +typedef struct AVContentLightMetadata { + /** + * Max content light level (cd/m^2). + */ + unsigned MaxCLL; + + /** + * Max average light level per frame (cd/m^2). + */ + unsigned MaxFALL; +} AVContentLightMetadata; + +/** + * Allocate an AVContentLightMetadata structure and set its fields to + * default values. The resulting struct can be freed using av_freep(). + * + * @return An AVContentLightMetadata filled with default values or NULL + * on failure. + */ +AVContentLightMetadata *av_content_light_metadata_alloc(size_t *size); + +/** + * Allocate a complete AVContentLightMetadata and add it to the frame. + * + * @param frame The frame which side data is added to. + * + * @return The AVContentLightMetadata structure to be filled by caller. + */ +AVContentLightMetadata *av_content_light_metadata_create_side_data(AVFrame *frame); + #endif /* AVUTIL_MASTERING_DISPLAY_METADATA_H */ diff --git a/libavutil/version.h b/libavutil/version.h index a14c0e7eb2..fedc6fa177 100644 --- a/libavutil/version.h +++ b/libavutil/version.h @@ -79,7 +79,7 @@ */ #define LIBAVUTIL_VERSION_MAJOR 55 -#define LIBAVUTIL_VERSION_MINOR 53 +#define LIBAVUTIL_VERSION_MINOR 54 #define LIBAVUTIL_VERSION_MICRO 100 #define LIBAVUTIL_VERSION_INT AV_VERSION_INT(LIBAVUTIL_VERSION_MAJOR, \