cron job: media_tree daily build: WARNINGS

2012-11-21 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date:Wed Nov 21 19:00:23 CET 2012 git hash:40ad4a30299fd01ae2a7b3c76089e64517f05d3b gcc version: i686-linux-gcc (GCC

Re: [PATCH v2 00/12] Media Controller capture driver for DM365

2012-11-21 Thread Sakari Ailus
Hi Prabhakar, On Fri, Nov 16, 2012 at 08:15:02PM +0530, Prabhakar Lad wrote: > From: Manjunath Hadli > > This patch set adds media controller based capture driver for > DM365. > > This driver bases its design on Laurent Pinchart's Media Controller Design > whose patches for Media Controller and

Re: [PATCH] adv7604: convert struct i2c_msg initialization to C99 format

2012-11-21 Thread Hans Verkuil
On Thu October 25 2012 07:02:36 Shubhrajyoti D wrote: > Convert the struct i2c_msg initialization to C99 format. This makes > maintaining and editing the code simpler. Also helps once other fields > like transferred are added in future. > > Signed-off-by: Shubhrajyoti D Acked-by: Hans Verkuil

Re: [PATCH] [media] exynos-gsc: propagate timestamps from src to dst buffers

2012-11-21 Thread Sakari Ailus
Hi Sylwester and Shaik, On Mon, Nov 19, 2012 at 11:06:34PM +0100, Sylwester Nawrocki wrote: > On 11/07/2012 07:40 AM, Shaik Ameer Basha wrote: > >Make gsc-m2m propagate the timestamp field from source to destination > >buffers > > We probably need some means for letting know the mem-to-mem driver

[PATCH v1.2 1/4] v4l: Define video buffer flags for timestamp types

2012-11-21 Thread Sakari Ailus
Define video buffer flags for different timestamp types. Everything up to now have used either realtime clock or monotonic clock, without a way to tell which clock the timestamp was taken from. Also document that the clock source of the timestamp in the timestamp field depends on buffer flags. Si

Fw: [PATCH] [media] ivtv: ivtv-driver: Replace 'flush_work_sync()'

2012-11-21 Thread Mauro Carvalho Chehab
Hi Andy, I'm understanding that you'll be reviewing this patch. So, I'm marking it as under_review at patchwork. Thanks, Mauro Forwarded message: Date: Wed, 24 Oct 2012 10:14:16 -0200 From: Fabio Estevam To: awa...@md.metrocast.net Cc: mche...@infradead.org, linux-media@vger.kernel.org, t...@k

Re: [PATCH v12 4/6] fbmon: add of_videomode helpers

2012-11-21 Thread Tomi Valkeinen
On 2012-11-21 18:24, Steffen Trumtrar wrote: > On Wed, Nov 21, 2012 at 02:49:30PM +0200, Tomi Valkeinen wrote: >>> @@ -715,6 +717,11 @@ extern void fb_destroy_modedb(struct fb_videomode >>> *modedb); >>> extern int fb_find_mode_cvt(struct fb_videomode *mode, int margins, int >>> rb); >>> exter

Re: [PATCH v12 1/6] video: add display_timing and videomode

2012-11-21 Thread Tomi Valkeinen
On 2012-11-21 18:11, Steffen Trumtrar wrote: > Hi, > > On Wed, Nov 21, 2012 at 01:37:08PM +0200, Tomi Valkeinen wrote: >>> +#include >> >> What is this needed for? u32? I don't see it defined in types.h >> > > Yes, u32. What would be the right header for that if not types.h? Yes, it looks like

Re: [PATCH v12 4/6] fbmon: add of_videomode helpers

2012-11-21 Thread Steffen Trumtrar
On Wed, Nov 21, 2012 at 02:49:30PM +0200, Tomi Valkeinen wrote: > On 2012-11-20 17:54, Steffen Trumtrar wrote: > > Add helper to get fb_videomode from devicetree. > > > > Signed-off-by: Steffen Trumtrar > > Reviewed-by: Thierry Reding > > Acked-by: Thierry Reding > > Tested-by: Thierry Reding

Re: [PATCH v12 1/6] video: add display_timing and videomode

2012-11-21 Thread Steffen Trumtrar
Hi, On Wed, Nov 21, 2012 at 01:37:08PM +0200, Tomi Valkeinen wrote: > Hi, > > On 2012-11-20 17:54, Steffen Trumtrar wrote: > > Add display_timing structure and the according helper functions. This allows > > the description of a display via its supported timing parameters. > > > > Every timing p

[PATCH] s5p-fimc: Don't use mutex_lock_interruptible() in device release()

2012-11-21 Thread Sylwester Nawrocki
Use uninterruptible mutex_lock in the release() file op to make sure all resources are properly freed when a process is being terminated. Returning -ERESTARTSYS has no effect for a terminating process and this caused driver resources no to be released. Not releasing the buffer queue also prevented

[PATCH] exynos-gsc: Don't use mutex_lock_interruptible() in device release()

2012-11-21 Thread Sylwester Nawrocki
Use uninterruptible mutex_lock in the release() file op to make sure all resources are properly freed when a process is being terminated. Returning -ERESTARTSYS has no effect for a terminating process and this may cause driver resources no to be released. Cc: Shaik Ameer Basha Reported-by: Marek

Re: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Thierry Reding
On Wed, Nov 21, 2012 at 09:03:38AM -0600, Rob Herring wrote: > On 11/21/2012 05:52 AM, Thierry Reding wrote: > > On Wed, Nov 21, 2012 at 12:48:43PM +0100, Steffen Trumtrar wrote: > >> Hi! > >> > >> On Wed, Nov 21, 2012 at 10:12:43AM +, Manjunathappa, Prakash wrote: > >>> Hi Steffen, > >>> > >>>

Re: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Rob Herring
On 11/21/2012 05:52 AM, Thierry Reding wrote: > On Wed, Nov 21, 2012 at 12:48:43PM +0100, Steffen Trumtrar wrote: >> Hi! >> >> On Wed, Nov 21, 2012 at 10:12:43AM +, Manjunathappa, Prakash wrote: >>> Hi Steffen, >>> >>> On Tue, Nov 20, 2012 at 21:24:52, Steffen Trumtrar wrote: +/** + *

[GIT PULL FOR 3.8] Miscellaneous fixes

2012-11-21 Thread Hans de Goede
Hi Mauro, Please pull from my tree for a bunch of miscellaneous fixes for 3.8 . The following changes since commit 8f7e91a31fb95c50880c76505b416630c0326d93: [media] smiapp-pll: Constify limits argument to smiapp_pll_calculate() (2012-10-29 09:51:38 -0200) are available in the git repository

Re: [PATCH v12 0/6] of: add display helper

2012-11-21 Thread Laurent Pinchart
On Wednesday 21 November 2012 09:28:22 Steffen Trumtrar wrote: > On Tue, Nov 20, 2012 at 08:35:31PM +0100, Thierry Reding wrote: > > On Tue, Nov 20, 2012 at 07:11:29PM +0100, Robert Schwebel wrote: > > > On Tue, Nov 20, 2012 at 05:13:19PM +0100, Laurent Pinchart wrote: > > > > On Tuesday 20 Novembe

Re: [PATCH v12 4/6] fbmon: add of_videomode helpers

2012-11-21 Thread Laurent Pinchart
Hi Tomi, On Wednesday 21 November 2012 14:49:30 Tomi Valkeinen wrote: > On 2012-11-20 17:54, Steffen Trumtrar wrote: > > Add helper to get fb_videomode from devicetree. > > > > Signed-off-by: Steffen Trumtrar > > Reviewed-by: Thierry Reding > > Acked-by: Thierry Reding > > Tested-by: Thierry R

Re: [PATCH v12 6/6] drm_modes: add of_videomode helpers

2012-11-21 Thread Tomi Valkeinen
On 2012-11-20 17:54, Steffen Trumtrar wrote: > Add helper to get drm_display_mode from devicetree. > > Signed-off-by: Steffen Trumtrar > Reviewed-by: Thierry Reding > Acked-by: Thierry Reding > Tested-by: Thierry Reding > Tested-by: Philipp Zabel > Reviewed-by: Laurent Pinchart > --- > driv

Re: [PATCH v12 5/6] drm_modes: add videomode helpers

2012-11-21 Thread Tomi Valkeinen
On 2012-11-20 17:54, Steffen Trumtrar wrote: > Add conversion from videomode to drm_display_mode > > Signed-off-by: Steffen Trumtrar > Reviewed-by: Thierry Reding > Acked-by: Thierry Reding > Tested-by: Thierry Reding > Tested-by: Philipp Zabel > Reviewed-by: Laurent Pinchart > --- > driver

Re: [PATCH v12 4/6] fbmon: add of_videomode helpers

2012-11-21 Thread Tomi Valkeinen
On 2012-11-20 17:54, Steffen Trumtrar wrote: > Add helper to get fb_videomode from devicetree. > > Signed-off-by: Steffen Trumtrar > Reviewed-by: Thierry Reding > Acked-by: Thierry Reding > Tested-by: Thierry Reding > Tested-by: Philipp Zabel > Reviewed-by: Laurent Pinchart > --- > drivers/

Re: [PATCH v12 3/6] fbmon: add videomode helpers

2012-11-21 Thread Tomi Valkeinen
On 2012-11-20 17:54, Steffen Trumtrar wrote: > diff --git a/include/linux/fb.h b/include/linux/fb.h > index c7a9571..920cbe3 100644 > --- a/include/linux/fb.h > +++ b/include/linux/fb.h > @@ -14,6 +14,7 @@ > #include > #include > #include > +#include No need for this, just add "struct xxx;

Re: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Tomi Valkeinen
On 2012-11-20 17:54, Steffen Trumtrar wrote: > +timings subnode > +--- > + > +required properties: > + - hactive, vactive: Display resolution > + - hfront-porch, hback-porch, hsync-len: Horizontal Display timing parameters > + in pixels > + vfront-porch, vback-porch, vsync-len: Ver

Re: [PATCH 133/493] remove use of __devexit_p

2012-11-21 Thread Guennadi Liakhovetski
On Mon, 19 Nov 2012, Bill Pemberton wrote: > drivers/media/platform/sh_vou.c | 2 +- > drivers/media/platform/soc_camera/atmel-isi.c| 2 +- > drivers/media/platform/soc_camera/mx2_camera.c | 2 +- > drivers/media/platform/soc_camera/mx3_camera.c

Re: [PATCH] exynos-gsc: Add missing video device vfl_dir flag initialization

2012-11-21 Thread Shaik Ameer Basha
Hi Sylwester, I applied this patch and tested exynos-gsc driver basic features on exynos5250. It's working fine for me. Thanks, Shaik Ameer Basha On Tue, Nov 20, 2012 at 1:36 AM, Sylwester Nawrocki wrote: > Hi Shaik, > > Could you let us know if the driver is working fine with this patch > appl

Re: [PATCH v12 3/6] fbmon: add videomode helpers

2012-11-21 Thread Steffen Trumtrar
Hi! On Wed, Nov 21, 2012 at 04:39:01PM +0530, Leela Krishna Amudala wrote: > Yes, > Even I got the same build error. > later I fixed it by including "#include " > > Best Wishes, > Leela Krishna. > > On Wed, Nov 21, 2012 at 3:39 PM, Manjunathappa, Prakash > wrote: > > Hi Steffen, > > > > I am tr

Re: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Thierry Reding
On Wed, Nov 21, 2012 at 12:48:43PM +0100, Steffen Trumtrar wrote: > Hi! > > On Wed, Nov 21, 2012 at 10:12:43AM +, Manjunathappa, Prakash wrote: > > Hi Steffen, > > > > On Tue, Nov 20, 2012 at 21:24:52, Steffen Trumtrar wrote: > > > +/** > > > + * of_get_display_timings - parse all display_tim

Re: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Steffen Trumtrar
Hi! On Wed, Nov 21, 2012 at 10:12:43AM +, Manjunathappa, Prakash wrote: > Hi Steffen, > > On Tue, Nov 20, 2012 at 21:24:52, Steffen Trumtrar wrote: > > +/** > > + * of_get_display_timings - parse all display_timing entries from a > > device_node > > + * @np: device_node with the subnodes > >

Re: [PATCH v12 1/6] video: add display_timing and videomode

2012-11-21 Thread Tomi Valkeinen
Hi, On 2012-11-20 17:54, Steffen Trumtrar wrote: > Add display_timing structure and the according helper functions. This allows > the description of a display via its supported timing parameters. > > Every timing parameter can be specified as a single value or a range > . > > Also, add helper fu

[GIT PULL FOR v3.8] Davinci VPIF trivial fixes

2012-11-21 Thread Prabhakar Lad
Hi Mauro, Can you please pull the following patches for vpif, which fixes some trivial issues. Thanks and Regards, --Prabhakar Lad The following changes since commit 2c4e11b7c15af70580625657a154ea7ea70b8c76: [media] siano: fix RC compilation (2012-11-07 11:09:08 +0100) are available in the g

Re: [PATCH v12 3/6] fbmon: add videomode helpers

2012-11-21 Thread Leela Krishna Amudala
Yes, Even I got the same build error. later I fixed it by including "#include " Best Wishes, Leela Krishna. On Wed, Nov 21, 2012 at 3:39 PM, Manjunathappa, Prakash wrote: > Hi Steffen, > > I am trying to add DT support for da8xx-fb driver on top of your patches. > Encountered below build error.

RE: [PATCH v12 2/6] video: add of helper for videomode

2012-11-21 Thread Manjunathappa, Prakash
Hi Steffen, On Tue, Nov 20, 2012 at 21:24:52, Steffen Trumtrar wrote: > This adds support for reading display timings from DT or/and convert one of > those > timings to a videomode. > The of_display_timing implementation supports multiple children where each > property can have up to 3 values. Al

RE: [PATCH v12 3/6] fbmon: add videomode helpers

2012-11-21 Thread Manjunathappa, Prakash
Hi Steffen, I am trying to add DT support for da8xx-fb driver on top of your patches. Encountered below build error. Sorry for reporting it late. On Tue, Nov 20, 2012 at 21:24:53, Steffen Trumtrar wrote: > Add a function to convert from the generic videomode to a fb_videomode. > > Signed-off-by:

Re: [PATCH v12 0/6] of: add display helper

2012-11-21 Thread Steffen Trumtrar
On Tue, Nov 20, 2012 at 08:35:31PM +0100, Thierry Reding wrote: > On Tue, Nov 20, 2012 at 07:11:29PM +0100, Robert Schwebel wrote: > > On Tue, Nov 20, 2012 at 05:13:19PM +0100, Laurent Pinchart wrote: > > > On Tuesday 20 November 2012 16:54:50 Steffen Trumtrar wrote: > > > > Hi! > > > > > > > > Ch