RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-19 Thread Karicheri, Muralidharan
Mauro,

Kevin has approved the architecture part of this patch. When can I expect these 
to be merged to linux-next?

Thanks.

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
email: m-kariche...@ti.com

-Original Message-
From: Karicheri, Muralidharan
Sent: Tuesday, August 18, 2009 5:51 PM
To: 'Mauro Carvalho Chehab'
Cc: Mauro Carvalho Chehab; linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com; Hans Verkuil
Subject: RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

Mauro,

Here are the patches from Chaithrika that I am referring to.
http://www.mail-archive.com/linux-me...@vger.kernel.org/msg08254.html
http://www.mail-archive.com/linux-me...@vger.kernel.org/msg07676.html

Let me know once they are merged...

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Mauro Carvalho Chehab [mailto:mche...@infradead.org]
Sent: Tuesday, August 18, 2009 1:28 PM
To: Karicheri, Muralidharan
Cc: Mauro Carvalho Chehab; linux-me...@vger.kernel.org; davinci-linux-
open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com; Hans Verkuil
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

Em Tue, 18 Aug 2009 11:06:54 -0500
Karicheri, Muralidharan m-kariche...@ti.com escreveu:

 Mauro,

 I need to send a set of patches for adding vpif capture driver.
Currently
the linux-next doesn't have the last patch from Chaithrika applied for
vpif
display. Is it possible to apply this asap so that I can create the vpif
capture patch today?

Sure. Could you please point me what's the patchwork ID(s)[1] of the patch
you need
me to apply at our development tree and at linux-next?
  [1] http://patchwork.kernel.org/project/linux-media/list/

Cheers,
Mauro

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-18 Thread Hans Verkuil
On Tuesday 18 August 2009 01:23:10 Karicheri, Muralidharan wrote:
 Hans,
 
 I have re-send vpfe capture patch. I will re-send vpif patches tomorrow.

These patches apply fine. I'll merge them in my v4l-dvb-dm646x tree tonight.

Thanks!

Hans

 
 Murali Karicheri
 Software Design Engineer
 Texas Instruments Inc.
 Germantown, MD 20874
 new phone: 301-407-9583
 Old Phone : 301-515-3736 (will be deprecated)
 email: m-kariche...@ti.com
 
 -Original Message-
 From: Hans Verkuil [mailto:hverk...@xs4all.nl]
 Sent: Monday, August 17, 2009 4:27 PM
 To: Karicheri, Muralidharan
 Cc: linux-me...@vger.kernel.org; davinci-linux-open-
 sou...@linux.davincidsp.com; khil...@deeprootsystems.com
 Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
 capture driver
 
 On Monday 17 August 2009 22:10:04 Karicheri, Muralidharan wrote:
  Hans,
 
  Would you like the architecture specific changes against v4l-dvb linux-
 next tree or linux-davinci ? I will rework both the vpfe and vpif patches
 as per your comment.
 
 v4l-dvb linux-next. The current v4l-dvb at least compiles against that one,
 so
 that is the most appropriate tree to do the patches against.
 
 Regards,
 
  Hans
 
 --
 Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom
 
 
 
 



-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-18 Thread Hans Verkuil
On Tuesday 18 August 2009 08:49:13 Hans Verkuil wrote:
 On Tuesday 18 August 2009 01:23:10 Karicheri, Muralidharan wrote:
  Hans,
  
  I have re-send vpfe capture patch. I will re-send vpif patches tomorrow.
 
 These patches apply fine. I'll merge them in my v4l-dvb-dm646x tree tonight.

Oops, wrong tree. It's v4l-dvb-vpif.

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-18 Thread Karicheri, Muralidharan
Mauro,

I need to send a set of patches for adding vpif capture driver. Currently the 
linux-next doesn't have the last patch from Chaithrika applied for vpif 
display. Is it possible to apply this asap so that I can create the vpif 
capture patch today?

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Hans Verkuil [mailto:hverk...@xs4all.nl]
Sent: Tuesday, August 18, 2009 2:51 AM
To: Karicheri, Muralidharan
Cc: linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

