[PATCH v2][for_v3.5] media: mx2_camera: Fix mbus format handling
javier Martin
javier.martin at vista-silicon.com
Fri Jun 1 04:50:24 EDT 2012
Sorry, forget about this patch.
I've just read that there is a merge conflict too.
I will send a v3 to address this also.
On 1 June 2012 10:28, Javier Martin <javier.martin at vista-silicon.com> wrote:
> Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags
> so that the driver can negotiate with the attached sensor
> whether the mbus format needs convertion from UYUV to YUYV
> or not.
>
> Signed-off-by: Javier Martin <javier.martin at vista-silicon.com>
> ---
> Previous version of this patch was requested to be reverted by
> Guennadi.
>
> I'm sending a new version which fixes the issue; when mbus fmt
> is not found let the driver use default pass-through mode.
>
> Could this patch replace the old version in for_v3.5? It is part of
> the following series where patches 1 and 2 have been applied:
>
> [PATCH v2 1/3] media: tvp5150: Fix mbus format.
> [PATCH v2 2/3] i.MX27: visstrim_m10: Remove use of MX2_CAMERA_SWAP16.
> [PATCH v2 3/3] media: mx2_camera: Fix mbus format handling.
>
> ---
> arch/arm/plat-mxc/include/mach/mx2_cam.h | 2 -
> drivers/media/video/mx2_camera.c | 49 ++++++++++++++++++++++++++---
> 2 files changed, 44 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/plat-mxc/include/mach/mx2_cam.h b/arch/arm/plat-mxc/include/mach/mx2_cam.h
> index 3c080a3..7ded6f1 100644
> --- a/arch/arm/plat-mxc/include/mach/mx2_cam.h
> +++ b/arch/arm/plat-mxc/include/mach/mx2_cam.h
> @@ -23,7 +23,6 @@
> #ifndef __MACH_MX2_CAM_H_
> #define __MACH_MX2_CAM_H_
>
> -#define MX2_CAMERA_SWAP16 (1 << 0)
> #define MX2_CAMERA_EXT_VSYNC (1 << 1)
> #define MX2_CAMERA_CCIR (1 << 2)
> #define MX2_CAMERA_CCIR_INTERLACE (1 << 3)
> @@ -31,7 +30,6 @@
> #define MX2_CAMERA_GATED_CLOCK (1 << 5)
> #define MX2_CAMERA_INV_DATA (1 << 6)
> #define MX2_CAMERA_PCLK_SAMPLE_RISING (1 << 7)
> -#define MX2_CAMERA_PACK_DIR_MSB (1 << 8)
>
> /**
> * struct mx2_camera_platform_data - optional platform data for mx2_camera
> diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c
> index 18afaee..bdbb7c9 100644
> --- a/drivers/media/video/mx2_camera.c
> +++ b/drivers/media/video/mx2_camera.c
> @@ -344,6 +344,19 @@ static struct mx2_fmt_cfg mx27_emma_prp_table[] = {
> PRP_INTR_CH2OVF,
> }
> },
> + {
> + .in_fmt = V4L2_MBUS_FMT_UYVY8_2X8,
> + .out_fmt = V4L2_PIX_FMT_YUV420,
> + .cfg = {
> + .channel = 2,
> + .in_fmt = PRP_CNTL_DATA_IN_YUV422,
> + .out_fmt = PRP_CNTL_CH2_OUT_YUV420,
> + .src_pixel = 0x22000888, /* YUV422 (YUYV) */
> + .irq_flags = PRP_INTR_RDERR | PRP_INTR_CH2WERR |
> + PRP_INTR_CH2FC | PRP_INTR_LBOVF |
> + PRP_INTR_CH2OVF,
> + }
> + },
> };
>
> static struct mx2_fmt_cfg *mx27_emma_prp_get_format(
> @@ -980,6 +993,7 @@ static int mx2_camera_set_bus_param(struct soc_camera_device *icd)
> struct soc_camera_host *ici = to_soc_camera_host(icd->parent);
> struct mx2_camera_dev *pcdev = ici->priv;
> struct v4l2_mbus_config cfg = {.type = V4L2_MBUS_PARALLEL,};
> + const struct soc_camera_format_xlate *xlate;
> unsigned long common_flags;
> int ret;
> int bytesperline;
> @@ -1024,14 +1038,28 @@ static int mx2_camera_set_bus_param(struct soc_camera_device *icd)
> return ret;
> }
>
> + xlate = soc_camera_xlate_by_fourcc(icd, pixfmt);
> + if (!xlate) {
> + dev_warn(icd->parent, "Format %x not found\n", pixfmt);
> + return -EINVAL;
> + }
> +
> + if (xlate->code == V4L2_MBUS_FMT_YUYV8_2X8) {
> + csicr1 |= CSICR1_PACK_DIR;
> + csicr1 &= ~CSICR1_SWAP16_EN;
> + dev_dbg(icd->parent, "already yuyv format, don't convert\n");
> + } else if (xlate->code == V4L2_MBUS_FMT_UYVY8_2X8) {
> + csicr1 &= ~CSICR1_PACK_DIR;
> + csicr1 |= CSICR1_SWAP16_EN;
> + dev_dbg(icd->parent, "convert uyvy mbus format into yuyv\n");
> + }
> +
> if (common_flags & V4L2_MBUS_PCLK_SAMPLE_RISING)
> csicr1 |= CSICR1_REDGE;
> if (common_flags & V4L2_MBUS_VSYNC_ACTIVE_HIGH)
> csicr1 |= CSICR1_SOF_POL;
> if (common_flags & V4L2_MBUS_HSYNC_ACTIVE_HIGH)
> csicr1 |= CSICR1_HSYNC_POL;
> - if (pcdev->platform_flags & MX2_CAMERA_SWAP16)
> - csicr1 |= CSICR1_SWAP16_EN;
> if (pcdev->platform_flags & MX2_CAMERA_EXT_VSYNC)
> csicr1 |= CSICR1_EXT_VSYNC;
> if (pcdev->platform_flags & MX2_CAMERA_CCIR)
> @@ -1042,8 +1070,6 @@ static int mx2_camera_set_bus_param(struct soc_camera_device *icd)
> csicr1 |= CSICR1_GCLK_MODE;
> if (pcdev->platform_flags & MX2_CAMERA_INV_DATA)
> csicr1 |= CSICR1_INV_DATA;
> - if (pcdev->platform_flags & MX2_CAMERA_PACK_DIR_MSB)
> - csicr1 |= CSICR1_PACK_DIR;
>
> pcdev->csicr1 = csicr1;
>
> @@ -1118,7 +1144,8 @@ static int mx2_camera_get_formats(struct soc_camera_device *icd,
> return 0;
> }
>
> - if (code == V4L2_MBUS_FMT_YUYV8_2X8) {
> + if (code == V4L2_MBUS_FMT_YUYV8_2X8 ||
> + code == V4L2_MBUS_FMT_UYVY8_2X8) {
> formats++;
> if (xlate) {
> /*
> @@ -1134,6 +1161,18 @@ static int mx2_camera_get_formats(struct soc_camera_device *icd,
> }
> }
>
> + if (code == V4L2_MBUS_FMT_UYVY8_2X8) {
> + formats++;
> + if (xlate) {
> + xlate->host_fmt =
> + soc_mbus_get_fmtdesc(V4L2_MBUS_FMT_YUYV8_2X8);
> + xlate->code = code;
> + dev_dbg(dev, "Providing host format %s for sensor code %d\n",
> + xlate->host_fmt->name, code);
> + xlate++;
> + }
> + }
> +
> /* Generic pass-trough */
> formats++;
> if (xlate) {
> --
> 1.7.0.4
>
--
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com
More information about the linux-arm-kernel
mailing list