Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-31 Thread Sylwester Nawrocki
On 05/30/2013 05:21 AM, Laurent Pinchart wrote: > Hi Sylwester, > > On Saturday 25 May 2013 16:11:52 Sylwester Nawrocki wrote: >> On 05/25/2013 11:17 AM, Prabhakar Lad wrote: [...] > And for synchronisation method on the analog part we could perhaps > define 'component-sync' or similar

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-31 Thread Sylwester Nawrocki
On 05/30/2013 05:21 AM, Laurent Pinchart wrote: Hi Sylwester, On Saturday 25 May 2013 16:11:52 Sylwester Nawrocki wrote: On 05/25/2013 11:17 AM, Prabhakar Lad wrote: [...] And for synchronisation method on the analog part we could perhaps define 'component-sync' or similar property that

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-29 Thread Laurent Pinchart
Hi Sylwester, On Saturday 25 May 2013 16:11:52 Sylwester Nawrocki wrote: > On 05/25/2013 11:17 AM, Prabhakar Lad wrote: > >> > From looking at Figure 8 "TVP7002 Application Example" in the TVP7002's > >> > datasheet ([2], p. 52) and your initial TVP7002 patches it looks like > >> > what you want

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-29 Thread Laurent Pinchart
Hi Sylwester, On Saturday 25 May 2013 16:11:52 Sylwester Nawrocki wrote: On 05/25/2013 11:17 AM, Prabhakar Lad wrote: From looking at Figure 8 TVP7002 Application Example in the TVP7002's datasheet ([2], p. 52) and your initial TVP7002 patches it looks like what you want is to specify

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Sylwester Nawrocki
On 05/25/2013 04:26 PM, Prabhakar Lad wrote: Thus it might make sense to have only following integer properties (added > as needed): > > composite-sync-active > sync-on-green-active > sync-on-comp-active > sync-on-luma-active > > This would allow to specify polarity of each signal and at

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Prabhakar Lad
Hi Sylwester, On Sat, May 25, 2013 at 7:41 PM, Sylwester Nawrocki wrote: > Hi, > > > On 05/25/2013 11:17 AM, Prabhakar Lad wrote: >>> >>> From looking at Figure 8 "TVP7002 Application Example" in the TVP7002's >>> > datasheet >>> > ([2], p. 52) and your initial TVP7002 patches it looks like

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Sylwester Nawrocki
Hi, On 05/25/2013 11:17 AM, Prabhakar Lad wrote: From looking at Figure 8 "TVP7002 Application Example" in the TVP7002's > datasheet > ([2], p. 52) and your initial TVP7002 patches it looks like what you want is > to > specify polarity of the SOGOUT signal, so the processor that receives

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Prabhakar Lad
Hi Sylwester, On Fri, May 24, 2013 at 4:41 PM, Sylwester Nawrocki wrote: > Prabhakar, > > > On 05/16/2013 03:18 PM, Lad Prabhakar wrote: >> >> From: Lad, Prabhakar >> >> This patch adds "sync-on-green" property as part of >> endpoint properties and also support to parse them in the parser. > >

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Prabhakar Lad
Hi Sylwester, On Fri, May 24, 2013 at 4:41 PM, Sylwester Nawrocki sylvester.nawro...@gmail.com wrote: Prabhakar, On 05/16/2013 03:18 PM, Lad Prabhakar wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com This patch adds sync-on-green property as part of endpoint properties and also

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Sylwester Nawrocki
Hi, On 05/25/2013 11:17 AM, Prabhakar Lad wrote: From looking at Figure 8 TVP7002 Application Example in the TVP7002's datasheet ([2], p. 52) and your initial TVP7002 patches it looks like what you want is to specify polarity of the SOGOUT signal, so the processor that receives this

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Prabhakar Lad
Hi Sylwester, On Sat, May 25, 2013 at 7:41 PM, Sylwester Nawrocki sylvester.nawro...@gmail.com wrote: Hi, On 05/25/2013 11:17 AM, Prabhakar Lad wrote: From looking at Figure 8 TVP7002 Application Example in the TVP7002's datasheet ([2], p. 52) and your initial TVP7002 patches it

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-25 Thread Sylwester Nawrocki
On 05/25/2013 04:26 PM, Prabhakar Lad wrote: Thus it might make sense to have only following integer properties (added as needed): composite-sync-active sync-on-green-active sync-on-comp-active sync-on-luma-active This would allow to specify polarity of each signal and at the same

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-24 Thread Sylwester Nawrocki
Prabhakar, On 05/16/2013 03:18 PM, Lad Prabhakar wrote: From: Lad, Prabhakar This patch adds "sync-on-green" property as part of endpoint properties and also support to parse them in the parser. --- a/Documentation/devicetree/bindings/media/video-interfaces.txt +++

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-24 Thread Sylwester Nawrocki
Prabhakar, On 05/16/2013 03:18 PM, Lad Prabhakar wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com This patch adds sync-on-green property as part of endpoint properties and also support to parse them in the parser. --- a/Documentation/devicetree/bindings/media/video-interfaces.txt +++

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-21 Thread Prabhakar Lad
Hi Laurent/Sylwester, On Thu, May 16, 2013 at 6:48 PM, Lad Prabhakar wrote: > From: Lad, Prabhakar > > This patch adds "sync-on-green" property as part of > endpoint properties and also support to parse them in the parser. > If you are ok with the patch can I send non RFC version of it ?

Re: [PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-21 Thread Prabhakar Lad
Hi Laurent/Sylwester, On Thu, May 16, 2013 at 6:48 PM, Lad Prabhakar prabhakar.cse...@gmail.com wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com This patch adds sync-on-green property as part of endpoint properties and also support to parse them in the parser. If you are ok with the

[PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-16 Thread Lad Prabhakar
From: Lad, Prabhakar This patch adds "sync-on-green" property as part of endpoint properties and also support to parse them in the parser. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc:

[PATCH RFC v2] media: OF: add sync-on-green endpoint property

2013-05-16 Thread Lad Prabhakar
From: Lad, Prabhakar prabhakar.cse...@gmail.com This patch adds sync-on-green property as part of endpoint properties and also support to parse them in the parser. Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com Cc: Hans Verkuil hans.verk...@cisco.com Cc: Laurent Pinchart