Same patch encoded as base64. I hope this time it won't get trashed.

Nicolas DEROUINEAU
Research Engineer
VITEC

T.  +33 1 46 73 06 06
E.  nicolas.derouin...@vitec.com
W. www.vitec.com

________________________________________
De : ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> de la part de Nicolas 
Derouineau <nicolas.derouin...@vitec.com>
Envoyé : jeudi 24 décembre 2015 11:31
À : FFmpeg development discussions and patches
Cc : vittorio.giov...@gmail.com; Michael Niedermayer; Carl Eugen Hoyos
Objet : Re: [FFmpeg-devel] [PATCH] Adding frame side data       about   green   
metadata

Please find here attached the updated patch with the suggested modifications.

Best regards,

Nicolas DEROUINEAU

________________________________________
De : ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> de la part de Michael 
Niedermayer <mich...@niedermayer.cc>
Envoyé : mercredi 23 décembre 2015 14:49
À : FFmpeg development discussions and patches
Objet : Re: [FFmpeg-devel] [PATCH] Adding frame side data about green   metadata

On Wed, Dec 23, 2015 at 12:43:48PM +0000, Nicolas Derouineau wrote:
> > Sorry if I missed something:
> > What is green metadata?
>
> Greenmetadata is a set of metrics specified in ISO/IEC 23001-11.
>
> They are used to predict the decoding complexity (in terms of cycles) of AVC 
> frames. It is useful to perform DVFS (ie: CPU frequency scaling) in order to 
> reduce decoder power consumption.
>
> By the way, I'm not sure the patch was correctly attached. I'm trying one 
> more time to attach it.
[...]

> index e69de29..7b383e7 100644
> --- a/libavutil/greenmetadata.h
> +++ b/libavutil/greenmetadata.h
> @@ -0,0 +1,38 @@
> +/*
> + * Copyright (c) 2015 Nicolas DEROUINEAU <nicolas.derouin...@vitec.com>
> + *
> + * This file is part of FFmpeg.
> + *
> + * FFmpeg is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * FFmpeg is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with FFmpeg; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
> USA
> + */
> +
> +#ifndef AVUTIL_GREENMD_H
> +#define AVUTIL_GREENMD_H
> +
> +#include <stdint.h>

> +#include <libavcodec/h264.h>

libavutil should not include stuff from libavcodec
also it would be "libavcodec/h264.h" not <libavcodec/h264.h>


[...]

