Re: Mystique SaTiX-S2 Dual

2010-07-23 Thread Emmanuel
OJ a écrit : I am using this card: http://www.linuxtv.org/wiki/index.php/Mystique_SaTiX-S2_Dual (v2) According to the wiki I should use the ngene driver, but I am not able to compile it. Downloaded the latest version from Mercury yesterday. Error message when compiling: $ make ngene make -C /ho

Re: [RFC] Per-subdev, host-specific data

2010-07-23 Thread Andy Walls
On Fri, 2010-07-23 at 16:31 +0200, Laurent Pinchart wrote: > Hi Hans, > > On Friday 23 July 2010 15:46:29 Hans Verkuil wrote: > > On Friday 23 July 2010 15:01:29 Laurent Pinchart wrote: > > > Hi everybody, > > > > > > Trying to implement support for multiple sensors connected to the same > > > OM

[cron job] v4l-dvb daily build 2.6.22 and up: ERRORS, 2.6.16-2.6.21: ERRORS

2010-07-23 Thread Hans Verkuil
This message is generated daily by a cron job that builds v4l-dvb for the kernels and architectures in the list below. Results of the daily build of v4l-dvb: date:Fri Jul 23 19:00:21 CEST 2010 path:http://www.linuxtv.org/hg/v4l-dvb changeset: 14993:9652f85e688a git master:

[PATCH v3 1/8] ARM: Samsung: Add register definitions for Samsung S5P SoC camera interface

2010-07-23 Thread Sylwester Nawrocki
Add register definitions for the camera interface/video postprocessor contained in Samsung's S5P SoC series. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park Reviewed-by: Pawel Osciak Reviewed-by: Marek Szyprowski --- arch/arm/plat-samsung/include/plat/regs-fimc.h | 294 +++

RE: [SAMPLE v2 04/12] v4l-subdev: Add pads operations

2010-07-23 Thread Karicheri, Muralidharan
Laurent, Could you explain the probe and active usage using an example such as below? Link1Link2 input sensor -> ccdc -> video node. Assume Link2 we can have either format 1 or format 2 for capture. Thanks. Murali Karicheri Software Design Engineer Texas Instruments Inc. Germa

RE: [RFC] Per-subdev, host-specific data

2010-07-23 Thread Sylwester Nawrocki
Hello, > -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Laurent Pinchart > Sent: Friday, July 23, 2010 3:01 PM > To: Linux Media Mailing List > Cc: Sakari Ailus; Guennadi Liakhovetski > Subject: [RFC] Per-subdev, host

Re: [RFC] Per-subdev, host-specific data

2010-07-23 Thread Laurent Pinchart
Hi Hans, On Friday 23 July 2010 15:46:29 Hans Verkuil wrote: > On Friday 23 July 2010 15:01:29 Laurent Pinchart wrote: > > Hi everybody, > > > > Trying to implement support for multiple sensors connected to the same > > OMAP3 ISP input (all but one of the sensors need to be kept in reset > > obvi

Re: [RFC] Per-subdev, host-specific data

2010-07-23 Thread Hans Verkuil
On Friday 23 July 2010 15:01:29 Laurent Pinchart wrote: > Hi everybody, > > Trying to implement support for multiple sensors connected to the same OMAP3 > ISP input (all but one of the sensors need to be kept in reset obviously), I > need to associate host-specific data to the sensor subdevs. >

Re: [patch -next] V4L: media/IR: testing the wrong variable

2010-07-23 Thread Jarod Wilson
On Fri, Jul 23, 2010 at 12:08:26PM +0200, Dan Carpenter wrote: > There is a typo here. We meant to test "rbuf" instead of "drv". We > already tested "drv" earlier. > > Signed-off-by: Dan Carpenter Gah. I swear that got fixed once already... Thanks! Acked-by: Jarod Wilson -- Jarod Wilson j

Re: [patch -next] V4L: au0828: move dereference below sanity checks

2010-07-23 Thread Devin Heitmueller
On Fri, Jul 23, 2010 at 6:09 AM, Dan Carpenter wrote: > This function has sanity checks to make sure that "dev" is non-null.  I > moved the dereference down below the checks.  In the current code "dev" > is never actually null. > > Signed-off-by: Dan Carpenter > > diff --git a/drivers/media/video

[RFC] Per-subdev, host-specific data

2010-07-23 Thread Laurent Pinchart
Hi everybody, Trying to implement support for multiple sensors connected to the same OMAP3 ISP input (all but one of the sensors need to be kept in reset obviously), I need to associate host-specific data to the sensor subdevs. The terms host and bridge are considered as synonyms in the rest of

Re: [patch -next] V4L: ivtv: remove unneeded NULL checks

2010-07-23 Thread Dan Carpenter
On Fri, Jul 23, 2010 at 07:46:47AM -0400, Andy Walls wrote: > On Fri, 2010-07-23 at 12:12 +0200, Dan Carpenter wrote: > > In ivtvfb_callback_cleanup() we dereference "itv" before checking that > > it's NULL. "itv" is assigned with container_of() which basically never > > returns a NULL pointer so

Re: [patch -next] V4L: ivtv: remove unneeded NULL checks

2010-07-23 Thread Andy Walls
On Fri, 2010-07-23 at 12:12 +0200, Dan Carpenter wrote: > In ivtvfb_callback_cleanup() we dereference "itv" before checking that > it's NULL. "itv" is assigned with container_of() which basically never > returns a NULL pointer so the check is pointless. I removed it, along > with a similar check

RE: [PATCH] mediabus: add MIPI CSI-2 pixel format codes

2010-07-23 Thread Guennadi Liakhovetski
Hi Sylwester On Fri, 23 Jul 2010, Sylwester Nawrocki wrote: > Hi Laurent, > > > -Original Message- > > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > > ow...@vger.kernel.org] On Behalf Of Laurent Pinchart > > Sent: Friday, July 23, 2010 10:35 AM > > To: Guennadi Liakhovet

