[FFmpeg-devel] [PATCH 1/5] avutil: Add YUV444P10_LSB and YUV444P12_LSB pixel formats

Timo Rothenpieler timo at rothenpieler.org
Mon Oct 8 22:59:11 EEST 2018


>> So, to be able to use these hardware compatible formats, we need
>> definitions for them in ffmpeg. Right now, I need this for nvdec,
>> but Vulkan also uses the same format definitions.
> 
> Sorry if this was already done and I forgot but please explain why
> you cannot use YUV444P16 for this use-case.
> 
> If the only thing missing is libavfilter understanding bits_per_raw_sample
> we should add it there: It is needed in any case and would save us a few
> pix_fmts and could speed up many use cases compared to your solution.

It's pretty much all of ffmpeg and all client applications understanding it.
First of all, bits_per_raw_sample, or an equivalent bit depth field, is 
missing from AVFrame.
"In" avfilter it's primarily swscale I guess, but it also affects each 
and every filter supporting YUV444P16.
Also, for encoders, it would mean they need a way to indicate what bit 
depths they can take, and that also has to be per pix_fmt they support.
Then all of avcodec, ffmpeg.c, ... needs support for that.
It's quite a deep rabit hole of changes with quite a good chance of 
something going horribly wrong in the process. Probably even in more 
places than just the ones mentioned here.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4538 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181008/e454cb42/attachment.bin>


More information about the ffmpeg-devel mailing list