[FFmpeg-devel] [PATCH 03/15] lavu/videotoolbox: add 422 and 444 pixel format mappings

Ridley Combs rcombs at rcombs.me
Tue Nov 16 05:03:19 EET 2021



> On Nov 15, 2021, at 19:35, Wang Bin <wbsecg1 at gmail.com> wrote:
> 
>> 
>> +#if HAVE_KCVPIXELFORMATTYPE_422YPCBCR10BIPLANARVIDEORANGE
>> +    { kCVPixelFormatType_422YpCbCr10BiPlanarVideoRange, false,
>> AV_PIX_FMT_NV20 },
>> +    { kCVPixelFormatType_422YpCbCr10BiPlanarFullRange,  true,
>> AV_PIX_FMT_NV20 },
>> +#endif
>> 
> 
> It's p210, not nv20.

I didn't add a P210 format (since that would've been equivalent to the existing NV20), only P410/P216/P416. I guess I could add P210 as an alias with appropriate enum values and macros? In which case maybe defining the rest of the group (P008 for NV12, P208 for NV16, P408 for NV24) would be worthwhile.

> 
> Regards
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".



More information about the ffmpeg-devel mailing list