RE: [PATCH v2 1/4] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-04-22 Thread chaithrika
Hans, Thank you for taking time to review these patches. Please see my response inline. I will be able to post the updated patches next week. -Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Saturday, April 18, 2009 4:58 PM To: Chaithrika U S Cc: linux-media

RE: [PATCH v2 2/4] ARM: DaVinci: DM646x Video: VPIF driver

2009-04-22 Thread chaithrika
-Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Saturday, April 18, 2009 5:16 PM To: Chaithrika U S Cc: linux-media@vger.kernel.org; davinci-linux-open- sou...@linux.davincidsp.com; Manjunath Hadli; Brijesh Jadav Subject: Re: [PATCH v2 2/4] ARM: DaVinci

RE: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-davinci

2009-05-05 Thread chaithrika
Hans, -Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Tuesday, May 05, 2009 5:41 PM To: Mauro Carvalho Chehab Cc: Mauro Carvalho Chehab; linux-media@vger.kernel.org; Chaithrika U S Subject: Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-davinci Hi

[PATCH v3] v4l: TI THS7303 video amplifier driver code

2009-05-07 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com This patch adds driver for TI THS7303 video amplifier. This driver is implemented as a v4l2 sub device. Tested on TI DM646x EVM. This version has updates based on review comments by Mauro Chehab. Signed-off-by: Chaithrika U S chaithr...@ti.com Reviewed

[PATCH v4] v4l: Analog Devices ADV7343 video encoder driver

2009-05-07 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Add ADV7343 I2C based video encoder driver. This follows the v4l2-subdev framework. This driver has been tested on TI DM646x EVM. It has been tested for Composite and Component outputs. Updates as per review by Mauro Chehab, added support for more standards

RE: v4l-dvb rev 11757 broke building under Ubuntu Hardy

2009-05-15 Thread chaithrika
Andy, Thanks for the patch. Please see my response below. -Original Message- From: Andy Walls [mailto:awa...@radix.net] Sent: Friday, May 15, 2009 5:24 AM To: david.w...@gatech.edu Cc: Chaithrika U S; linux-media@vger.kernel.org Subject: Re: v4l-dvb rev 11757 broke building under

RE: [PATCH] Subject: [PATCH v3 1/4] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-06-22 Thread chaithrika
Kevin, I think this patch has to be taken into DaVinci tree so that it can be submitted upstream. This patch has to be present in the Linux tree for Hans to prepare a pull request for DM646x display driver patches. Regards, Chaithrika -Original Message- From: linux-media-ow

[RFC 0/7] ARM: DaVinci: DM646x Video: DM646x display driver

2009-03-13 Thread chaithrika
Display driver for TI DM646x EVM Signed-off-by: Chaithrika U S chaithr...@ti.com This patch set is being submitted to get review and opinion on the approaches used to implement the sub devices and display drivers. This set adds the display driver support for TI DM646x EVM. This patch set has

[RFC 1/7] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Add pin mux definitions, display device setup, clock setup functions Add pin mux related code for the display device, also add platform device and resource structures. Also define a platform specific clock setup function that can be accessed by the driver

[RFC 0/7] ARM: DaVinci: DM646x Video: DM646x display driver

2009-03-13 Thread chaithrika
Display driver for TI DM646x EVM Signed-off-by: Chaithrika U S chaithr...@ti.com This patch set is being submitted to get review and opinion on the approaches used to implement the sub devices and display drivers. This set adds the display driver support for TI DM646x EVM. This patch set has

[RFC 1/7] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Add pin mux definitions, display device setup, clock setup functions Add pin mux related code for the display device, also add platform device and resource structures. Also define a platform specific clock setup function that can be accessed by the driver

[RFC 3/7] ARM: DaVinci: DM646x Video: THS7303 video amplifier driver

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com THS7303 video amplifier driver code This patch implements driver for TI THS7303 video amplifier . This driver is implemented as a v4l2-subdev. --- Applies to v4l-dvb repository located at http://linuxtv.org/hg/v4l-dvb/rev/1fd54a62abde drivers/media/video

[RFC 4/7] ARM: DaVinci: DM646x Video: Defintions for standards supported by display

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Add defintions for Digital TV Standards supported by display driver Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository located at http://linuxtv.org/hg/v4l-dvb/rev/1fd54a62abde include/linux/videodev2.h | 12

