[FFmpeg-devel] [PATCH v4 4/5] lavc: Implement Dolby Vision RPU parsing

[Andreas Rheinhardt]

Niklas Haas:
> On Sat, 11 Dec 2021 15:02:14 +0100 Andreas Rheinhardt <andreas.rheinhardt at outlook.com> wrote:
>> verification
> 
> Fixed
> 
>> lavu
> 
> Fixed
> 
>> Why is this always compiled when only the HEVC decoder needs it?
>> (The way it is now will cause linking errors when CONFIG_GOLOMB is false.)
> 
> In principle, parsing the RPU contents is orthogonal from their delivery
> within HEVC. According to Dolby the same RPU may be attached to AVC
> (though admittedly nobody does so), and VVC also supports it (although
> this one is not implemented in lavc yet).
> 
> How about instead adding CONFIG_DOLBYVISION as an extra configure option
> (similar to CONFIG_GOLOMB), and have hevc_decoder auto-select it?
> 

Sounds fine to me (I thought about just adding this file to the list of
files compiled when the HEVC decoder is selected, but the end result is
the same).

- Andreas