[FFmpeg-devel] [PATCH 0/2] avcodec: metadata for hardware backed decoders

[Philip Langdale]

On 2017-12-11 07:34, wm4 wrote:
> On Mon, 27 Nov 2017 20:45:59 -0800
> Philip Langdale <philipl at overt.org> wrote:
> 
>> We have a number of hardware backed implementations in the codebase
>> that are done as full decoders instead of HWAccels for various
>> reasons. These decoders cannot be discovered today, and clients
>> which care end up needing to hardcode their names. It would be
>> helpful to provide a discovery mechanism.
>> 
>> Philip Langdale (2):
>>   avcodec: Add metadata to identify hardware backed codecs
>>   avcodec: Add hardware metadata to hardware backed decoders
>> 
>>  Changelog                  |  1 +
>>  doc/APIchanges             |  5 +++++
>>  libavcodec/avcodec.h       | 19 +++++++++++++++++++
>>  libavcodec/crystalhd.c     |  4 +++-
>>  libavcodec/cuviddec.c      |  4 +++-
>>  libavcodec/mediacodecdec.c | 24 ++++++++++++++++++------
>>  libavcodec/mmaldec.c       |  3 ++-
>>  libavcodec/qsvdec_h2645.c  |  8 ++++++--
>>  libavcodec/qsvdec_other.c  | 12 +++++++++---
>>  libavcodec/v4l2_m2m_dec.c  |  2 ++
>>  10 files changed, 68 insertions(+), 14 deletions(-)
>> 
> 
> Sent a similar patch to Libav:
> https://lists.libav.org/pipermail/libav-devel/2017-December/085505.html

Looks fine by me. I was hoping Mark would review this, but I'm just as 
happy
if your version goes in; I think your flag names are better.

--phil