[PATCH 7/9] gpu: ipu-v3: add YUV 4:4:4 support

[Ying Liu]

On Wed, Oct 19, 2016 at 12:07 AM, Philipp Zabel <p.zabel at pengutronix.de> wrote:
> The IDMAC does support reading and writing DRM_FORMAT_YUV444 and
> DRM_FORMAT_YVU444.
>
> Signed-off-by: Philipp Zabel <p.zabel at pengutronix.de>
> ---
>  drivers/gpu/ipu-v3/ipu-common.c |  2 ++
>  drivers/gpu/ipu-v3/ipu-cpmem.c  | 17 +++++++++++++++++
>  2 files changed, 19 insertions(+)
>
> diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c
> index b9539f7..b7d7bd6 100644
> --- a/drivers/gpu/ipu-v3/ipu-common.c
> +++ b/drivers/gpu/ipu-v3/ipu-common.c
> @@ -88,6 +88,8 @@ enum ipu_color_space ipu_drm_fourcc_to_colorspace(u32 drm_fourcc)
>         case DRM_FORMAT_YVU420:
>         case DRM_FORMAT_YUV422:
>         case DRM_FORMAT_YVU422:
> +       case DRM_FORMAT_YUV444:
> +       case DRM_FORMAT_YVU444:
>         case DRM_FORMAT_NV12:
>         case DRM_FORMAT_NV21:
>         case DRM_FORMAT_NV16:
> diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c b/drivers/gpu/ipu-v3/ipu-cpmem.c
> index e6b8bae..4db7475 100644
> --- a/drivers/gpu/ipu-v3/ipu-cpmem.c
> +++ b/drivers/gpu/ipu-v3/ipu-cpmem.c
> @@ -445,6 +445,16 @@ void ipu_cpmem_set_yuv_planar(struct ipuv3_channel *ch,
>                 v_offset = stride * height;
>                 u_offset = v_offset + (uv_stride * height);
>                 break;
> +       case DRM_FORMAT_YUV444:
> +               uv_stride = stride;
> +               u_offset = stride * height;
> +               v_offset = u_offset + (uv_stride * height);
> +               break;
> +       case DRM_FORMAT_YVU444:
> +               uv_stride = stride;
> +               v_offset = stride * height;
> +               u_offset = v_offset + (uv_stride * height);
> +               break;

This function can be removed probably as I mentioned in comments
for patch 6/9.

With that addressed:
Acked-by: Liu Ying <gnuiyl at gmail.com>

Regards,
Liu Ying


>         case DRM_FORMAT_NV12:
>         case DRM_FORMAT_NV16:
>                 uv_stride = stride;
> @@ -595,6 +605,13 @@ int ipu_cpmem_set_fmt(struct ipuv3_channel *ch, u32 drm_fourcc)
>                 /* burst size */
>                 ipu_ch_param_write_field(ch, IPU_FIELD_NPB, 31);
>                 break;
> +       case DRM_FORMAT_YUV444:
> +       case DRM_FORMAT_YVU444:
> +               /* pix format */
> +               ipu_ch_param_write_field(ch, IPU_FIELD_PFS, 0);
> +               /* burst size */
> +               ipu_ch_param_write_field(ch, IPU_FIELD_NPB, 31);
> +               break;
>         case DRM_FORMAT_NV12:
>                 /* pix format */
>                 ipu_ch_param_write_field(ch, IPU_FIELD_PFS, 4);
> --
> 2.9.3
>