[PATCH v2] V4L - Adding Digital Video Timings APIs

2009-11-18 Thread m-karicheri2
From: Muralidharan Karicheri m-kariche...@ti.com 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

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

2009-11-18 Thread Karicheri, Muralidharan
...@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 m-kariche...@ti.com This is the second version of the digital video timings APIs implementation

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 m-kariche...@ti.com 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 :-)

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

2009-11-18 Thread Karicheri, Muralidharan
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 APIs On Wednesday 18 November

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 packed

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