[FFmpeg-devel] [PATCH] Add support for RockChip Media Process Platform This adds hardware decoding for h264 / HEVC / VP8 using MPP Rockchip API. Will return frames holding an AVDRMFrameDescriptor struct in buf[0] that allows drm / dmabuf usage. Was...

Mark Thompson sw at jkqxz.net
Sun Sep 3 16:27:06 EEST 2017


On 03/09/17 13:36, LongChair . wrote:
> Le 01/09/2017 à 21:13, Carl Eugen Hoyos a écrit :
>> This is Apache 2.0 (afaict) meaning it requires version 3 in case of --enable-gpl.
> That is right, although it seems MPP clearly contains some ffmpeg 2.2 or 
> something old code.
> see : https://github.com/rockchip-linux/mpp/blob/release/inc/mpp_frame.h#L59

These are standard values from ISO 11664-1; them being in common with ffmpeg is unsurprising.

> and https://github.com/rockchip-linux/mpp/tree/release/mpp/codec/dec/h265

Not sure what you're pointing at here.

> which probbaly voids the Apache 2.0 and makes it GPL2 ..

ffmpeg is LGPL 2.1 or later, not GPL.

Neither LGPL 2.1 nor GPL 2 are compatible for distribution with Apache 2.0 (I think I was confused over this earlier, apologies if I said the wrong thing on IRC), so we invoke the "or later" part with --enable-version3 to use LGPL 3 instead (or GPL 3 if --enable-gpl is also present), at which point the result should be distributable.  (Hopefully: IANAL, does not constitute legal advice, consult a lawyer yourself, etc.)

I think putting it in EXTERNAL_LIBRARY_VERSION3_LIST has the right effect.

- Mark


More information about the ffmpeg-devel mailing list