Re: [PATCH 16/28] gpu: ipu-v3: Add ipu_stride_to_bytes()

2014-07-31 Thread Philipp Zabel
Am Mittwoch, den 25.06.2014, 18:05 -0700 schrieb Steve Longerbeam:
 Adds ipu_stride_to_bytes(), which converts a pixel stride to bytes,
 suitable for passing to cpmem.

This is not IPU specific. You already have the bytesperline information
from the V4L2 driver or have to calculate it there, and that shouldn't
be done by calling into IPU core code.

regards
Philipp

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 16/28] gpu: ipu-v3: Add ipu_stride_to_bytes()

2014-06-25 Thread Steve Longerbeam
Adds ipu_stride_to_bytes(), which converts a pixel stride to bytes,
suitable for passing to cpmem.

Signed-off-by: Steve Longerbeam steve_longerb...@mentor.com
---
 drivers/gpu/ipu-v3/ipu-common.c |   30 ++
 include/video/imx-ipu-v3.h  |1 +
 2 files changed, 31 insertions(+)

diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c
index 7701974..30afef4 100644
--- a/drivers/gpu/ipu-v3/ipu-common.c
+++ b/drivers/gpu/ipu-v3/ipu-common.c
@@ -576,6 +576,36 @@ enum ipu_color_space ipu_mbus_code_to_colorspace(u32 
mbus_code)
 }
 EXPORT_SYMBOL_GPL(ipu_mbus_code_to_colorspace);
 
+int ipu_stride_to_bytes(u32 pixel_stride, u32 pixelformat)
+{
+   switch (pixelformat) {
+   case V4L2_PIX_FMT_YUV420:
+   case V4L2_PIX_FMT_YVU420:
+   /*
+* for the planar YUV formats, the stride passed to
+* cpmem must be the stride in bytes of the Y plane.
+* And all the planar YUV formats have an 8-bit
+* Y component.
+*/
+   return (8 * pixel_stride)  3;
+   case V4L2_PIX_FMT_RGB565:
+   case V4L2_PIX_FMT_YUYV:
+   case V4L2_PIX_FMT_UYVY:
+   return (16 * pixel_stride)  3;
+   case V4L2_PIX_FMT_BGR24:
+   case V4L2_PIX_FMT_RGB24:
+   return (24 * pixel_stride)  3;
+   case V4L2_PIX_FMT_BGR32:
+   case V4L2_PIX_FMT_RGB32:
+   return (32 * pixel_stride)  3;
+   default:
+   break;
+   }
+
+   return -EINVAL;
+}
+EXPORT_SYMBOL_GPL(ipu_stride_to_bytes);
+
 int ipu_degrees_to_rot_mode(enum ipu_rotate_mode *mode, int degrees,
bool hflip, bool vflip)
 {
diff --git a/include/video/imx-ipu-v3.h b/include/video/imx-ipu-v3.h
index 9ed1c75..17a0bb8 100644
--- a/include/video/imx-ipu-v3.h
+++ b/include/video/imx-ipu-v3.h
@@ -480,6 +480,7 @@ int ipu_cpmem_set_image(struct ipu_ch_param __iomem *cpmem,
 enum ipu_color_space ipu_drm_fourcc_to_colorspace(u32 drm_fourcc);
 enum ipu_color_space ipu_pixelformat_to_colorspace(u32 pixelformat);
 enum ipu_color_space ipu_mbus_code_to_colorspace(u32 mbus_code);
+int ipu_stride_to_bytes(u32 pixel_stride, u32 pixelformat);
 bool ipu_pixelformat_is_planar(u32 pixelformat);
 int ipu_degrees_to_rot_mode(enum ipu_rotate_mode *mode, int degrees,
bool hflip, bool vflip);
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html