On Tuesday 18 August 2009 08:49:13 Hans Verkuil wrote:
 On Tuesday 18 August 2009 01:23:10 Karicheri, Muralidharan wrote:
  Hans,
 
  I have re-send vpfe capture patch. I will re-send vpif patches tomorrow.

 These patches apply fine. I'll merge them in my v4l-dvb-dm646x tree
tonight.

Oops, wrong tree. It's v4l-dvb-vpif.

Regards,

   Hans

--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-18 Thread Karicheri, Muralidharan
Mauro,

Here are the patches from Chaithrika that I am referring to.
http://www.mail-archive.com/linux-me...@vger.kernel.org/msg08254.html
http://www.mail-archive.com/linux-me...@vger.kernel.org/msg07676.html

Let me know once they are merged...

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Mauro Carvalho Chehab [mailto:mche...@infradead.org]
Sent: Tuesday, August 18, 2009 1:28 PM
To: Karicheri, Muralidharan
Cc: Mauro Carvalho Chehab; linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com; Hans Verkuil
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

Em Tue, 18 Aug 2009 11:06:54 -0500
Karicheri, Muralidharan m-kariche...@ti.com escreveu:

 Mauro,

 I need to send a set of patches for adding vpif capture driver. Currently
the linux-next doesn't have the last patch from Chaithrika applied for vpif
display. Is it possible to apply this asap so that I can create the vpif
capture patch today?

Sure. Could you please point me what's the patchwork ID(s)[1] of the patch
you need
me to apply at our development tree and at linux-next?
   [1] http://patchwork.kernel.org/project/linux-media/list/

Cheers,
Mauro

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Karicheri, Muralidharan
Hans,

They are applied against davinci tree (also mentioned in the patch). General 
procedure what I follow is to create platform code against davinci tree and v4l 
patches against v4l-dvb linux-next tree. The architecture part of linux-next is 
not up to date.

Davinci tree is at

git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-davinci.git

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Hans Verkuil [mailto:hverk...@xs4all.nl]
Sent: Saturday, August 15, 2009 8:10 AM
To: Karicheri, Muralidharan
Cc: linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

On Friday 14 August 2009 23:01:41 m-kariche...@ti.com wrote:
 From: Muralidharan Karicheri m-kariche...@ti.com

 This patch makes the following changes:-
  1) Modify vpif_subdev_info to add board_info, routing information
 and vpif interface configuration. Remove addr since it is
 part of board_info

  2) Add code to setup channel mode and input decoder path for
 vpif capture driver

 Also incorporated comments against version v0 of the patch series and
 added a spinlock to protect writes to common registers

A quick question: against which git tree are these arch changes applied?
I've lost track of that :-)

Regards,

   Hans

--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Hans Verkuil
On Monday 17 August 2009 16:52:20 Karicheri, Muralidharan wrote:
 Hans,
 
 They are applied against davinci tree (also mentioned in the patch). General 
 procedure what I follow is to create platform code against davinci tree and 
 v4l patches against v4l-dvb linux-next tree. The architecture part of 
 linux-next is not up to date.
 
 Davinci tree is at
 
 git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-davinci.git

I must have missed the mention of this tree.

I have a problem, though, as the current v4l-dvb repository doesn't compile
against the linux-davinci git tree. And the only way I can get it to compile
is to apply all five patches first.

However, the whole tree should still compile after each patch is applied. And
that goes wrong with your second patch where the Kconfig and Makefile are
modified when the new sources aren't even added yet!

What I would like to see is a patch series that starts with one patch that
makes the current v4l-dvb tree compile again, then the arch patch is added,
then a series of v4l-dvb patches in such an order that everything compiles
after each step.

Merging this is already complicated enough without breaking compilation in
this way.

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Karicheri, Muralidharan
Hans,

Ok. I will rework the patch and send you the same.

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Hans Verkuil [mailto:hverk...@xs4all.nl]
Sent: Monday, August 17, 2009 2:47 PM
To: Karicheri, Muralidharan
Cc: linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

On Monday 17 August 2009 16:52:20 Karicheri, Muralidharan wrote:
 Hans,

 They are applied against davinci tree (also mentioned in the patch).
General procedure what I follow is to create platform code against davinci
tree and v4l patches against v4l-dvb linux-next tree. The architecture part
of linux-next is not up to date.

 Davinci tree is at

 git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-davinci.git