[RFC 5/7] ARM: DaVinci: DM646x Video: Makefile and config files modifications for Display

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Makefile and Kconfig changes Modifies and adds the video Makefiles and Kconfig files to support DM646x Video display device Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository located at http://linuxtv.org/hg/v4l-dvb/rev

[RFC 6/7] ARM: DaVinci: DM646x Video: Add VPIF driver

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com Video Port Interface driver Add VPIF driver for DM646x. This code be used by the display and capture drivers. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository located at http://linuxtv.org/hg/v4l-dvb/rev/1fd54a62abde

[RFC 2/7] ARM: DaVinci: DM646x Video: ADV7343 video encoder driver

2009-03-13 Thread chaithrika
From: Chaithrika U S chaithr...@ti.com ADV7343 video encoder driver Add ADV7473 I2C based video encoder driver. This follows the v4l2-subdev framework. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository located at http://linuxtv.org/hg/v4l-dvb/rev/1fd54a62abde

RE: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-vpfe-cap

2009-07-13 Thread chaithrika
by him in order to forward the arm dm646x patch (and the rest of the series) upstream. Mauro, I have submitted the updated patch last week[1]. I have marked li...@arm.linux.org.uk in cc. A patch for VPIF updates has also been posted[2] . Regards, Chaithrika [1] http://www.mail-archive.com

RE: [PATCH v2] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-07-14 Thread chaithrika
Russell, Requesting your review and ack on this updated patch. Regards, Chaithrika On Tue, Jul 07, 2009 at 17:22:57, Chaithrika U S wrote: Platform specific display device setup for DM646x EVM Add platform device and resource structures. Also define a platform specific clock setup function

RE: [PATCH v2] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-07-20 Thread chaithrika
-Original Message- From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk] Sent: Sunday, July 19, 2009 4:13 PM To: Chaithrika U S Cc: linux-media@vger.kernel.org; mche...@infradead.org; hverk...@xs4all.nl; davinci-linux-open-sou...@linux.davincidsp.com; Manjunath Hadli

RE: [PATCH v3] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-08-02 Thread chaithrika
Mauro/Russell, The previous version (v2) of this patch is on the linux-next tree. This patch has some updates done on top of that patch. Should I post an incremental patch for those changes to the Linux-next tree? Please suggest. Regards, Chaithrika On Mon, Jul 20, 2009 at 13:31:22, Chaithrika

RE: [PATCH v3] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-08-03 Thread chaithrika
Mauro, OK. Thank you for taking care of this. On Mon, Aug 03, 2009 at 11:20:03, Mauro Carvalho Chehab wrote: Em Mon, 3 Aug 2009 09:17:06 +0530 chaithrika chaithr...@ti.com escreveu: Mauro/Russell, The previous version (v2) of this patch is on the linux-next tree. This patch has some

RE: vpif_display.c bug

