[FFmpeg-devel] [PATCH v2 01/12] lavc/videotoolboxenc: Use shared pixel buffer pool

[Richard Kern]

> On Apr 26, 2016, at 8:14 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> 
> Richard Kern <kernrj <at> gmail.com> writes:
> 
>>>> static const enum AVPixelFormat pix_fmts[] = {
>>>>    AV_PIX_FMT_NV12,
>>>> -#if !TARGET_OS_IPHONE
>>>>    AV_PIX_FMT_YUV420P,
>>>> -#endif
>>>>    AV_PIX_FMT_NONE
>>> 
>>> Sorry: How is this related?
>> Using a shared pixel buffer pool also sets up an internal 
>> color converter when needed.
> 
> And is this generally wanted?
> (I assume so.)
The pool? Probably. Using it reduces memcpy calls in the media server process.

I dug into this a little - yuv420p is supported natively (at least on some devices), so it shouldn’t have been excluded in the first place.

> 
> Carl Eugen
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel