RE: [PATCH 2/5] [media] v4l: Add mt9t111 sensor driver

2011-09-27 Thread Ravi, Deepthy
Hi,
 
 From: Gary Thomas [g...@mlbassoc.com]
 Sent: Tuesday, September 27, 2011 7:21 PM
 To: Ravi, Deepthy
 Cc: laurent.pinch...@ideasonboard.com; mche...@infradead.org; 
 t...@atomide.com; Hiremath, Vaibhav; linux-me...@vger.kernel.org; 
 li...@arm.linux.org.uk; linux-arm-ker...@lists.infradead.org; 
 kyungmin.p...@samsung.com; hverk...@xs4all.nl; m.szyprow...@samsung.com; 
 g.liakhovet...@gmx.de; Shilimkar, Santosh; khil...@deeprootsystems.com; 
 linux-ker...@vger.kernel.org; linux-omap@vger.kernel.org
 Subject: Re: [PATCH v2 0/5] OMAP3EVM: Add support for MT9T111 sensor

 On 2011-09-27 07:40, Deepthy Ravi wrote:
 This patchset
   -adds support for MT9T111 sensor on omap3evm.
   Currently the sensor driver supports only
   VGA resolution.
   -enables MT9T111 sensor in omap2plus_defconfig.

 This is dependent on the following patchset
 http://www.spinics.net/lists/linux-media/msg37270.html
 which adds YUYV input support for OMAP3ISP. And is
 applied on top of rc1-for-3.2 of gliakhovetski/v4l-dvb.git

 Why not use the same base as Lennart?
   The set is based on
   
 http://git.linuxtv.org/pinchartl/media.git/shortlog/refs/heads/omap3isp-omap3isp-next

[Deepthy Ravi] Because the patches for making mt9t112 driver usable outside the 
soc-camera subsystem are present in that base only . Its not there in Laurent's.
 ---
 Changes in v2:
   As per the discussion here,
   https://lkml.org/lkml/2011/9/20/280
   the existing mt9t112 driver is reused for
   adding support for mt9t111 sensor.
 Deepthy Ravi (3):
[media] v4l: Add support for mt9t111 sensor driver
ispccdc: Configure CCDC_SYN_MODE register
omap2plus_defconfig: Enable omap3isp and MT9T111 sensor drivers

 Vaibhav Hiremath (2):
omap3evm: Enable regulators for camera interface
omap3evm: Add Camera board init/hookup file

   arch/arm/configs/omap2plus_defconfig|9 +
   arch/arm/mach-omap2/Makefile|5 +
   arch/arm/mach-omap2/board-omap3evm-camera.c |  185 
   arch/arm/mach-omap2/board-omap3evm.c|   26 +
   drivers/media/video/Kconfig |7 +
   drivers/media/video/Makefile|1 +
   drivers/media/video/mt9t111_reg.h   | 1367 
 +++
   drivers/media/video/mt9t112.c   |  320 ++-
   drivers/media/video/omap3isp/ispccdc.c  |   11 +-
   include/media/mt9t111.h |   45 +
   10 files changed, 1937 insertions(+), 39 deletions(-)
   create mode 100644 arch/arm/mach-omap2/board-omap3evm-camera.c
   create mode 100644 drivers/media/video/mt9t111_reg.h
   create mode 100644 include/media/mt9t111.h

 --
 
 Gary Thomas |  Consulting for the
 MLB Associates  |Embedded world
 


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


Re: [PATCH 2/5] [media] v4l: Add mt9t111 sensor driver

2011-09-20 Thread Guennadi Liakhovetski
On Tue, 20 Sep 2011, Deepthy Ravi wrote:

 From: Vaibhav Hiremath hvaib...@ti.com
 
 The MT9T111 is a 3.1Mp CMOS sensor from Aptina with
 its latest image signal processing (ISP) and 1.75ÎŒm
 pixel technology.
 The sensor driver currently supports only VGA
 resolution.
 
 Signed-off-by: Vaibhav Hiremath hvaib...@ti.com
 Signed-off-by: Deepthy Ravi deepthy.r...@ti.com
 ---
  drivers/media/video/Kconfig   |7 +
  drivers/media/video/Makefile  |1 +
  drivers/media/video/mt9t111.c |  793 +
  drivers/media/video/mt9t111_reg.h | 1367 
 +
  include/media/mt9t111.h   |   45 ++
  5 files changed, 2213 insertions(+), 0 deletions(-)
  create mode 100644 drivers/media/video/mt9t111.c
  create mode 100644 drivers/media/video/mt9t111_reg.h
  create mode 100644 include/media/mt9t111.h

NAK. The mt9t112 driver claims to also support mt9t111. I'm not sure, 
whether the driver has indeed been tested with mt9t111, but this is 
definitely something to verify. If the chips are indeed similar, please 
use the tree at

http://git.linuxtv.org/gliakhovetski/v4l-dvb.git/shortlog/refs/heads/rc1-for-3.2

which has converted the mt9t112 driver to be also used outside ot the 
soc-camera subsystem, and use that driver instead of adding another one.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html