--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
RnJvbSAwNGUzMzg0MGE1YTlmMTY0MTE1MjdmNjEyOWFiOWRjYWVlMmUzM2JhIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBOaWNvbGFzIERFUk9VSU5FQVUgPG5pY29sYXMuZGVyb3VpbmVh
dUB2aXRlYy5jb20+CkRhdGU6IFRodSwgMjQgRGVjIDIwMTUgMTE6MTY6NTcgKzAxMDAKU3ViamVj
dDogW1BBVENIXSBBZGQgZnJhbWUgc2lkZSBkYXRhIHdoZW4gU0VJIGdyZWVuIG1ldGFkYXRhIGFy
ZSBkZXRlY3RlZAoKLS0tCiBsaWJhdmNvZGVjL2gyNjQuYyAgICAgfCAyMSArKysrKysrKysrKysr
KysrKysrKysKIGxpYmF2Y29kZWMvaDI2NC5oICAgICB8ICAxICsKIGxpYmF2Y29kZWMvaDI2NF9z
ZWkuYyB8ICAzICsrKwogbGliYXZ1dGlsL2ZyYW1lLmggICAgIHwgIDggKysrKysrKysKIDQgZmls
ZXMgY2hhbmdlZCwgMzMgaW5zZXJ0aW9ucygrKQoKZGlmZiAtLWdpdCBhL2xpYmF2Y29kZWMvaDI2
NC5jIGIvbGliYXZjb2RlYy9oMjY0LmMKaW5kZXggMDg5YTg2Zi4uZTkwYmNjMCAxMDA2NDQKLS0t
IGEvbGliYXZjb2RlYy9oMjY0LmMKKysrIGIvbGliYXZjb2RlYy9oMjY0LmMKQEAgLTg3OSw2ICs4
NzksMjcgQEAgc3RhdGljIHZvaWQgZGVjb2RlX3Bvc3Rpbml0KEgyNjRDb250ZXh0ICpoLCBpbnQg
c2V0dXBfZmluaXNoZWQpCiAgICAgICAgIH0KICAgICB9CiAKKyAgICBpZiAoaC0+c2VpX2dyZWVu
X21ldGFkYXRhX3ByZXNlbnQpIHsKKyAgICAgICAgQVZGcmFtZVNpZGVEYXRhICpHcmVlbm1kID0g
YXZfZnJhbWVfbmV3X3NpZGVfZGF0YShjdXItPmYsIEFWX0ZSQU1FX0RBVEFfR1JFRU5NRCwKKyAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzaXplb2Yo
R3JlZW5NZXRhRGF0YSkpOworICAgICAgICBpZiAoR3JlZW5tZCkgeworICAgICAgICAgICAgbWVt
c2V0KCh1aW50OF90KilHcmVlbm1kLT5kYXRhLCAwLCBzaXplb2YoR3JlZW5NZXRhRGF0YSkpOwor
ICAgICAgICAgICAgR3JlZW5tZC0+ZGF0YVswXSA9IGgtPnNlaV9ncmVlbl9tZXRhZGF0YS5ncmVl
bl9tZXRhZGF0YV90eXBlOworICAgICAgICAgICAgR3JlZW5tZC0+ZGF0YVsxXSA9IGgtPnNlaV9n
cmVlbl9tZXRhZGF0YS5wZXJpb2RfdHlwZTsKKyAgICAgICAgICAgIEdyZWVubWQtPmRhdGFbMl0g
PSAodWludDhfdCkoaC0+c2VpX2dyZWVuX21ldGFkYXRhLm51bV9zZWNvbmRzPj44KTsKKyAgICAg
ICAgICAgIEdyZWVubWQtPmRhdGFbM10gPSAodWludDhfdCkoaC0+c2VpX2dyZWVuX21ldGFkYXRh
Lm51bV9zZWNvbmRzJjB4RkYpOworICAgICAgICAgICAgR3JlZW5tZC0+ZGF0YVs0XSA9ICh1aW50
OF90KShoLT5zZWlfZ3JlZW5fbWV0YWRhdGEubnVtX3BpY3R1cmVzPj44KTsKKyAgICAgICAgICAg
IEdyZWVubWQtPmRhdGFbNV0gPSAodWludDhfdCkoaC0+c2VpX2dyZWVuX21ldGFkYXRhLm51bV9w
aWN0dXJlcyYweEZGKTsKKyAgICAgICAgICAgIEdyZWVubWQtPmRhdGFbNl0gPSBoLT5zZWlfZ3Jl
ZW5fbWV0YWRhdGEucGVyY2VudF9ub25femVyb19tYWNyb2Jsb2NrczsKKyAgICAgICAgICAgIEdy
ZWVubWQtPmRhdGFbN10gPSBoLT5zZWlfZ3JlZW5fbWV0YWRhdGEucGVyY2VudF9pbnRyYV9jb2Rl
ZF9tYWNyb2Jsb2NrczsKKyAgICAgICAgICAgIEdyZWVubWQtPmRhdGFbOF0gPSBoLT5zZWlfZ3Jl
ZW5fbWV0YWRhdGEucGVyY2VudF9zaXhfdGFwX2ZpbHRlcmluZzsKKyAgICAgICAgICAgIEdyZWVu
bWQtPmRhdGFbOV0gPSBoLT5zZWlfZ3JlZW5fbWV0YWRhdGEucGVyY2VudF9hbHBoYV9wb2ludF9k
ZWJsb2NraW5nX2luc3RhbmNlOworICAgICAgICAgICAgR3JlZW5tZC0+ZGF0YVsxMF0gPSBoLT5z
ZWlfZ3JlZW5fbWV0YWRhdGEueHNkX21ldHJpY190eXBlOworICAgICAgICAgICAgR3JlZW5tZC0+
ZGF0YVsxMV0gPSAodWludDhfdCkoaC0+c2VpX2dyZWVuX21ldGFkYXRhLnhzZF9tZXRyaWNfdmFs
dWU+PjgpOworICAgICAgICAgICAgR3JlZW5tZC0+ZGF0YVsxMl0gPSAodWludDhfdCkoaC0+c2Vp
X2dyZWVuX21ldGFkYXRhLnhzZF9tZXRyaWNfdmFsdWUmMHhGRik7CisgICAgICAgIH0KKyAgICB9
CisKICAgICBpZiAoaC0+c2VpX3JlZ3VzZXJkYXRhX2FmZF9wcmVzZW50KSB7CiAgICAgICAgIEFW
RnJhbWVTaWRlRGF0YSAqc2QgPSBhdl9mcmFtZV9uZXdfc2lkZV9kYXRhKGN1ci0+ZiwgQVZfRlJB
TUVfREFUQV9BRkQsCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgIHNpemVvZih1aW50OF90KSk7CmRpZmYgLS1naXQgYS9saWJhdmNvZGVjL2gyNjQu
aCBiL2xpYmF2Y29kZWMvaDI2NC5oCmluZGV4IDVkOWFlY2QuLjUxNDkwZDYgMTAwNjQ0Ci0tLSBh
L2xpYmF2Y29kZWMvaDI2NC5oCisrKyBiL2xpYmF2Y29kZWMvaDI2NC5oCkBAIC04MzksNiArODM5
LDcgQEAgdHlwZWRlZiBzdHJ1Y3QgSDI2NENvbnRleHQgewogICAgIHFwZWxfbWNfZnVuYyAoKnFw
ZWxfYXZnKVsxNl07CiAKICAgICAvKkdyZWVuIE1ldGFkYXRhICovCisgICAgaW50IHNlaV9ncmVl
bl9tZXRhZGF0YV9wcmVzZW50OwogICAgIEdyZWVuTWV0YURhdGEgc2VpX2dyZWVuX21ldGFkYXRh
OwogCiB9IEgyNjRDb250ZXh0OwpkaWZmIC0tZ2l0IGEvbGliYXZjb2RlYy9oMjY0X3NlaS5jIGIv
bGliYXZjb2RlYy9oMjY0X3NlaS5jCmluZGV4IDA0MTFiODcuLjRhMDIxYjggMTAwNjQ0Ci0tLSBh
L2xpYmF2Y29kZWMvaDI2NF9zZWkuYworKysgYi9saWJhdmNvZGVjL2gyNjRfc2VpLmMKQEAgLTQz
LDYgKzQzLDcgQEAgdm9pZCBmZl9oMjY0X3Jlc2V0X3NlaShIMjY0Q29udGV4dCAqaCkKICAgICBo
LT5zZWlfZnJhbWVfcGFja2luZ19wcmVzZW50ICAgID0gIDA7CiAgICAgaC0+c2VpX2Rpc3BsYXlf
b3JpZW50YXRpb25fcHJlc2VudCA9IDA7CiAgICAgaC0+c2VpX3JlZ3VzZXJkYXRhX2FmZF9wcmVz
ZW50ICA9ICAwOworICAgIGgtPnNlaV9ncmVlbl9tZXRhZGF0YV9wcmVzZW50ICAgPSAgMDsKIAog
ICAgIGgtPmE1M19jYXB0aW9uX3NpemUgPSAwOwogICAgIGF2X2ZyZWVwKCZoLT5hNTNfY2FwdGlv
bik7CkBAIC0zNjMsNiArMzY0LDggQEAgc3RhdGljIGludCBkZWNvZGVfR3JlZW5NZXRhZGF0YShI
MjY0Q29udGV4dCAqaCkKICAgICBpZiAoaC0+YXZjdHgtPmRlYnVnICYgRkZfREVCVUdfR1JFRU5f
TUQpCiAgICAgICAgIGF2X2xvZyhoLT5hdmN0eCwgQVZfTE9HX0RFQlVHLCAgICAgICAgICAiR3Jl
ZW4gTWV0YWRhdGEgSW5mbyBTRUkgbWVzc2FnZVxuIik7CiAKKyAgICBoLT5zZWlfZ3JlZW5fbWV0
YWRhdGFfcHJlc2VudCA9IDE7CisKICAgICBoLT5zZWlfZ3JlZW5fbWV0YWRhdGEuZ3JlZW5fbWV0
YWRhdGFfdHlwZT1nZXRfYml0cygmaC0+Z2IsIDgpOwogCiAgICAgaWYgKGgtPmF2Y3R4LT5kZWJ1
ZyAmIEZGX0RFQlVHX0dSRUVOX01EKQpkaWZmIC0tZ2l0IGEvbGliYXZ1dGlsL2ZyYW1lLmggYi9s
aWJhdnV0aWwvZnJhbWUuaAppbmRleCA5YzYwNjFhLi44OWE1N2FkIDEwMDY0NAotLS0gYS9saWJh
dnV0aWwvZnJhbWUuaAorKysgYi9saWJhdnV0aWwvZnJhbWUuaApAQCAtMTEyLDYgKzExMiwxNCBA
QCBlbnVtIEFWRnJhbWVTaWRlRGF0YVR5cGUgewogICAgICAqIGVudW0gQVZBdWRpb1NlcnZpY2VU
eXBlIGRlZmluZWQgaW4gYXZjb2RlYy5oLgogICAgICAqLwogICAgIEFWX0ZSQU1FX0RBVEFfQVVE
SU9fU0VSVklDRV9UWVBFLAorCisKKyAgICAvKioKKyAgICAgKiBUaGlzIHNpZGUgZGF0YSBtdXN0
IGJlIGFzc29jaWF0ZWQgd2l0aCBhIHZpZGVvIGZyYW1lIGFuZCBjb3JyZXNwb25kcyB0bworICAg
ICAqIHN0cnVjdCBHcmVlbk1EYXRhVHlwZSBkZWZpbmVkIGluIGF2Y29kZWMuaC4gVGhlIGdyZWVu
IG1ldGFkYXRhIHNwZWNpZmljYXRpb24KKyAgICAgKiBpcyBnaXZlbiBpbiBJU08vSUVDIDIzMDAx
LgorICAgICAqLworICAgIEFWX0ZSQU1FX0RBVEFfR1JFRU5NRCwKIH07CiAKIGVudW0gQVZBY3Rp
dmVGb3JtYXREZXNjcmlwdGlvbiB7Ci0tIAoxLjkuMQoK
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to