[vlc-devel] [PATCH] vaapi: add YUVJ420P pixfmt

[Steve Lhomme]

On 2019-10-18 0:29, Mathieu Velten wrote:
> Thanks for merging the patch, would it be possible to backport to branch 3 ?

Yes, I'll do it.

>  > I see what you mean. I think it comes from this code:
>  > https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/h264_slice.c#L832
>  >
> 
> Indeed good catch. After a quick search AV_PIX_FMT_YUVJ420P is 
> deprecated, but still heavily referenced inside ffmpeg ^^
> 
>  > I do not reproduce the issue with the downloaded file from that link,
>  > maybe it's not the original file you uploaded. Anyway, the fix is
>  > needed, it's already present in VDPAU.
> 
> Hum this is because Google Photos is stupid apparently :)
> It seems like I shared the video before it transcodes in all res so it 
> only shared the lower ones that were already encoded...
> Here is a new link in case you want it for a sample collection (you need 
> to select the higher 4k res before download to have the original file):
> https://photos.app.goo.gl/EJHekpZQr5q4ii3A9

Yes, indeed. It's good to have odd samples. This one is even more odd as 
it has BT.2020 transfer characterstics, in other words HDR, with non-HDR 
colour primaries.