Hi Timo Thankyou for the clarification.
Attached are what should be the final versions of these patches then, with the support for YUV420P10 (and related conversion code) now dropped. 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 25 Aug 2016, at 13:48, Timo Rothenpieler <t...@rothenpieler.org> wrote: > > Am 24.08.2016 um 12:30 schrieb Oliver Collyer: >> 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. > > I'm not a fan of format-conversion code in nvenc. That's the job of swscale. > If a needed conversion is missing/performs poorly, it should be fixed in > sws instead. > >> Regards >> >> Oliver >> > > Unfortunately I'm still on my old GTX760, so I can't test all the > hevc/10bit stuff. > The patch looks Ok though and should generally be fine to merge minus > the format-conversion. > > Might have to get myself an intermediary GTX1060 to upgrade my old PC > once again. > > > _______________________________________________ > 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
