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.
signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
