Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-03 Thread Manjunath Hadli
Hi Sakari, On Saturday 01 September 2012 10:55 PM, Sakari Ailus wrote: > Hi Manju, > > My apologies for the delayed answer. > > On Wed, Aug 22, 2012 at 02:26:50PM +0530, Manjunath Hadli wrote: >> On Thursday 16 August 2012 09:53 PM, Sakari Ailus wrote: >>> On Thu, Aug 09, 2012 at 09:13:52AM +053

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-02 Thread Hans Verkuil
On Sat September 1 2012 16:22:30 Laurent Pinchart wrote: > Hi Sakari, > > On Saturday 01 September 2012 12:57:07 Sakari Ailus wrote: > > On Wed, Aug 29, 2012 at 08:11:50PM +0530, Prabhakar Lad wrote: > > [snip] > > > > For test pattern you meant control to enable/disable it ? > > > > There are

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Laurent Pinchart
On Saturday 01 September 2012 19:11:56 Sakari Ailus wrote: > Hi Prabhakar, > > On Sat, Sep 01, 2012 at 08:23:58PM +0530, Prabhakar Lad wrote: > > On Sat, Sep 1, 2012 at 7:52 PM, Laurent Pinchart > > > > wrote: > > > Hi Sakari, > > > > > > On Saturday 01 September 2012 12:57:07 Sakari Ailus wrot

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Sakari Ailus
On Sat, Sep 01, 2012 at 08:25:30PM +0300, Sakari Ailus wrote: > That would be against the V4L2 spec. It's explicitly defined that the source > compose rectangle defines the output size of the scaler. This should be sink, not source. -- Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@re

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Sakari Ailus
Hi Manju, My apologies for the delayed answer. On Wed, Aug 22, 2012 at 02:26:50PM +0530, Manjunath Hadli wrote: > On Thursday 16 August 2012 09:53 PM, Sakari Ailus wrote: > > On Thu, Aug 09, 2012 at 09:13:52AM +0530, Manjunath Hadli wrote: > >> On Thursday 02 August 2012 05:37 AM, Sakari Ailus wr

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Sakari Ailus
Hi Prabhakar, On Sat, Sep 01, 2012 at 08:23:58PM +0530, Prabhakar Lad wrote: > On Sat, Sep 1, 2012 at 7:52 PM, Laurent Pinchart > wrote: > > Hi Sakari, > > > > On Saturday 01 September 2012 12:57:07 Sakari Ailus wrote: > >> On Wed, Aug 29, 2012 at 08:11:50PM +0530, Prabhakar Lad wrote: > > > > [s

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Prabhakar Lad
Hi Laurent, On Sat, Sep 1, 2012 at 7:52 PM, Laurent Pinchart wrote: > Hi Sakari, > > On Saturday 01 September 2012 12:57:07 Sakari Ailus wrote: >> On Wed, Aug 29, 2012 at 08:11:50PM +0530, Prabhakar Lad wrote: > > [snip] > >> > For test pattern you meant control to enable/disable it ? >> >> There

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Laurent Pinchart
Hi Sakari, On Saturday 01 September 2012 12:57:07 Sakari Ailus wrote: > On Wed, Aug 29, 2012 at 08:11:50PM +0530, Prabhakar Lad wrote: [snip] > > For test pattern you meant control to enable/disable it ? > > There are two approaches I can think of. > > One is a menu control which can be used t

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-09-01 Thread Sakari Ailus
Hi Prabhakar, On Wed, Aug 29, 2012 at 08:11:50PM +0530, Prabhakar Lad wrote: ... > >> >> +unsigned short len; > >> >> +void *config; > >> >> +}; > >> >> + > >> >> +5: IOCTL: VPFE_CMD_S_CCDC_RAW_PARAMS/VPFE_CMD_G_CCDC_RAW_PARAMS > >> >> +Description: > >> >> +Sets/Gets the C

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-29 Thread Prabhakar Lad
Hi Sakari, On Thu, Aug 16, 2012 at 9:53 PM, Sakari Ailus wrote: > Hi Manju, > > On Thu, Aug 09, 2012 at 09:13:52AM +0530, Manjunath Hadli wrote: >> Hi Sakari, >> >> Thank you for the comments. > > Thanks for the graphs! > >> On Thursday 02 August 2012 05:37 AM, Sakari Ailus wrote: >> > Hi Manju,

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-22 Thread Manjunath Hadli
Hi Sakari, On Thursday 16 August 2012 09:53 PM, Sakari Ailus wrote: > Hi Manju, > > On Thu, Aug 09, 2012 at 09:13:52AM +0530, Manjunath Hadli wrote: >> Hi Sakari, >> >> Thank you for the comments. > > Thanks for the graphs! > >> On Thursday 02 August 2012 05:37 AM, Sakari Ailus wrote: >>> Hi

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-16 Thread Sakari Ailus
Hi Manju, On Thu, Aug 09, 2012 at 09:13:52AM +0530, Manjunath Hadli wrote: > Hi Sakari, > > Thank you for the comments. Thanks for the graphs! > On Thursday 02 August 2012 05:37 AM, Sakari Ailus wrote: > > Hi Manju, > > > > Thanks for the patch. > > > > Please make sure these patches reach li

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-16 Thread Prabhakar Lad
Hi Rob, On Thursday 16 August 2012 06:40 PM, Rob Landley wrote: > On 07/11/2012 10:39 AM, Manjunath Hadli wrote: >> Add documentation on the Davinci VPFE driver. Document the subdevs, >> and private IOTCLs the driver implements >> >> Signed-off-by: Manjunath Hadli >> Signed-off-by: Lad, Prabhakar

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-16 Thread Rob Landley
On 07/11/2012 10:39 AM, Manjunath Hadli wrote: > Add documentation on the Davinci VPFE driver. Document the subdevs, > and private IOTCLs the driver implements > > Signed-off-by: Manjunath Hadli > Signed-off-by: Lad, Prabhakar I saw the comment on the 8th, is there another version of this docum

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-08 Thread Manjunath Hadli
Hi Sakari, Thank you for the comments. On Thursday 02 August 2012 05:37 AM, Sakari Ailus wrote: > Hi Manju, > > Thanks for the patch. > > Please make sure these patches reach linux-media next time. If they do > not, > it severely limits the number of potential reviewers. I don't know > why, but

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-02 Thread Laurent Pinchart
Hi Manjunath, On Tuesday 31 July 2012 13:15:27 Manju wrote: > On Friday 27 July 2012 04:19 PM, Laurent Pinchart wrote: > > On Friday 27 July 2012 05:49:24 Hadli, Manjunath wrote: > >> On Thu, Jul 26, 2012 at 05:55:31, Laurent Pinchart wrote: > >>> On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-08-01 Thread Sakari Ailus
Hi Manju, Thanks for the patch. Please make sure these patches reach linux-media next time. If they do not, it severely limits the number of potential reviewers. I don't know why, but the original patch isn't on linux-media even if the list was cc'd. Dropping linux-kernel from cc. Manjunath Ha

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-31 Thread Manju
Hi Laurent, On Friday 27 July 2012 04:19 PM, Laurent Pinchart wrote: Hi Manjunath, On Friday 27 July 2012 05:49:24 Hadli, Manjunath wrote: On Thu, Jul 26, 2012 at 05:55:31, Laurent Pinchart wrote: On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath wrote: On Sun, Jul 15, 2012 at 18:16:25, Laur

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-27 Thread Laurent Pinchart
Hi Manjunath, On Friday 27 July 2012 05:49:24 Hadli, Manjunath wrote: > On Thu, Jul 26, 2012 at 05:55:31, Laurent Pinchart wrote: > > On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath wrote: > > > On Sun, Jul 15, 2012 at 18:16:25, Laurent Pinchart wrote: > > > > On Wednesday 11 July 2012 21:09:26

