[PATCH 1/3] stb0899: store successful inversion for next run

2013-05-08 Thread Reinhard Nißl
Usually, inversion doesn't change in a system. Storing the last successful inversion value speeds up tuning of DVB-S2 transponders. Signed-off-by: Reinhard Nißl --- drivers/media/dvb-frontends/stb0899_algo.c | 8 +++- drivers/media/dvb-frontends/stb0899_drv.c | 5 ++--- 2 files changed, 9 i

[PATCH 2/3] stb0899: store autodetected inversion while tuning in non S2 mode

2013-05-08 Thread Reinhard Nißl
In non S2 mode, the device is able to autodetect inversion. So let's store it for tuning to S2 transponders. Signed-off-by: Reinhard Nißl --- drivers/media/dvb-frontends/stb0899_algo.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/media/dvb-frontends/stb0899_algo.c b/drive

[PATCH 3/3] stb0899: use autodetected inversion instead of configured inversion

2013-05-08 Thread Reinhard Nißl
For consistency, it is necessary to use the autodetected inversion instead of the configured one. Signed-off-by: Reinhard Nißl --- drivers/media/dvb-frontends/stb0899_algo.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/media/dvb-frontends/stb0899

Re: [RFC] Motion Detection API

2013-05-08 Thread Sylwester Nawrocki
Hi Sakari :-) On 05/08/2013 06:26 PM, Sakari Ailus wrote: On Tue, May 07, 2013 at 04:04:10PM +0200, Sylwester Nawrocki wrote: On 05/07/2013 02:35 PM, Hans Verkuil wrote: A metadata plane works well if you have substantial amounts of data (e.g. histogram data) but it has the disadvantage of re

[PATCH 2/2] stb0899: remove commented value from IQ_SWAP_ON/OFF usages

2013-05-08 Thread Reinhard Nißl
As the enum values have changed recently, the comments are void. Signed-off-by: Reinhard Nißl --- drivers/media/pci/mantis/mantis_vp1041.c | 2 +- drivers/media/pci/ttpci/budget-av.c | 2 +- drivers/media/pci/ttpci/budget-ci.c | 2 +- drivers/media/usb/dvb-usb/az6027.c | 2 +- dr

[PATCH 1/2] stb0899: fix inversion enum values to match usage with CFR

2013-05-08 Thread Reinhard Nißl
Throughout the zig-zag-implementations, inversion is taken into account when reading and writing the CFR register, which contains the derotator frequency. As swapping IQ signals changes the sign of that register for example, the idea is to compensate that sign change by multiplying the register val

[PATCH -next v2] media/usb: fix kconfig dependencies (aka bool depending on tristate considered harmful)

2013-05-08 Thread Randy Dunlap
egister_driver' drivers/built-in.o: In function `pvr_init': pvrusb2-main.c:(.init.text+0x4662): undefined reference to `usb_register_driver' drivers/built-in.o: In function `stk1160_usb_driver_init': stk1160-core.c:(.init.text+0x467d): undefined reference to `usb_register_driver' drivers

Re: [PATCH -next] media/usb: fix kconfig dependencies (aka bool depending on tristate considered harmful)

2013-05-08 Thread Yann E. MORIN
; Signed-off-by: Randy Dunlap > --- > drivers/media/usb/Kconfig |2 ++ > 1 file changed, 2 insertions(+) > > --- linux-next-20130508.orig/drivers/media/usb/Kconfig > +++ linux-next-20130508/drivers/media/usb/Kconfig > @@ -5,6 +5,7 @@ menuconfig MEDIA_USB_SUPPORT >

Re: [PATCH RESEND 9/9] s5k6aa: Convert to devm_gpio_request_one()

2013-05-08 Thread Sylwester Nawrocki
On 05/08/2013 03:46 PM, Laurent Pinchart wrote: Use the devm_gpio_request_one() managed function to simplify cleanup code paths. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to m

Re: [PATCH RESEND 8/9] s5c73m3: Convert to devm_gpio_request_one()

2013-05-08 Thread Sylwester Nawrocki
On 05/08/2013 03:46 PM, Laurent Pinchart wrote: Use the devm_gpio_request_one() managed function to simplify cleanup code paths. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to m

