Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-30 Thread Sascha Hauer
Hi, One more point for your devicetree conversions, On Mon, Apr 29, 2013 at 01:30:01PM +0530, Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com add OF support for the mt9p031 sensor driver. +static struct mt9p031_platform_data + *mt9p031_get_pdata(struct i2c_client

[PATCH 1/4] [media] s3c-camif: Remove redundant NULL check

2013-04-30 Thread Sachin Kamat
clk_unprepare checks for NULL pointer. Hence convert IS_ERR_OR_NULL to IS_ERR only. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/s3c-camif/camif-core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/4] [media] s3c-camif: Fix incorrect variable type

2013-04-30 Thread Sachin Kamat
'rotation' was an 8 bit variable and hence could not have values greater than 255. Since we need higher values, chnage it to 16 bit type. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/s3c-camif/camif-core.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 3/4] [media] s3c-camif: Staticize local symbols

2013-04-30 Thread Sachin Kamat
These symbols are local to the file and should be static. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/s3c-camif/camif-regs.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/s3c-camif/camif-regs.c

[PATCH 4/4] [media] s3c-camif: Use dev_info instead of printk

2013-04-30 Thread Sachin Kamat
dev_info is preferred to printk. Silences the related checkpatch warning. WARNING: Prefer netdev_info(netdev, ... then dev_info(dev, ... then pr_info(... to printk(KERN_INFO ... Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/s3c-camif/camif-regs.c |2 +- 1

Re: [PATCH] [media] s5c73m3: Fix off-by-one valid range checking for fie-index

2013-04-30 Thread Andrzej Hajda
Hi Axel, Thanks for the fix. On 30.04.2013 06:42, Axel Lin wrote: Current code uses fie-index as array subscript, thus the valid value range is 0 ... ARRAY_SIZE(s5c73m3_intervals) - 1. Signed-off-by: Axel Lin axel@ingics.com Acked-by: Andrzej Hajda a.ha...@samsung.com ---

[PATCH 1/1] [media] s5c73m3: Fix whitespace related warnings

2013-04-30 Thread Sachin Kamat
Silences the following type of warning: WARNING: space prohibited before semicolon Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: Andrzej Hajda a.ha...@samsung.com --- drivers/media/i2c/s5c73m3/s5c73m3-spi.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 1/2] [media] soc_camera: Constify dev_pm_ops in mt9t031.c

2013-04-30 Thread Sachin Kamat
Silences the following warning: WARNING: struct dev_pm_ops should normally be const Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/i2c/soc_camera/mt9t031.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/soc_camera/mt9t031.c

[PATCH 2/2] [media] soc_camera: Fix checkpatch warning in ov9640.c

2013-04-30 Thread Sachin Kamat
Silences the following warning: WARNING: please, no space before tabs Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/i2c/soc_camera/ov9640.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/soc_camera/ov9640.c

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-30 Thread Laurent Pinchart
Hi Sascha, On Tuesday 30 April 2013 08:16:25 Sascha Hauer wrote: On Mon, Apr 29, 2013 at 01:30:01PM +0530, Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com add OF support for the mt9p031 sensor driver. +static struct mt9p031_platform_data +

Re: [PATCH v9 02/20] V4L2: support asynchronous subdevice registration

2013-04-30 Thread Sascha Hauer
Hi Guennadi, On Fri, Apr 12, 2013 at 05:40:22PM +0200, Guennadi Liakhovetski wrote: Currently bridge device drivers register devices for all subdevices synchronously, tupically, during their probing. E.g. if an I2C CMOS sensor is attached to a video bridge device, the bridge driver will create

Re: [PATCH v9 02/20] V4L2: support asynchronous subdevice registration

2013-04-30 Thread Guennadi Liakhovetski
Hi Sascha On Tue, 30 Apr 2013, Sascha Hauer wrote: Hi Guennadi, On Fri, Apr 12, 2013 at 05:40:22PM +0200, Guennadi Liakhovetski wrote: Currently bridge device drivers register devices for all subdevices synchronously, tupically, during their probing. E.g. if an I2C CMOS sensor is

[PATCH] [media] exynos4-is: Fix off-by-one valid range checking for is-config_index

2013-04-30 Thread Axel Lin
Current code uses is-config_index as array subscript, thus the valid value range is 0 ... ARRAY_SIZE(cmd) - 1. Signed-off-by: Axel Lin axel@ingics.com --- drivers/media/platform/exynos4-is/fimc-is-regs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2 1/4] V4L2: soc_camera: Renesas R-Car VIN driver

2013-04-30 Thread Vladimir Barinov
Hi, Guennadi, Thank you for the review! Sergei Shtylyov wrote: I also strongly suspent some #include media/v4l2-*.h headers are missing above. Hm, I wonder which. I'm certainly not V4L2 expert... added following: #include media/v4l2-common.h #include media/v4l2-dev.h #include

cron job: media_tree daily build: WARNINGS

2013-04-30 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: Tue Apr 30 19:00:27 CEST 2013 git branch: test git hash: 02615ed5e1b2283db2495af3cf8f4ee172c77d80 gcc

Re: [alsa-devel] [PATCH] MAINTAINERS: Add linux-samsung-soc list to all related entries

2013-04-30 Thread Mark Brown
On Mon, Apr 22, 2013 at 03:23:29PM +0900, Kyungmin Park wrote: I don't think it's not required, each tree has each own mailing list. don't need to post all patches to samsung-soc list. It can be useful to get system level input on some stuff, I guess it mostly depends if the people on the

[PATCH] [RFC] mutex: w/w mutex slowpath debugging

2013-04-30 Thread Daniel Vetter
Injects EDEADLK conditions at pseudo-random interval, with exponential backoff up to UINT_MAX (to ensure that every lock operation still completes in a reasonable time). This way we can test the wound slowpath even for ww mutex users where contention is never expected, and the ww deadlock

Re: [PATCH v3 2/3] mutex: add support for wound/wait style locks, v3

2013-04-30 Thread Daniel Vetter
On Sun, Apr 28, 2013 at 07:04:07PM +0200, Maarten Lankhorst wrote: Changes since RFC patch v1: - Updated to use atomic_long instead of atomic, since the reservation_id was a long. - added mutex_reserve_lock_slow and mutex_reserve_lock_intr_slow - removed mutex_locked_set_reservation_id

Re: [PATCH] [RFC] mutex: w/w mutex slowpath debugging

2013-04-30 Thread Steven Rostedt
On Tue, 2013-04-30 at 20:45 +0200, Daniel Vetter wrote: /** diff --git a/kernel/mutex.c b/kernel/mutex.c index 66807c7..1cc3487 100644 --- a/kernel/mutex.c +++ b/kernel/mutex.c @@ -827,6 +827,35 @@ int __sched mutex_trylock(struct mutex *lock) EXPORT_SYMBOL(mutex_trylock); #ifndef

[PATCH] [RFC] mutex: w/w mutex slowpath debugging

2013-04-30 Thread Daniel Vetter
Injects EDEADLK conditions at pseudo-random interval, with exponential backoff up to UINT_MAX (to ensure that every lock operation still completes in a reasonable time). This way we can test the wound slowpath even for ww mutex users where contention is never expected, and the ww deadlock

Re: [PATCH v2 1/4] V4L2: soc_camera: Renesas R-Car VIN driver

2013-04-30 Thread Guennadi Liakhovetski
On Tue, 30 Apr 2013, Vladimir Barinov wrote: [snip] +static int rcar_vin_init_videobuf2(struct vb2_queue *vq, + struct soc_camera_device *icd) +{ +vq-type = V4L2_BUF_TYPE_VIDEO_CAPTURE; +vq-io_modes = VB2_MMAP | VB2_USERPTR; +vq-drv_priv =