RE: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-19 Thread Hadli, Manjunath
Hans, Thank you for the review comments. I will include a section on customizing for additions and changes to different boards. Thanks again, -Manju On Sat, Nov 13, 2010 at 18:22:19, Hans Verkuil wrote: Hi Manju, I've reviewed the other patches as well. The only one for which I had

Re: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-13 Thread Hans Verkuil
Hi Manju, I've reviewed the other patches as well. The only one for which I had comments was patch 2/6. However, what I think would be useful here is to have an overview document, either as part of a source or header, or as a separate text document. It is not easy to get a good overview of how

RE: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-10 Thread Hadli, Manjunath
Murali, I will change the name order. Thanks and Regards, -manju On Wed, Nov 10, 2010 at 18:35:16, Muralidharan Karicheri wrote: Manjunath, Thank you for putting up this patch together. I didn't see the 1/6 of this series in the mailing list. Also it appears as if the patch came from me.

Re: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-10 Thread Muralidharan Karicheri
Hans, Is it possible to extend the sub device ops to include SoC ip sub device specific ops? I remember I had posted this question some time back and you had proposed to add something like this. Just want to check if that is still valid. This would make this driver implementation little more

Re: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-10 Thread Hans Verkuil
Hans, Is it possible to extend the sub device ops to include SoC ip sub device specific ops? I remember I had posted this question some time back and you had proposed to add something like this. Just want to check if that is still valid. This would make this driver implementation little

Re: [PATCH 0/6] davinci vpbe: V4L2 Display driver for DM644X

2010-11-10 Thread Hans Verkuil
Hans, Is it possible to extend the sub device ops to include SoC ip sub device specific ops? I remember I had posted this question some time back and you had proposed to add something like this. Just want to check if that is still valid. This would make this driver implementation little