Re: [OMAPZOOM][PATCH 3/6] IMX046: Add support for Sony imx046 sensor.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dominic!

First a small thing: please post to the new linux-media list in the future 
as this list is being phased out.

On Friday 30 January 2009 01:53:38 Dominic Curran wrote:
> From: Dominic Curran <dcurran@xxxxxx>
> Subject: [OMAPZOOM][PATCH 3/6] IMX046: Add support for Sony imx046
> sensor.
>
> This patch adds the driver files for the Sony IMX046 8MP camera sensor.
> Driver sets up the sensor to send frame data via the MIPI CSI2 i/f.
> Sensor is setup to output the following base sizes:
>  - 3280 x 2464 (8MP)
>  - 3280 x 616  (2MP)
>  - 820  x 616
> Sensor's output image format is Bayer10 (GrR/BGb).
>
> Driver has V4L2 controls for:
>  - Exposure
>  - Analog Gain
>
> Signed-off-by: Greg Hofer <greg.hofer@xxxxxx>
> Signed-off-by: Dominic Curran <dcurran@xxxxxx>
> ---
>  drivers/media/video/Kconfig  |    8
>  drivers/media/video/Makefile |    1
>  drivers/media/video/imx046.c | 1635
> +++++++++++++++++++++++++++++++++++++++++++ drivers/media/video/imx046.h
> |  326 ++++++++
>  4 files changed, 1970 insertions(+)
>  create mode 100644 drivers/media/video/imx046.c
>  create mode 100644 drivers/media/video/imx046.h
>
> Index: omapzoom04/drivers/media/video/Kconfig
> ===================================================================
> --- omapzoom04.orig/drivers/media/video/Kconfig
> +++ omapzoom04/drivers/media/video/Kconfig
> @@ -334,6 +334,14 @@ config VIDEO_OV3640_CSI2
>  	  This enables the use of the CSI2 serial bus for the ov3640
>  	  camera.
>
> +config VIDEO_IMX046
> +	tristate "Sony IMX046 sensor driver (8MP)"
> +	depends on I2C && VIDEO_V4L2
> +	---help---
> +	  This is a Video4Linux2 sensor-level driver for the Sony
> +	  IMX046 camera.  It is currently working with the TI OMAP3
> +          camera controller.
> +

Does this need an OMAP3 dependency? Or is it fully independent from omap?

>  config VIDEO_SAA7110
>  	tristate "Philips SAA7110 video decoder"
>  	depends on VIDEO_V4L1 && I2C
> Index: omapzoom04/drivers/media/video/Makefile
> ===================================================================
> --- omapzoom04.orig/drivers/media/video/Makefile
> +++ omapzoom04/drivers/media/video/Makefile
> @@ -115,6 +115,7 @@ obj-$(CONFIG_VIDEO_OV9640)	+= ov9640.o
>  obj-$(CONFIG_VIDEO_MT9P012)	+= mt9p012.o
>  obj-$(CONFIG_VIDEO_DW9710) += dw9710.o
>  obj-$(CONFIG_VIDEO_OV3640)     += ov3640.o
> +obj-$(CONFIG_VIDEO_IMX046)     += imx046.o
>
>  obj-$(CONFIG_USB_DABUSB)        += dabusb.o
>  obj-$(CONFIG_USB_OV511)         += ov511.o
> Index: omapzoom04/drivers/media/video/imx046.c
> ===================================================================
> --- /dev/null
> +++ omapzoom04/drivers/media/video/imx046.c
> @@ -0,0 +1,1635 @@
> +/*
> + * drivers/media/video/imx046.c
> + *
> + * Sony imx046 sensor driver
> + *
> + *
> + * Copyright (C) 2008 Hewlett Packard
> + *
> + * Leverage mt9p012.c
> + *
> + * This file is licensed under the terms of the GNU General Public
> License + * version 2. This program is licensed "as is" without any
> warranty of any + * kind, whether express or implied.
> + */
> +
> +#include <linux/i2c.h>
> +#include <linux/delay.h>
> +#include <media/v4l2-int-device.h>

A general note on the usage of v4l2-int-device.h: this will be phased out 
soon in favor of v4l2-subdev (see 
Documentation/video4linux/v4l2-framework.txt, introduced in 2.6.29).

You might want to discuss this with Vaibhav Hiremath regarding the timescale 
of this conversion and whether it is better to wait until omap has been 
converted before merging this driver. I'm willing to accept the driver 
using the v4l2-int-device interface as long as I get the assurance that it 
will be converted later.

Regards,

	Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux