Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: fixed jp2k_rsiz and 240M matrix

2021-06-05 Thread Valerii Zapodovnikov
Of course it is a PeRfEct copy paste, I should have mentioned that (and I asked Rémi to send the patch about RSIZ for XYZ, BTW). Also what this patch actually fixes is encoding 170M, not 240M. Will fix that too. Also should note that SMPTE ST 422:2019 is now free of charge. Yeah! __

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: fixed jp2k_rsiz and 240M matrix

2021-06-05 Thread Marton Balint
On Sat, 5 Jun 2021, Marton Balint wrote: On Sat, 5 Jun 2021, Valerii Zapodovnikov wrote: -static const uint8_t mxf_jp2k_rsiz[] = { 0x06,0x0e,0x2b,0x34,0x02,0x05,0x01,0x01,0x0d,0x01,0x02,0x01,0x01,0x02,0x01,0x00 }; +static const uint8_t mxf_jp2k_rsiz[]

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: fixed jp2k_rsiz and 240M matrix

2021-06-05 Thread Marton Balint
On Sat, 5 Jun 2021, Valerii Zapodovnikov wrote: Again. 240M matrix is different from BT.601! And 170M is the same as BT.601. It is primaries that are the same in 240M and 170M, as for jp2k_rsiz see page 10 of S422M-2006. Yes, IT IS THERE. --- libavformat/mxf.c| 2 +- libavformat/mxfdec.c |