[FFmpeg-devel] [PATCH 2/8] swscale: Add swscale input support for Y210

[Carl Eugen Hoyos]

Am So., 29. Dez. 2019 um 17:40 Uhr schrieb Linjie Fu <linjie.fu at intel.com>:
>
> Add swscale input support for Y210, output support and fate
> test could be added later if there is requirement for software
> CSC to this packed format.
>
> Signed-off-by: Linjie Fu <linjie.fu at intel.com>
> ---
>  libswscale/input.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
>  libswscale/utils.c |  2 ++
>  2 files changed, 50 insertions(+)
>
> diff --git a/libswscale/input.c b/libswscale/input.c
> index 064f8da..f79b983 100644
> --- a/libswscale/input.c
> +++ b/libswscale/input.c
> @@ -552,6 +552,42 @@ static void yvy2ToUV_c(uint8_t *dstU, uint8_t *dstV, const uint8_t *unused0, con
>      av_assert1(src1 == src2);
>  }
>
> +static void y210le_UV_c(uint8_t *dstU, uint8_t *dstV, const uint8_t *unused0, const uint8_t *src,
> +                        const uint8_t *unused1, int width, uint32_t *unused2)
> +{
> +    int i;
> +    for (i = 0; i < width; i++) {
> +        AV_WN16(dstU + i * 2, AV_RL16(src + i * 8 + 2) >> 6);
> +        AV_WN16(dstV + i * 2, AV_RL16(src + i * 8 + 6) >> 6);
> +    }
> +}
> +
> +static void y210be_UV_c(uint8_t *dstU, uint8_t *dstV, const uint8_t *unused0, const uint8_t *src,
> +                        const uint8_t *unused1, int width, uint32_t *unused2)
> +{
> +    int i;
> +    for (i = 0; i < width; i++) {
> +        AV_WN16(dstU + i * 2, AV_RB16(src + i * 8 + 2) >> 6);
> +        AV_WN16(dstV + i * 2, AV_RB16(src + i * 8 + 6) >> 6);
> +    }
> +}
> +
> +static void y210le_Y_c(uint8_t *dst, const uint8_t *src, const uint8_t *unused0,
> +                       const uint8_t *unused1, int width, uint32_t *unused2)
> +{
> +    int i;
> +    for (i = 0; i < width; i++)
> +        AV_WN16(dst + i * 2, AV_RL16(src + i * 4) >> 6);
> +}
> +
> +static void y210be_Y_c(uint8_t *dst, const uint8_t *src, const uint8_t *unused0,
> +                       const uint8_t *unused1, int width, uint32_t *unused2)
> +{
> +    int i;
> +    for (i = 0; i < width; i++)
> +        AV_WN16(dst + i * 2 ,AV_RB16(src + i * 4) >> 6);
> +}
> +
>  static void bswap16Y_c(uint8_t *_dst, const uint8_t *_src, const uint8_t *unused1, const uint8_t *unused2, int width,
>                         uint32_t *unused)
>  {
> @@ -1154,6 +1190,12 @@ av_cold void ff_sws_init_input_funcs(SwsContext *c)
>      case AV_PIX_FMT_P016BE:
>          c->chrToYV12 = p016BEToUV_c;
>          break;
> +    case AV_PIX_FMT_Y210LE:
> +        c->chrToYV12 = y210le_UV_c;
> +        break;
> +    case AV_PIX_FMT_Y210BE:
> +        c->chrToYV12 = y210be_UV_c;
> +        break;
>      }
>      if (c->chrSrcHSubSample) {
>          switch (srcFormat) {
> @@ -1586,6 +1628,12 @@ av_cold void ff_sws_init_input_funcs(SwsContext *c)
>          c->lumToYV12 = grayf32ToY16_bswap_c;
>  #endif
>          break;
> +    case AV_PIX_FMT_Y210LE:
> +        c->lumToYV12 = y210le_Y_c;
> +        break;
> +    case AV_PIX_FMT_Y210BE:
> +        c->chrToYV12 = y210be_Y_c;
> +        break;
>      }
>      if (c->needAlpha) {
>          if (is16BPS(srcFormat) || isNBPS(srcFormat)) {
> diff --git a/libswscale/utils.c b/libswscale/utils.c
> index 57c4fd2..bff498f 100644
> --- a/libswscale/utils.c
> +++ b/libswscale/utils.c
> @@ -266,6 +266,8 @@ static const FormatEntry format_entries[AV_PIX_FMT_NB] = {
>      [AV_PIX_FMT_YUVA444P12LE] = { 1, 1 },
>      [AV_PIX_FMT_NV24]        = { 1, 1 },
>      [AV_PIX_FMT_NV42]        = { 1, 1 },
> +    [AV_PIX_FMT_Y210BE]      = { 1, 0 },
> +    [AV_PIX_FMT_Y210LE]      = { 1, 0 },

Am I correct that this functions only work on LE because the vaapi drivers
only exist for LE?
Or do I misunderstand the code?

Carl Eugen