[patch -next] V4L: ivtv: remove unneeded NULL checks

2010-07-23 Thread Dan Carpenter
In ivtvfb_callback_cleanup() we dereference "itv" before checking that it's NULL. "itv" is assigned with container_of() which basically never returns a NULL pointer so the check is pointless. I removed it, along with a similar check in ivtvfb_callback_init(). I considered adding a check for v4l2

[patch -next] V4L: au0828: move dereference below sanity checks

2010-07-23 Thread Dan Carpenter
This function has sanity checks to make sure that "dev" is non-null. I moved the dereference down below the checks. In the current code "dev" is never actually null. Signed-off-by: Dan Carpenter diff --git a/drivers/media/video/au0828/au0828-video.c b/drivers/media/video/au0828/au0828-video.c

[patch -next] V4L: media/IR: testing the wrong variable

2010-07-23 Thread Dan Carpenter
There is a typo here. We meant to test "rbuf" instead of "drv". We already tested "drv" earlier. Signed-off-by: Dan Carpenter diff --git a/drivers/media/IR/ir-lirc-codec.c b/drivers/media/IR/ir-lirc-codec.c index 178bc5b..87e 100644 --- a/drivers/media/IR/ir-lirc-codec.c +++ b/drivers/medi

RE: [PATCH] mediabus: add MIPI CSI-2 pixel format codes

2010-07-23 Thread Sylwester Nawrocki
Hi Laurent, > -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Laurent Pinchart > Sent: Friday, July 23, 2010 10:35 AM > To: Guennadi Liakhovetski > Cc: Linux Media Mailing List; Hans Verkuil > Subject: Re: [PATCH] medi

Re: [PATCH] mediabus: add MIPI CSI-2 pixel format codes

2010-07-23 Thread Guennadi Liakhovetski
On Fri, 23 Jul 2010, Laurent Pinchart wrote: > Hi Guennadi, > > On Friday 23 July 2010 10:13:37 Guennadi Liakhovetski wrote: > > Add pixel format codes, defined in the MIPI CSI-2 specification. > > > > Signed-off-by: Guennadi Liakhovetski > > --- > > > > Even though it affects the same enum as

Re: [PATCH] mediabus: add MIPI CSI-2 pixel format codes

2010-07-23 Thread Laurent Pinchart
Hi Guennadi, On Friday 23 July 2010 10:13:37 Guennadi Liakhovetski wrote: > Add pixel format codes, defined in the MIPI CSI-2 specification. > > Signed-off-by: Guennadi Liakhovetski > --- > > Even though it affects the same enum as my patch from yesterday, they are > independent, Hans and Laure

[PATCH] mediabus: add MIPI CSI-2 pixel format codes

2010-07-23 Thread Guennadi Liakhovetski
Add pixel format codes, defined in the MIPI CSI-2 specification. Signed-off-by: Guennadi Liakhovetski --- Even though it affects the same enum as my patch from yesterday, they are independent, Hans and Laurent CCed just to avoid possible conflicts, when further patching this file. include/me