drm/rockchip: Support YUV formats with U/V swapped

The VOP in Rockchip SoCs that support YUV planes also support swapping
of the U and V elements. Supporting the swapped variants, especially
NV21, would be beneficial for multimedia applications, as the hardware
video decoders only output NV21, and supporting this pixel format in
the display pipeline would allow the decoded video frames to be output
directly.

Add support for this to support the various formats that have U/V
swapped.

Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20220114074038.2633848-1-wenst@chromium.org

diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
index ff6fff0..74562d4 100644 (file)
--- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
@@ -262,6 +262,18 @@ static bool has_rb_swapped(uint32_t format)
        }
 }
 
+static bool has_uv_swapped(uint32_t format)
+{
+       switch (format) {
+       case DRM_FORMAT_NV21:
+       case DRM_FORMAT_NV61:
+       case DRM_FORMAT_NV42:
+               return true;
+       default:
+               return false;
+       }
+}
+
 static enum vop_data_format vop_convert_format(uint32_t format)
 {
        switch (format) {
@@ -277,10 +289,13 @@ static enum vop_data_format vop_convert_format(uint32_t format)
        case DRM_FORMAT_BGR565:
                return VOP_FMT_RGB565;
        case DRM_FORMAT_NV12:
+       case DRM_FORMAT_NV21:
                return VOP_FMT_YUV420SP;
        case DRM_FORMAT_NV16:
+       case DRM_FORMAT_NV61:
                return VOP_FMT_YUV422SP;
        case DRM_FORMAT_NV24:
+       case DRM_FORMAT_NV42:
                return VOP_FMT_YUV444SP;
        default:
                DRM_ERROR("unsupported format[%08x]\n", format);
@@ -899,7 +914,7 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
        unsigned long offset;
        dma_addr_t dma_addr;
        uint32_t val;
-       bool rb_swap;
+       bool rb_swap, uv_swap;
        int win_index = VOP_WIN_TO_INDEX(vop_win);
        int format;
        int is_yuv = fb->format->is_yuv;
@@ -988,6 +1003,9 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
                                                        y2r_coefficients[i],
                                                        bt601_yuv2rgb[i]);
                }
+
+               uv_swap = has_uv_swapped(fb->format->format);
+               VOP_WIN_SET(vop, win, uv_swap, uv_swap);
        }
 
        if (win->phy->scl)

diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.h b/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
index 857d97c..3aa95fd 100644 (file)
--- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
@@ -166,6 +166,7 @@ struct vop_win_phy {
        struct vop_reg gate;
        struct vop_reg format;
        struct vop_reg rb_swap;
+       struct vop_reg uv_swap;
        struct vop_reg act_info;
        struct vop_reg dsp_info;
        struct vop_reg dsp_st;

diff --git a/drivers/gpu/drm/rockchip/rockchip_vop_reg.c b/drivers/gpu/drm/rockchip/rockchip_vop_reg.c
index 798b542..d03dd04 100644 (file)
--- a/drivers/gpu/drm/rockchip/rockchip_vop_reg.c
+++ b/drivers/gpu/drm/rockchip/rockchip_vop_reg.c
@@ -46,8 +46,11 @@ static const uint32_t formats_win_full[] = {
        DRM_FORMAT_RGB565,
        DRM_FORMAT_BGR565,
        DRM_FORMAT_NV12,
+       DRM_FORMAT_NV21,
        DRM_FORMAT_NV16,
+       DRM_FORMAT_NV61,
        DRM_FORMAT_NV24,
+       DRM_FORMAT_NV42,
 };
 
 static const uint64_t format_modifiers_win_full[] = {
@@ -272,6 +275,7 @@ static const struct vop_win_phy px30_win0_data = {
        .enable = VOP_REG(PX30_WIN0_CTRL0, 0x1, 0),
        .format = VOP_REG(PX30_WIN0_CTRL0, 0x7, 1),
        .rb_swap = VOP_REG(PX30_WIN0_CTRL0, 0x1, 12),
+       .uv_swap = VOP_REG(PX30_WIN0_CTRL0, 0x1, 15),
        .act_info = VOP_REG(PX30_WIN0_ACT_INFO, 0xffffffff, 0),
        .dsp_info = VOP_REG(PX30_WIN0_DSP_INFO, 0xffffffff, 0),
        .dsp_st = VOP_REG(PX30_WIN0_DSP_ST, 0xffffffff, 0),
@@ -291,6 +295,7 @@ static const struct vop_win_phy px30_win1_data = {
        .enable = VOP_REG(PX30_WIN1_CTRL0, 0x1, 0),
        .format = VOP_REG(PX30_WIN1_CTRL0, 0x7, 4),
        .rb_swap = VOP_REG(PX30_WIN1_CTRL0, 0x1, 12),
+       .uv_swap = VOP_REG(PX30_WIN1_CTRL0, 0x1, 15),
        .dsp_info = VOP_REG(PX30_WIN1_DSP_INFO, 0xffffffff, 0),
        .dsp_st = VOP_REG(PX30_WIN1_DSP_ST, 0xffffffff, 0),
        .yrgb_mst = VOP_REG(PX30_WIN1_MST, 0xffffffff, 0),
@@ -368,6 +373,7 @@ static const struct vop_win_phy rk3066_win0_data = {
        .enable = VOP_REG(RK3066_SYS_CTRL1, 0x1, 0),
        .format = VOP_REG(RK3066_SYS_CTRL1, 0x7, 4),
        .rb_swap = VOP_REG(RK3066_SYS_CTRL1, 0x1, 19),
+       .uv_swap = VOP_REG(RK3066_SYS_CTRL1, 0x1, 22),
        .act_info = VOP_REG(RK3066_WIN0_ACT_INFO, 0x1fff1fff, 0),
        .dsp_info = VOP_REG(RK3066_WIN0_DSP_INFO, 0x0fff0fff, 0),
        .dsp_st = VOP_REG(RK3066_WIN0_DSP_ST, 0x1fff1fff, 0),
@@ -386,6 +392,7 @@ static const struct vop_win_phy rk3066_win1_data = {
        .enable = VOP_REG(RK3066_SYS_CTRL1, 0x1, 1),
        .format = VOP_REG(RK3066_SYS_CTRL1, 0x7, 7),
        .rb_swap = VOP_REG(RK3066_SYS_CTRL1, 0x1, 23),
+       .uv_swap = VOP_REG(RK3066_SYS_CTRL1, 0x1, 26),
        .act_info = VOP_REG(RK3066_WIN1_ACT_INFO, 0x1fff1fff, 0),
        .dsp_info = VOP_REG(RK3066_WIN1_DSP_INFO, 0x0fff0fff, 0),
        .dsp_st = VOP_REG(RK3066_WIN1_DSP_ST, 0x1fff1fff, 0),
@@ -489,6 +496,7 @@ static const struct vop_win_phy rk3188_win0_data = {
        .enable = VOP_REG(RK3188_SYS_CTRL, 0x1, 0),
        .format = VOP_REG(RK3188_SYS_CTRL, 0x7, 3),
        .rb_swap = VOP_REG(RK3188_SYS_CTRL, 0x1, 15),
+       .uv_swap = VOP_REG(RK3188_SYS_CTRL, 0x1, 18),
        .act_info = VOP_REG(RK3188_WIN0_ACT_INFO, 0x1fff1fff, 0),
        .dsp_info = VOP_REG(RK3188_WIN0_DSP_INFO, 0x0fff0fff, 0),
        .dsp_st = VOP_REG(RK3188_WIN0_DSP_ST, 0x1fff1fff, 0),
@@ -619,6 +627,7 @@ static const struct vop_win_phy rk3288_win01_data = {
        .enable = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 0),
        .format = VOP_REG(RK3288_WIN0_CTRL0, 0x7, 1),
        .rb_swap = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 12),
+       .uv_swap = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 15),
        .act_info = VOP_REG(RK3288_WIN0_ACT_INFO, 0x1fff1fff, 0),
        .dsp_info = VOP_REG(RK3288_WIN0_DSP_INFO, 0x0fff0fff, 0),
        .dsp_st = VOP_REG(RK3288_WIN0_DSP_ST, 0x1fff1fff, 0),
@@ -753,6 +762,7 @@ static const struct vop_win_phy rk3368_win01_data = {
        .enable = VOP_REG(RK3368_WIN0_CTRL0, 0x1, 0),
        .format = VOP_REG(RK3368_WIN0_CTRL0, 0x7, 1),
        .rb_swap = VOP_REG(RK3368_WIN0_CTRL0, 0x1, 12),
+       .uv_swap = VOP_REG(RK3368_WIN0_CTRL0, 0x1, 15),
        .x_mir_en = VOP_REG(RK3368_WIN0_CTRL0, 0x1, 21),
        .y_mir_en = VOP_REG(RK3368_WIN0_CTRL0, 0x1, 22),
        .act_info = VOP_REG(RK3368_WIN0_ACT_INFO, 0x1fff1fff, 0),
@@ -902,6 +912,7 @@ static const struct vop_win_phy rk3399_win01_data = {
        .enable = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 0),
        .format = VOP_REG(RK3288_WIN0_CTRL0, 0x7, 1),
        .rb_swap = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 12),
+       .uv_swap = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 15),
        .x_mir_en = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 21),
        .y_mir_en = VOP_REG(RK3288_WIN0_CTRL0, 0x1, 22),
        .act_info = VOP_REG(RK3288_WIN0_ACT_INFO, 0x1fff1fff, 0),