RE: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-26 Thread Hadli, Manjunath
Laurent, Thank you for your comments. On Thu, Jul 26, 2012 at 05:55:31, Laurent Pinchart wrote: > Hi Manjunath, > > On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath wrote: > > On Sun, Jul 15, 2012 at 18:16:25, Laurent Pinchart wrote: > > > On Wednesday 11 July 2012 21:09:26 Manjunath Hadli wrot

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-25 Thread Laurent Pinchart
Hi Manjunath, On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath wrote: > On Sun, Jul 15, 2012 at 18:16:25, Laurent Pinchart wrote: > > On Wednesday 11 July 2012 21:09:26 Manjunath Hadli wrote: > > > Add documentation on the Davinci VPFE driver. Document the subdevs, > > > and private IOTCLs the dr

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-25 Thread Laurent Pinchart
Hi Manjunath, Please ignore the previous reply, I've hit the sent button too soon by mistake. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-25 Thread Laurent Pinchart
Hi Manjunath, On Tuesday 17 July 2012 10:43:54 Hadli, Manjunath wrote: > On Sun, Jul 15, 2012 at 18:16:25, Laurent Pinchart wrote: > > On Wednesday 11 July 2012 21:09:26 Manjunath Hadli wrote: > > > Add documentation on the Davinci VPFE driver. Document the subdevs, > > > and private IOTCLs the dr

RE: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-17 Thread Hadli, Manjunath
Hi Laurent, Thank you for your comments. On Sun, Jul 15, 2012 at 18:16:25, Laurent Pinchart wrote: > Hi Manjunath, > > Thanks for the patch. > > On Wednesday 11 July 2012 21:09:26 Manjunath Hadli wrote: > > Add documentation on the Davinci VPFE driver. Document the subdevs, > > and private I

Re: [PATCH] [media] davinci: vpfe: Add documentation

2012-07-15 Thread Laurent Pinchart
Hi Manjunath, Thanks for the patch. On Wednesday 11 July 2012 21:09:26 Manjunath Hadli wrote: > Add documentation on the Davinci VPFE driver. Document the subdevs, > and private IOTCLs the driver implements > > Signed-off-by: Manjunath Hadli > Signed-off-by: Lad, Prabhakar > --- > Documentati