2009-08-10 Thread chaithrika
On Mon, Aug 10, 2009 at 11:37:23, Hans Verkuil wrote: Hi Chaithrika, This code in vpif_display.c is not correct: for (i = 0; i subdev_count; i++) { vpif_obj.sd[i] = v4l2_i2c_new_probed_subdev(vpif_obj.v4l2_dev

RE: vpif_display.c bug

2009-08-11 Thread chaithrika
On Mon, Aug 10, 2009 at 20:39:05, Karicheri, Muralidharan wrote: Chaithrika, No need to change this since this is already corrected as part of my vpif capture patch set that I had submitted for review. I had mentioned this to Hans as well. Murali, Thank you for correcting this bug

RE: [PATCH 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

2009-03-29 Thread Subrahmanya, Chaithrika
On Thu, 2009-03-26 at 09:21 -0400, Chaithrika U S wrote: Display driver for TI DM646x EVM Signed-off-by: Chaithrika U S chaithr...@ti.com This patch set has been updated with the review comments for the RFC sent earlier. These patches add the display driver support for TI DM646x

RE: [PATCH 2/4] ARM: DaVinci: DM646x Video: VPIF driver

2009-03-31 Thread Subrahmanya, Chaithrika
Hans, Thanks for the comments. I will post the updated patches sometime next week. On Thursday 26 March 2009 14:22:20 Chaithrika U S wrote: Video Port Interface driver Add VPIF driver for DM646x. This code be used by the display and capture drivers. Signed-off-by: Chaithrika U S

RE: [PATCH 3/4] ARM: DaVinci: DM646x Video: Add VPIF display driver

2009-03-31 Thread Subrahmanya, Chaithrika
On Mon, 30 Mar 2009, Hans Verkuil wrote: On Thursday 26 March 2009 14:22:32 Chaithrika U S wrote: + /* one field is displayed configure the next + frame if it is available else hold on current + frame

RE: [RFC 3/7] ARM: DaVinci: DM646x Video: THS7303 video amplifier driver

2009-03-16 Thread Subrahmanya, Chaithrika
Hans, Thank you for the comments. Please see my response inline. I have agreed to most of your comments and will do the changes accordingly. Hi Chaithrika, This is the first pass of this i2c driver. Note that several of the comments here also apply to adv7343. OK, will do

RE: [RFC 2/7] ARM: DaVinci: DM646x Video: ADV7343 video encoder driver

2009-03-16 Thread Subrahmanya, Chaithrika
Hans, Please see my response inline. For the comments common to THS7303 and ADV7343 driver, I will do the changes to both the drivers. Hi Chaithrika, Here is the review of this driver. On Friday 13 March 2009 10:08:39 chaithr...@ti.com wrote: From: Chaithrika U S chaithr...@ti.com

RE: [RFC 4/7] ARM: DaVinci: DM646x Video: Defintions for standards supported by display

2009-03-19 Thread Subrahmanya, Chaithrika
related patches Thanks, Chaithrika From: Hans Verkuil [hverk...@xs4all.nl] Sent: Thursday, March 19, 2009 4:05 AM To: davinci-linux-open-sou...@linux.davincidsp.com; Subrahmanya, Chaithrika Cc: linux-media@vger.kernel.org Subject: Re: [RFC 4/7] ARM: DaVinci

[PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

2009-05-08 Thread Chaithrika U S
Display driver for TI DM646x EVM Signed-off-by: Manjunath Hadli m...@ti.com Signed-off-by: Brijesh Jadav brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com These patches add the display driver support for TI DM646x EVM. This patch set has been tested for basic display functionality

[PATCH] Subject: [PATCH v3 1/4] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-05-08 Thread Chaithrika U S
and clock definitions for Video on DM646x. Signed-off-by: Manjunath Hadli m...@ti.com Signed-off-by: Brijesh Jadav brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to Davinci GIT tree arch/arm/mach-davinci/board-dm646x-evm.c| 122 +++ arch/arm

[PATCH v3 2/4] ARM: DaVinci: DM646x Video: VPIF driver

2009-05-08 Thread Chaithrika U S
Video Port Interface driver Add VPIF driver for DM646x. This code be used by the display and capture drivers. This version has some incorporated the review comments. Signed-off-by: Manjunath Hadli m...@ti.com Signed-off-by: Brijesh Jadav brijes...@ti.com Signed-off-by: Chaithrika U S chaithr

[PATCH] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-06-22 Thread Chaithrika U S
and clock definitions for Video on DM646x. Signed-off-by: Manjunath Hadli m...@ti.com Signed-off-by: Brijesh Jadav brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to Davinci GIT tree. This patch was previously submitted as a part of the series of patches that added display

[PATCH] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-06-22 Thread Chaithrika U S
and clock definitions for Video on DM646x. Signed-off-by: Manjunath Hadli m...@ti.com Signed-off-by: Brijesh Jadav brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to Davinci GIT tree. This patch was previously submitted as a part of the series of patches that added display

[PATCH] ARM: DaVinci: DM646x Video: Fix compile time warnings for mutex locking

2009-06-22 Thread Chaithrika U S
mutex_lock_interruptible return value has to be handled properly to indicate the status to the higher layers of the kernel. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb-dm646x repo maintained by Hans Verkuil at http://linuxtv.org/hg/~hverkuil/v4l-dvb-dm646x/ drivers

[PATCH] v4l2-subdev: ADV7343 video encoder driver

2009-03-23 Thread Chaithrika U S
Analog Devices ADV7343 video encoder driver Add ADV7343 I2C based video encoder driver. This follows the v4l2-subdev framework.This driver has been tested on TI DM646x EVM. It has been tested for Composite and Component outputs. Signed-off-by: Chaithrika U S chaithr...@ti.com --- drivers/media

[PATCH] v4l2-subdev: THS7303 video amplifier driver

2009-03-23 Thread Chaithrika U S
of the 'Kconfig', 'Makefile', 'v4l2-chip-ident.h' files by both the patches. Signed-off-by: Chaithrika U S chaithr...@ti.com --- drivers/media/video/Kconfig |9 ++ drivers/media/video/Makefile|1 + drivers/media/video/ths7303.c | 158 +++ include

[PATCH v2] v4l2-subdev: ADV7343 video encoder driver

2009-03-24 Thread Chaithrika U S
of adv7343_channel Note: There is no mention of SECAM support in the encoder datasheet. Signed-off-by: Chaithrika U S chaithr...@ti.com --- drivers/media/video/Kconfig|9 + drivers/media/video/Makefile |1 + drivers/media/video/adv7343.c | 545

[PATCH v2] v4l2-subdev: THS7303 video amplifier driver

2009-03-24 Thread Chaithrika U S
of the 'Kconfig', 'Makefiile', 'v4l2-chip-ident.h' files by both the patches. This updated version of the patch has incorporated the review comments. Signed-off-by: Chaithrika U S chaithr...@ti.com --- drivers/media/video/Kconfig |9 +++ drivers/media/video/Makefile|1 + drivers/media

[PATCH 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

2009-03-26 Thread Chaithrika U S
Display driver for TI DM646x EVM Signed-off-by: Chaithrika U S chaithr...@ti.com This patch set has been updated with the review comments for the RFC sent earlier. These patches add the display driver support for TI DM646x EVM. This patch set has been tested for basic display functionality

[PATCH 1/4] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-03-26 Thread Chaithrika U S
Mux and clock definitions for Video on DM646x. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to DaVinci GIT tree arch/arm/mach-davinci/board-dm646x-evm.c| 138 +++ arch/arm/mach-davinci/dm646x.c | 63 arch/arm/mach-davinci

[PATCH 3/4] ARM: DaVinci: DM646x Video: Add VPIF display driver

2009-03-26 Thread Chaithrika U S
. - changed the subdev calls to subdev_call_until_err Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository drivers/media/video/davinci/vpif_display.c | 1702 drivers/media/video/davinci/vpif_display.h | 204 2 files changed, 1906

[PATCH 4/4] ARM: DaVinci: DM646x Video: Makefile and config files modifications for Display

2009-03-26 Thread Chaithrika U S
Makefile and Kconfig changes Modifies and adds the video Makefiles and Kconfig files to support DM646x Video display device Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository drivers/media/video/Kconfig | 22 ++ drivers/media/video

[PATCH]: v4l: DaVinci: DM646x: Updates to VPIF display driver

2009-07-07 Thread Chaithrika U S
Minor fixes like change in the names of the standards, remove inclusion of version.h in the header file. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repo drivers/media/video/davinci/vpif_display.c | 16 +--- drivers/media/video/davinci/vpif_display.h

[PATCH v2] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-07-07 Thread Chaithrika U S
brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com Signed-off-by: Kevin Hilman khil...@deeprootsystems.com --- Applies to Davinci GIT tree.The current version has updates as per review comments from Russell King arch/arm/mach-davinci/board-dm646x-evm.c| 125

[PATCH v3] ARM: DaVinci: DM646x Video: Platform and board specific setup

2009-07-20 Thread Chaithrika U S
brijes...@ti.com Signed-off-by: Chaithrika U S chaithr...@ti.com Signed-off-by: Kevin Hilman khil...@deeprootsystems.com --- Applies to Davinci GIT tree. Minor updates like change in structure name- subdev_info to vpif_subdev_info and correction to VDD3P3V_VID_MASK value. arch/arm/mach-davinci/board

[PATCH] v4l: DaVinci: DM646x: Update the structure name as per header file changes

2009-07-20 Thread Chaithrika U S
In the platform header file, the subdev_info structure name has been changed to vpif_subdev_info. Update this change in the driver too. Signed-off-by: Chaithrika U S chaithr...@ti.com --- Applies to v4l-dvb repository. drivers/media/video/davinci/vpif_display.c |2 +- 1 files changed, 1