[PATCH] stb0899: enable auto inversion handling unconditionally

2013-05-08 Thread Reinhard Nißl
It seems that current inversion handling addresses only the signal routing on the PCB, i. e. IQ signals are either swapped or not. But when the device is operated in a Satellite Channel Router (SCR) environment, an additional inversion is required due to the way how the SCR works. Therefore it make

[PATCH 1/2] [media] v4l2: SI476X MFD - Do not use binary constants

2013-05-08 Thread Geert Uytterhoeven
Gcc < 4.3 doesn't understand binary constanrs (0b*): drivers/media/radio/radio-si476x.c:862:20: error: invalid suffix "b1000" on integer constant Hence use a hexadecimal constant (0x*) instead. Signed-off-by: Geert Uytterhoeven Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org ---

WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()

2013-05-08 Thread Borislav Petkov
This one looks legit: bw-qcam.c doesn't set q->timestamp_type. [ 146.989016] Colour QuickCam for Video4Linux v0.06 [ 147.713065] [ cut here ] [ 147.928854] WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142() [ 148.364433] CPU: 0 PID: 1

Capturing Nikon D7000 LiveView

2013-05-08 Thread Federico Prades Illanes
Hi all, I'm trying to connect a Nikon D7000 to my Linux box (a bit old Pentium IV) via USB. And I know from the camera spec and other software that it's able to send the Live View back to the computer. Obviously it's is working, I frankly have no idea how to debug this issue. I would expect to se

[PATCH -next] media/usb: fix kconfig dependencies (aka bool depending on tristate considered harmful)

