Mistake on the colorspace page in the API doc

2013-05-25 Thread Wouter Thielen
Hi all, I have been trying to get the colors right in the images grabbed from my webcam, and I tried the color conversion code on http://linuxtv.org/downloads/v4l-dvb-apis/colorspaces.html. It turned out to be very white, so I checked out the intermediate steps, and thought the part: ER = clamp

Re: [PATCH v2 0/4] media: i2c: ths7303 cleanup

2013-05-25 Thread Laurent Pinchart
On Saturday 25 May 2013 23:09:32 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Trivial cleanup of the driver. > > Changes for v2: > 1: Dropped the asynchronous probing and, OF >support patches will be handling them independently because of > dependencies. 2: Arranged the patches logically

Re: [PATCH v2 4/5] media: davinci: vpif_capture: Convert to devm_* api

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:35 Prabhakar Lad wrote: > From: Lad, Prabhakar > > use devm_request_irq() instead of request_irq(). > This ensures more consistent error values and simplifies error paths. > > use module_platform_driver to simplify the code.

Re: [PATCH v2 3/5] media: davinci: vpif: remove unnecessary braces around defines

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:34 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Signed-off-by: Lad, Prabhakar > --- > drivers/media/platform/davinci/vpif.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/med

Re: [PATCH v2 1/5] media: davinci: vpif: remove unwanted header mach/hardware.h and sort the includes alphabetically

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:32 Prabhakar Lad wrote: > From: Lad, Prabhakar > > This patch removes unwanted header include of mach/hardware.h > and along side sorts the header inclusion alphabetically. > > Signed-off-by: Lad, Prabhakar Acked-by: Lauren

Re: [PATCH v2 2/5] media: davinci: vpif: Convert to devm_* api

2013-05-25 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Saturday 25 May 2013 22:06:33 Prabhakar Lad wrote: > From: Lad, Prabhakar > > Use devm_ioremap_resource instead of reques_mem_region()/ioremap(). > This ensures more consistent error values and simplifies error paths. > > Signed-off-by: Lad, Prabhakar

cron job: media_tree daily build: WARNINGS

2013-05-25 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: Sat May 25 19:00:22 CEST 2013 git branch: test git hash: 6a084d6b3dc200b855ae8a3c6771abe285a3835d gcc versi

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 th

Re: [PATCH v2 1/4] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-25 Thread Prabhakar Lad
Hi Sekhar, On Sat, May 25, 2013 at 11:09 PM, Prabhakar Lad wrote: > From: Lad, Prabhakar > > remove init_enable from ths7303 pdata as it is being dropped > from ths7303_platform_data. > Can you please ack this patch as I intend to take this patch via media tree. Regards, --Prabhakar Lad -- To u

[PATCH v2 2/4] media: i2c: ths7303: remove init_enable option from pdata

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch removes init_enable option from pdata, the init_enable was intended that the device should start streaming video immediately but ideally the bridge drivers should call s_stream explicitly for such devices to start video. Signed-off-by: Lad, Prabhakar Cc: Hans Ver

[PATCH v2 1/4] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar remove init_enable from ths7303 pdata as it is being dropped from ths7303_platform_data. Signed-off-by: Lad, Prabhakar Cc: Sekhar Nori Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: linux-ker...@vger.kernel.org Cc: davinci-linux-open-sou...@linux.da

[PATCH v2 0/4] media: i2c: ths7303 cleanup

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Trivial cleanup of the driver. Changes for v2: 1: Dropped the asynchronous probing and, OF support patches will be handling them independently because of dependencies. 2: Arranged the patches logically so that git bisect succeeds. Lad, Prabhakar (4): ARM: davinci:

[PATCH v2 3/4] media: i2c: ths7303: remove unnecessary function ths7303_setup()

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar the ths7303_setup() was doing the same thing as ths7303_setval() except that ths7303_setval() sets it to some particular mode. This patch removes ths7303_setup() function and calls ths7303_setval() in the probe setting the device to 480I_576I filter mode in the probe. Signed

[PATCH v2 4/4] media: i2c: ths7303: make the pdata as a constant pointer

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar generally the pdata needs to be a constant pointer in the device state structure. This patch makes the pdata as a constant pointer and alongside returns -EINVAL when pdata is NULL. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Cheh

Re: InstantFM

2013-05-25 Thread Patrice Levesque
> This, as well as the "Invalid freq '12715'" and the "get_baseline: > min=65535.00 max=65535.00" messages seem to indicate that only > is being read from all the registers of the tuner chip, so > somehow the communication between the usb micro-controller and the > si470x tuner ch

