[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4caf39049a659343450127870a10598416fd3154.camel@collabora.com>
Date: Thu, 26 Oct 2023 10:02:33 +0100
From: Christopher Obbard <chris.obbard@...labora.com>
To: Jonas Karlman <jonas@...boo.se>, Heiko Stuebner <heiko@...ech.de>,
Sandy Huang <hjc@...k-chips.com>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>
Cc: Andy Yan <andy.yan@...k-chips.com>,
dri-devel@...ts.freedesktop.org,
linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org,
kernel <kernel@...labora.com>
Subject: Re: [PATCH] drm/rockchip: vop2: Add NV20 and NV30 support
Hi Jonas,
On Wed, 2023-10-25 at 21:32 +0000, Jonas Karlman wrote:
> Add support for the 10-bit 4:2:2 and 4:4:4 formats NV20 and NV30.
>
> These formats can be tested using modetest [1]:
>
> modetest -P <plane_id>@<crtc_id>:1920x1080@<format>
>
> e.g. on a ROCK 3 Model A (rk3568):
>
> modetest -P 43@67:1920x1080@...0 -F tiles,tiles
> modetest -P 43@67:1920x1080@...0 -F smpte,smpte
>
> [1] https://gitlab.freedesktop.org/mesa/drm/-/merge_requests/329
>
> Signed-off-by: Jonas Karlman <jonas@...boo.se>
Reviewed-by: Christopher Obbard <chris.obbard@...labora.com>
Tested-by: Christopher Obbard <chris.obbard@...labora.com>
> ---
> drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 5 +++++
> drivers/gpu/drm/rockchip/rockchip_vop2_reg.c | 2 ++
> 2 files changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c
> b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c
> index ab944010fe14..592f9d726f2e 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c
> @@ -326,11 +326,14 @@ static enum vop2_data_format vop2_convert_format(u32
> format)
> case DRM_FORMAT_NV16:
> case DRM_FORMAT_NV61:
> return VOP2_FMT_YUV422SP;
> + case DRM_FORMAT_NV20:
> case DRM_FORMAT_Y210:
> return VOP2_FMT_YUV422SP_10;
> case DRM_FORMAT_NV24:
> case DRM_FORMAT_NV42:
> return VOP2_FMT_YUV444SP;
> + case DRM_FORMAT_NV30:
> + return VOP2_FMT_YUV444SP_10;
> case DRM_FORMAT_YUYV:
> case DRM_FORMAT_YVYU:
> return VOP2_FMT_VYUY422;
> @@ -415,6 +418,8 @@ static bool vop2_win_uv_swap(u32 format)
> case DRM_FORMAT_NV16:
> case DRM_FORMAT_NV24:
> case DRM_FORMAT_NV15:
> + case DRM_FORMAT_NV20:
> + case DRM_FORMAT_NV30:
> case DRM_FORMAT_YUYV:
> case DRM_FORMAT_UYVY:
> return true;
> diff --git a/drivers/gpu/drm/rockchip/rockchip_vop2_reg.c
> b/drivers/gpu/drm/rockchip/rockchip_vop2_reg.c
> index fdb48571efce..0b4280218a59 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_vop2_reg.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_vop2_reg.c
> @@ -48,8 +48,10 @@ static const uint32_t formats_rk356x_esmart[] = {
> DRM_FORMAT_NV15, /* yuv420_10bit linear mode, 2 plane, no padding
> */
> DRM_FORMAT_NV16, /* yuv422_8bit linear mode, 2 plane */
> DRM_FORMAT_NV61, /* yuv422_8bit linear mode, 2 plane */
> + DRM_FORMAT_NV20, /* yuv422_10bit linear mode, 2 plane, no padding
> */
> DRM_FORMAT_NV24, /* yuv444_8bit linear mode, 2 plane */
> DRM_FORMAT_NV42, /* yuv444_8bit linear mode, 2 plane */
> + DRM_FORMAT_NV30, /* yuv444_10bit linear mode, 2 plane, no padding
> */
> DRM_FORMAT_YVYU, /* yuv422_8bit[YVYU] linear mode */
> DRM_FORMAT_VYUY, /* yuv422_8bit[VYUY] linear mode */
> };
Powered by blists - more mailing lists