I must have missed the mention of this tree.

I have a problem, though, as the current v4l-dvb repository doesn't compile
against the linux-davinci git tree. And the only way I can get it to
compile
is to apply all five patches first.

However, the whole tree should still compile after each patch is applied.
And
that goes wrong with your second patch where the Kconfig and Makefile are
modified when the new sources aren't even added yet!

What I would like to see is a patch series that starts with one patch that
makes the current v4l-dvb tree compile again, then the arch patch is added,
then a series of v4l-dvb patches in such an order that everything compiles
after each step.

Merging this is already complicated enough without breaking compilation in
this way.

Regards,

   Hans

--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Karicheri, Muralidharan
Hans,

Would you like the architecture specific changes against v4l-dvb linux-next 
tree or linux-davinci ? I will rework both the vpfe and vpif patches as per 
your comment.

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Hans Verkuil [mailto:hverk...@xs4all.nl]
Sent: Monday, August 17, 2009 2:47 PM
To: Karicheri, Muralidharan
Cc: linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

On Monday 17 August 2009 16:52:20 Karicheri, Muralidharan wrote:
 Hans,

 They are applied against davinci tree (also mentioned in the patch).
General procedure what I follow is to create platform code against davinci
tree and v4l patches against v4l-dvb linux-next tree. The architecture part
of linux-next is not up to date.

 Davinci tree is at

 git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-davinci.git

I must have missed the mention of this tree.

I have a problem, though, as the current v4l-dvb repository doesn't compile
against the linux-davinci git tree. And the only way I can get it to
compile
is to apply all five patches first.

However, the whole tree should still compile after each patch is applied.
And
that goes wrong with your second patch where the Kconfig and Makefile are
modified when the new sources aren't even added yet!

What I would like to see is a patch series that starts with one patch that
makes the current v4l-dvb tree compile again, then the arch patch is added,
then a series of v4l-dvb patches in such an order that everything compiles
after each step.

Merging this is already complicated enough without breaking compilation in
this way.

Regards,

   Hans

--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Hans Verkuil
On Monday 17 August 2009 22:10:04 Karicheri, Muralidharan wrote:
 Hans,
 
 Would you like the architecture specific changes against v4l-dvb linux-next 
 tree or linux-davinci ? I will rework both the vpfe and vpif patches as per 
 your comment.

v4l-dvb linux-next. The current v4l-dvb at least compiles against that one, so
that is the most appropriate tree to do the patches against.

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-17 Thread Karicheri, Muralidharan
Hans,

I have re-send vpfe capture patch. I will re-send vpif patches tomorrow.

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
new phone: 301-407-9583
Old Phone : 301-515-3736 (will be deprecated)
email: m-kariche...@ti.com

-Original Message-
From: Hans Verkuil [mailto:hverk...@xs4all.nl]
Sent: Monday, August 17, 2009 4:27 PM
To: Karicheri, Muralidharan
Cc: linux-me...@vger.kernel.org; davinci-linux-open-
sou...@linux.davincidsp.com; khil...@deeprootsystems.com
Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
capture driver

On Monday 17 August 2009 22:10:04 Karicheri, Muralidharan wrote:
 Hans,

 Would you like the architecture specific changes against v4l-dvb linux-
next tree or linux-davinci ? I will rework both the vpfe and vpif patches
as per your comment.

v4l-dvb linux-next. The current v4l-dvb at least compiles against that one,
so
that is the most appropriate tree to do the patches against.

Regards,

   Hans

--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source


Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

2009-08-15 Thread Hans Verkuil
On Friday 14 August 2009 23:01:41 m-kariche...@ti.com wrote:
 From: Muralidharan Karicheri m-kariche...@ti.com
 
 This patch makes the following changes:-
   1) Modify vpif_subdev_info to add board_info, routing information
  and vpif interface configuration. Remove addr since it is
  part of board_info

   2) Add code to setup channel mode and input decoder path for
  vpif capture driver
 
 Also incorporated comments against version v0 of the patch series and
 added a spinlock to protect writes to common registers

A quick question: against which git tree are these arch changes applied?
I've lost track of that :-)

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source