[PATCH v2 5/5] media: davinci: vpif_display: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar use devm_request_irq() instead of request_irq(). This ensures more consistent error values and simplifies error paths. use module_platform_driver to simplify the code. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif_display.c | 61 +

[PATCH v2 3/5] media: davinci: vpif: remove unnecessary braces around defines

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/davinci/vpif.c b/drivers/media/platform/davinci/vpif.c index 164c1b7..d9269c9 100644 --- a/drivers

[PATCH v2 4/5] media: davinci: vpif_capture: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar use devm_request_irq() instead of request_irq(). This ensures more consistent error values and simplifies error paths. use module_platform_driver to simplify the code. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif_capture.c | 73 +

[PATCH v2 2/5] media: davinci: vpif: Convert to devm_* api

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar Use devm_ioremap_resource instead of reques_mem_region()/ioremap(). This ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c | 27 --- 1 files changed, 4 insertio

[PATCH v2 1/5] media: davinci: vpif: remove unwanted header mach/hardware.h and sort the includes alphabetically

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch removes unwanted header include of mach/hardware.h and along side sorts the header inclusion alphabetically. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpif.c | 10 -- 1 files changed, 4 insertions(+), 6 deletions(-) diff --git a

[PATCH v2 0/5] media: davinci: vpif trivial cleanup

2013-05-25 Thread Prabhakar Lad
From: Lad, Prabhakar This patch series cleans the VPIF driver, uses devm_* api wherever required and uses module_platform_driver() to simplify the code. This patch series applies on http://git.linuxtv.org/hverkuil/media_tree.git/ shortlog/refs/heads/for-v3.11 and is tested on OMAP-L138. Changes

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 wh

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 th

[PATCH 5/5] s5p-mfc: Remove unused s5p_mfc_get_decoded_status_v6() function

2013-05-25 Thread Sylwester Nawrocki
This patch fixes following compilation warning: CC [M] drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.o drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c:1733:12: warning: ‘s5p_mfc_get_decoded_status_v6’ defined but not used It assigns existing but not used s5p_mfc_get_dec_status_v6() function t

[PATCH 4/5] s5p-tv: Don't ignore return value of regulator_bulk_enable() in hdmi_drv.c

2013-05-25 Thread Sylwester Nawrocki
This patch fixes following compilation warning: CC [M] drivers/media/platform/s5p-tv/hdmi_drv.o drivers/media/platform/s5p-tv/hdmi_drv.c: In function ‘hdmi_resource_poweron’: drivers/media/platform/s5p-tv/hdmi_drv.c:583:23: warning: ignoring return value of ‘regulator_bulk_enable’, declared with

[PATCH 3/5] s5p-tv: Do not ignore regulator/clk API return values in sdo_drv.c

2013-05-25 Thread Sylwester Nawrocki
This patch fixes following compilation warning: drivers/media/platform/s5p-tv/sdo_drv.c: In function ‘sdo_runtime_resume’: drivers/media/platform/s5p-tv/sdo_drv.c:268:18: warning: ignoring return value of ‘regulator_enable’, declared with attribute warn_unused_result drivers/media/platform/s5p-

[PATCH 2/5] s5p-tv: Don't ignore return value of regulator_enable() in sii9234_drv.c

2013-05-25 Thread Sylwester Nawrocki
This patch fixes following compilation warning: CC [M] drivers/media/platform/s5p-tv/sii9234_drv.o drivers/media/platform/s5p-tv/sii9234_drv.c: In function ‘sii9234_runtime_resume’: drivers/media/platform/s5p-tv/sii9234_drv.c:252:18: warning: ignoring return value of ‘regulator_enable’, decl

[PATCH 1/5] s5c73m3: Do not ignore errors from regulator_enable()

2013-05-25 Thread Sylwester Nawrocki
This fixes following compilation warning: drivers/media/i2c/s5c73m3/s5c73m3-core.c: In function ‘__s5c73m3_power_off’: drivers/media/i2c/s5c73m3/s5c73m3-core.c:1389:19: warning: ignoring return value of ‘regulator_enable’, declared with attribute warn_unused_result Cc: Andrzej Hajda Signed-off-b

[PATCH 0/5] arm/exynos compilation warning fixes

2013-05-25 Thread Sylwester Nawrocki
This patch set fixes couple issues that cause warnings seen in the media daily build. Any testing/ack are welcome, as these patches are completely untested yet. Sylwester Nawrocki (5): s5c73m3: Do not ignore errors from regulator_enable() s5p-tv: Don't ignore return value of regulator_enable

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. > > >