[PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread m-karicheri2
From: Muralidharan Karicheri This is the second version of the digital video timings APIs implementation. This adds comment from previous version. I would like to have this merged to 2.6.33 This adds the above APIs to the v4l2 core. This is based on version v1.2 of the RFC titled "V4L - Support

RE: [PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread Karicheri, Muralidharan
To: linux-me...@vger.kernel.org; hverk...@xs4all.nl >Cc: davinci-linux-open-source@linux.davincidsp.com; Karicheri, Muralidharan >Subject: [PATCH v2] V4L - Adding Digital Video Timings APIs > >From: Muralidharan Karicheri > >This is the second version of the digital video timings A

Re: [PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread Hans Verkuil
On Wednesday 18 November 2009 18:03:44 m-kariche...@ti.com wrote: > From: Muralidharan Karicheri > > This is the second version of the digital video timings APIs implementation. > This adds comment from previous version. I would like to have this > merged to 2.6.33 Me too :-) > This adds the a

RE: [PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread Karicheri, Muralidharan
nal Message- >From: Hans Verkuil [mailto:hverk...@xs4all.nl] >Sent: Wednesday, November 18, 2009 1:09 PM >To: Karicheri, Muralidharan >Cc: linux-me...@vger.kernel.org; davinci-linux-open- >sou...@linux.davincidsp.com >Subject: Re: [PATCH v2] V4L - Adding Digital Video Timings AP

Re: [PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread Hans Verkuil
On Wednesday 18 November 2009 20:01:43 Karicheri, Muralidharan wrote: > Hans, > > Thanks for reviewing this. I will try to send an updated patch today. > > BTW, I have posted the documentation patch to the list for review. I did some quick 64-bit tests and discovered that we need to add the pack

RE: [PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread Karicheri, Muralidharan
Hans, Thanks for doing the 64-bit test. I will incorporate the below comments along with the other comments and will post v3 of the patch with you included in the sign off. Regards, Murali > >I did some quick 64-bit tests and discovered that we need to add the packed >attribute to struct v4l2_bt