[libcamera-devel] [PATCH 4/4] cam: sdl_sink: Add NV12 texture support

[Eric Curtin]

On Sun, 7 Aug 2022 at 03:17, Laurent Pinchart
<laurent.pinchart at ideasonboard.com> wrote:
>
> Extend the SDL sink with support for NV12 textures, useful on platforms
> that don't support packed YUYV formats.
>

LGTM.

Reviewed-by: Eric Curtin <ecurtin at redhat.com>

> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/cam/sdl_sink.cpp        |  3 +++
>  src/cam/sdl_texture_yuv.cpp | 13 ++++++++++++-
>  src/cam/sdl_texture_yuv.h   |  9 ++++++++-
>  3 files changed, 23 insertions(+), 2 deletions(-)
>
> diff --git a/src/cam/sdl_sink.cpp b/src/cam/sdl_sink.cpp
> index 9675cf275186..23fe2b1e9f05 100644
> --- a/src/cam/sdl_sink.cpp
> +++ b/src/cam/sdl_sink.cpp
> @@ -67,6 +67,9 @@ int SDLSink::configure(const libcamera::CameraConfiguration &config)
>                 texture_ = std::make_unique<SDLTextureMJPG>(rect_);
>                 break;
>  #endif
> +       case libcamera::formats::NV12:
> +               texture_ = std::make_unique<SDLTextureNV12>(rect_, cfg.stride);
> +               break;
>         case libcamera::formats::YUYV:
>                 texture_ = std::make_unique<SDLTextureYUYV>(rect_, cfg.stride);
>                 break;
> diff --git a/src/cam/sdl_texture_yuv.cpp b/src/cam/sdl_texture_yuv.cpp
> index a5721182a68b..431e836d2d2c 100644
> --- a/src/cam/sdl_texture_yuv.cpp
> +++ b/src/cam/sdl_texture_yuv.cpp
> @@ -2,13 +2,24 @@
>  /*
>   * Copyright (C) 2022, Ideas on Board Oy
>   *
> - * sdl_texture_yuv.cpp - SDL Texture YUYV
> + * sdl_texture_yuv.cpp - SDL YUV Textures
>   */
>
>  #include "sdl_texture_yuv.h"
>
>  using namespace libcamera;
>
> +SDLTextureNV12::SDLTextureNV12(const SDL_Rect &rect, unsigned int stride)
> +       : SDLTexture(rect, SDL_PIXELFORMAT_NV12, stride)
> +{
> +}
> +
> +void SDLTextureNV12::update(const std::vector<libcamera::Span<const uint8_t>> &data)
> +{
> +       SDL_UpdateNVTexture(ptr_, &rect_, data[0].data(), pitch_,
> +                           data[1].data(), pitch_);
> +}
> +
>  SDLTextureYUYV::SDLTextureYUYV(const SDL_Rect &rect, unsigned int stride)
>         : SDLTexture(rect, SDL_PIXELFORMAT_YUY2, stride)
>  {
> diff --git a/src/cam/sdl_texture_yuv.h b/src/cam/sdl_texture_yuv.h
> index c9130298b91d..9da611fe0e36 100644
> --- a/src/cam/sdl_texture_yuv.h
> +++ b/src/cam/sdl_texture_yuv.h
> @@ -2,13 +2,20 @@
>  /*
>   * Copyright (C) 2022, Ideas on Board Oy
>   *
> - * sdl_texture_yuyv.h - SDL Texture YUYV
> + * sdl_texture_yuyv.h - SDL YUV Textures
>   */
>
>  #pragma once
>
>  #include "sdl_texture.h"
>
> +class SDLTextureNV12 : public SDLTexture
> +{
> +public:
> +       SDLTextureNV12(const SDL_Rect &rect, unsigned int stride);
> +       void update(const std::vector<libcamera::Span<const uint8_t>> &data) override;
> +};
> +
>  class SDLTextureYUYV : public SDLTexture
>  {
>  public:
> --
> Regards,
>
> Laurent Pinchart
>