Ok thanks, Timo. So I’ve split this into two patches and revised as per the discussions and they are attached here.
The only thing to be decided is whether my conversion code to enable YUV420P10 support should be included in this or not. It’s in the attached patch but I’m happy to remove it if necessary. Regards Oliver
0001-NVENC-added-support-for-10-bit-HEVC-encoding.patch
Description: Binary data
0002-NVENC-added-support-for-rate-control-lookahead.patch
Description: Binary data
> On 24 Aug 2016, at 12:04, Timo Rothenpieler <t...@rothenpieler.org> wrote: > > Am 24.08.2016 um 10:21 schrieb Oliver Collyer: >>> In any case, please split the rate control patch from the 10bit patch. >>> >> >> Just double-checking this - both changes require a bump of the minimum NVENC >> version to 7. Do you still want them as separate patches or does this tie >> them together? If they are to be separate patches then obviously one of them >> will need to be applied first, so there is a dependency between them. > > Just bump it with the first patch. > Also remember to bump lavc micro version. > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
