[FFmpeg-devel] [PATCH 2/9] swscale: Add swscale input support for Y210LE

[Fu, Linjie]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> Carl Eugen Hoyos
> Sent: Thursday, January 16, 2020 02:30
> To: FFmpeg development discussions and patches <ffmpeg-
> devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 2/9] swscale: Add swscale input support
> for Y210LE
> 
> Am Mi., 15. Jan. 2020 um 07:59 Uhr schrieb Linjie Fu <linjie.fu at intel.com>:
> >
> > Add swscale input support for Y210LE, output support and fate
> > test could be added later if there is requirement for software
> > CSC to this packed format.
> 
> Please confirm that this works as expected on big-endian hardware.
> 
If I understood correctly,  the concern is whether it works for Y210 software scale
on big-endian hardware, and whether it would break fate.

For Y210 on big-endian hardware, it'll be defined as Y210BE in pixfmt.h:
#define AV_PIX_FMT_Y210       AV_PIX_FMT_NE(Y210BE,  Y210LE)

And in the query of format_entries[] in utils.c, AV_PIX_FMT_Y210BE is not
declared to have the capability to support Input or Output in swscale. 

So Y210 software scale is not going to be supported on big-endian hardware.

Hence this won't break fate on big-endian hardware for Y210 IMHO, because this patch
didn't declare it has the capability for either input or output support.

If there is something wrong with my understanding, please correct me.

Thanks.