2013-05-08 Thread Randy Dunlap
0x4662): undefined reference to `usb_register_driver' drivers/built-in.o: In function `stk1160_usb_driver_init': stk1160-core.c:(.init.text+0x467d): undefined reference to `usb_register_driver' drivers/built-in.o: In function `zr364xx_driver_exit': zr364xx.c:(.exit.text+0x1377

cron job: media_tree daily build: WARNINGS

2013-05-08 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 May 8 19:00:21 CEST 2013 git branch: test git hash: 02615ed5e1b2283db2495af3cf8f4ee172c77d80 gcc versi

Re: [RFC] Motion Detection API

2013-05-08 Thread Sakari Ailus
Hi Sylwester, On Tue, May 07, 2013 at 04:04:10PM +0200, Sylwester Nawrocki wrote: > On 05/07/2013 02:35 PM, Hans Verkuil wrote: > > A metadata plane works well if you have substantial amounts of data (e.g. > > histogram > > data) but it has the disadvantage of requiring you to use the MPLANE buff

Media controller versus int device in V4L2

2013-05-08 Thread Phillip Norisez
I have the following question, who or what can help me with some information on the specific differences, from a programming viewpoint, between the "media controller" and "int device" frameworks for V4L2? A checklist for forward and back porting is what I seek, but I don't expect such a thing t

[PATCH RESEND 4/9] media: i2c: Convert to devm_kzalloc()

2013-05-08 Thread Laurent Pinchart
Using the managed function the kfree() calls can be removed from the probe error path and the remove handler. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki Acked-by: Lad, Prabhakar Acked-by: Andy Shevchenko Reviewed-by: Benoît Thébaudeau --- drivers/media/i2c/ad9389b.c

[PATCH RESEND 2/9] tvp514x: Fix double free

2013-05-08 Thread Laurent Pinchart
The tvp514x data structure is allocated using devm_kzalloc(). Freeing it explictly would result in a double free. Fix it. Signed-off-by: Laurent Pinchart Acked-by: Lad, Prabhakar --- drivers/media/i2c/tvp514x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/i2c/tvp514x.c b/dri

[PATCH RESEND 8/9] s5c73m3: Convert to devm_gpio_request_one()

2013-05-08 Thread Laurent Pinchart
Use the devm_gpio_request_one() managed function to simplify cleanup code paths. Signed-off-by: Laurent Pinchart --- drivers/media/i2c/s5c73m3/s5c73m3-core.c | 79 +++- 1 file changed, 28 insertions(+), 51 deletions(-) diff --git a/drivers/media/i2c/s5c73m3/s5c73m3-c

[PATCH RESEND 7/9] m5mols: Convert to devm_request_irq()

2013-05-08 Thread Laurent Pinchart
Using the managed function the free_irq() calls can be removed from the probe error path and the remove handler. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki --- drivers/media/i2c/m5mols/m5mols_core.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --g

[PATCH RESEND 9/9] s5k6aa: Convert to devm_gpio_request_one()

2013-05-08 Thread Laurent Pinchart
Use the devm_gpio_request_one() managed function to simplify cleanup code paths. Signed-off-by: Laurent Pinchart --- drivers/media/i2c/s5k6aa.c | 73 +- 1 file changed, 26 insertions(+), 47 deletions(-) diff --git a/drivers/media/i2c/s5k6aa.c b/driver

[PATCH RESEND 6/9] media: i2c: Convert to devm_regulator_bulk_get()

2013-05-08 Thread Laurent Pinchart
Using the managed function the regulator_bulk_put() calls can be removed from the probe error path and the remove handler. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki --- drivers/media/i2c/m5mols/m5mols_core.c | 8 +++- drivers/media/i2c/noon010pc30.c| 8 ++-- 2

[PATCH RESEND 5/9] media: i2c: Convert to devm_gpio_request_one()

2013-05-08 Thread Laurent Pinchart
Using the managed function the gpio_free() calls can be removed from the probe error path and the remove handler. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki --- drivers/media/i2c/adv7183.c| 24 drivers/media/i2c/m5mols/m5mols_core.c | 9 ++

[PATCH RESEND 3/9] media: i2c: Convert to gpio_request_one()

2013-05-08 Thread Laurent Pinchart
Replace gpio_request() with gpio_request_one() and remove the associated gpio_direction_output() calls. Signed-off-by: Laurent Pinchart Acked-by: Sylwester Nawrocki --- drivers/media/i2c/adv7183.c| 14 +++--- drivers/media/i2c/m5mols/m5mols_core.c | 6 -- drivers/media/

[PATCH RESEND 0/9] media: i2c: Convert to the devm_* API

2013-05-08 Thread Laurent Pinchart
Hello, I'm resending this patch set in the hope that it will reach the linux-media list this time. The previous version never made it to the list (and thus neither did it reach patchwork), possibly because of the large CC list. I've thus kept the list e-mail address only, sorry for the inconvenien

[PATCH RESEND 1/9] mt9v032: Free control handler in cleanup paths

2013-05-08 Thread Laurent Pinchart
The control handler must be freed in the probe error path and in the remove handler. Signed-off-by: Laurent Pinchart --- drivers/media/i2c/mt9v032.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/media/i2c/mt9v032.c b/drivers/media/i2c/mt9v032.c index 3f356cb..

Re: [PATCH 1/2] Print more detailed parse error messages

2013-05-08 Thread Sascha Hauer
On Wed, May 08, 2013 at 03:27:53PM +0200, Sascha Hauer wrote: > The following errors usually resulted in the same 'Unable to parse link' > message: > > - one of the given entities does not exist > - one of the pads of a given entity does not exist > - No link exists between given pads > - syntax e

[PATCH 2/2] Print parser position on error

2013-05-08 Thread Sascha Hauer
Most parser functions take a **endp argument to indicate the caller where parsing has stopped. This is currently only used after parsing something successfully. This patch sets **endp to the erroneous position in the error case and prints its position after an error has occured. Signed-off-by: Sas

[PATCH] media-ctl error messages

2013-05-08 Thread Sascha Hauer
Hi All, As a first time media-ctl user it was quite frustrating to see that whatever I did media-ctl only responded with "Unable to parse link". The following is an attempt to add some more detailed error messages. With this applied media-ctl can answer with something like: No pad '1' on entity "

[PATCH 1/2] Print more detailed parse error messages

2013-05-08 Thread Sascha Hauer
The following errors usually resulted in the same 'Unable to parse link' message: - one of the given entities does not exist - one of the pads of a given entity does not exist - No link exists between given pads - syntax error in link description Add more detailed error messages to give the user

[PATCH] [media] stb0899: allow minimum symbol rate of 1000000

2013-05-08 Thread Zoran Turalija
This makes minimum symbol rate driver capabilities on par with windows driver, and allows tuning on linux to transponders that have symbol rate below 500, too. Patch was tested successfully on Eutelsat 16A transponders that became reachable with it (100 < symbol rate < 500): * D

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

2013-05-08 Thread Prabhakar Lad
Hi Laurent, On Wed, May 8, 2013 at 4:07 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > On Wednesday 08 May 2013 10:19:57 Prabhakar Lad wrote: >> On Wed, May 8, 2013 at 7:32 AM, Laurent Pinchart wrote: >> > On Tuesday 07 May 2013 15:10:36 Prabhakar Lad wrote: >> >> On Mon, May 6, 2013 at 8:29 PM,

Re: [PATCH 1/1] mt9t031: Fix panic on probe

2013-05-08 Thread Guennadi Liakhovetski
Hi Philippe On Wed, 8 May 2013, Philippe Rétornaz wrote: > The video device is not yet valid when probe() is called. > Call directly soc_camera_power_on/off() instead of calling mt9t031_s_power(). > > Signed-off-by: Philippe Rétornaz There is already a patch for this: https://patchwork.kernel

[PATCH 1/1] mt9t031: Fix panic on probe

2013-05-08 Thread Philippe Rétornaz
The video device is not yet valid when probe() is called. Call directly soc_camera_power_on/off() instead of calling mt9t031_s_power(). Signed-off-by: Philippe Rétornaz --- drivers/media/i2c/soc_camera/mt9t031.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/

RE: [RFC PATCH for 3.10] Update Codec section in DocBook

2013-05-08 Thread Kamil Debski
Hi, > From: Hans Verkuil [mailto:hverk...@xs4all.nl] > Sent: Tuesday, May 07, 2013 6:47 PM > > On Tue 7 May 2013 17:42:57 Kamil Debski wrote: > > Hi Hans, > > > > Thanks for this patch. I remember that there was some discussion > about > > it and the conclusion was that a codec device is in fact

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

2013-05-08 Thread Laurent Pinchart
Hi Prabhakar, On Wednesday 08 May 2013 10:19:57 Prabhakar Lad wrote: > On Wed, May 8, 2013 at 7:32 AM, Laurent Pinchart wrote: > > On Tuesday 07 May 2013 15:10:36 Prabhakar Lad wrote: > >> On Mon, May 6, 2013 at 8:29 PM, Prabhakar Lad wrote: > >> > On Fri, May 3, 2013 at 8:04 PM, Arnd Bergmann wr

Re: Support of RTL2832U+R820T

2013-05-08 Thread Karsten Malcher
Am 08.05.2013 04:18, schrieb poma: Oh dear. :) http://git.linuxtv.org/media_build.git Typical usage is: … poma Thank you for the quick answer! I didn't see this. Next week i will try it after my holidays. Karsten -- To unsubscribe from this list: send the line "unsubscribe linux-media" i

Re: [RFC 0/2] V4L2 API for exposing flash subdevs as LED class device

2013-05-08 Thread Andrzej Hajda
On 07.05.2013 17:07, Laurent Pinchart wrote: > On Tuesday 07 May 2013 02:11:27 Kim, Milo wrote: >> On Monday, May 06, 2013 6:34 PM Andrzej Hajda wrote: >>> This RFC proposes generic API for exposing flash subdevices via LED >>> framework. >>> >>> Rationale >>> >>> Currently there are two frameworks

Re: Hauppauge WinTV HVR 930C-HD - new USB ID 2040:b130 ?

2013-05-08 Thread Jan Kiszka
On 2013-05-08 05:29, Devin Heitmueller wrote: > On Tue, May 7, 2013 at 6:39 PM, Jan Kiszka wrote: >> To pick up this old topic (as I just got the wrong 930C delivered :( ): >> >> What is blocking the development of a si2165 driver? Lacking specs (due >> to NDAs)? Or lacking interest / developer ba