cron job: media_tree daily build: ERRORS

2015-01-16 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 Jan 17 04:00:22 CET 2015 git branch: test git hash: 99f3cd52aee21091ce62442285a68873e3be833f gcc versio

RE: [possible BUG, cx23885] Dual tuner TV card, works using one tuner only, doesn't work if both tuners are used

2015-01-16 Thread dCrypt
Hi, James. After searching for somebody posting some issues similar to mine, I think this one you posted to the mailing list can be related: https://www.mail-archive.com/linux-media%40vger.kernel.org/msg80078.html I'm having problems using both tuners in a dual tuner card (Terratec Cinergy T P

[PATCH] [media] cx231xx: fix usbdev leak on failure paths in cx231xx_usb_probe()

2015-01-16 Thread Alexey Khoroshilov
Commit b7085c086475 ("cx231xx: convert from pr_foo to dev_foo") moves usb_get_dev(interface_to_usbdev(interface)) to the beginning of cx231xx_usb_probe() to use udev->dev in dev_err(), but it does not make sure usbdev is put on all failure paths. Later dev_err(udev->dev) was replaced by dev_err(d)

Re: [PATCH 2/2] si2168: add support for 1.7MHz bandwidth

2015-01-16 Thread Prabhakar Lad
Hi Olli, Thanks for the patch. On Fri, Jan 16, 2015 at 12:35 PM, Olli Salonen wrote: > This patch is based on Antti's silabs branch. > > Add support for 1.7 MHz bandwidth. Supported in all versions of Si2168 > according to short data sheets. > > Signed-off-by: Olli Salonen > --- > drivers/me

[PATCHv2] si2168: return error if set_frontend is called with invalid parameters

2015-01-16 Thread Olli Salonen
This patch is based on Antti's silabs branch. According to dvb-frontend.h set_frontend may be called with bandwidth_hz set to 0 if automatic bandwidth is required. Si2168 does not support automatic bandwidth and does not declare FE_CAN_BANDWIDTH_AUTO in caps. This patch will change the behaviou

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Hans Verkuil
On 01/16/2015 05:20 PM, Raimonds Cicans wrote: > On 16.01.2015 16:54, Hans Verkuil wrote: >> On 01/13/2015 06:55 PM, Raimonds Cicans wrote: >>> On 13.01.2015 16:01, Hans Verkuil wrote: Can you both test this patch? It should (I hope) solve the problems you both had with the cx23885 driver

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Hans Verkuil
On 01/16/2015 05:48 PM, Luis Alves wrote: > Hans, > > There is another guy having issues with TBS8820 card (uses cx88 and cx24116) > > His syslog: > http://paste.ubuntu.com/9284564/ > > The stackdump makes me believe that the issue also appeared since > "[media] cx88: convert to vb2" > (still to

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Luis Alves
Hans, There is another guy having issues with TBS8820 card (uses cx88 and cx24116) His syslog: http://paste.ubuntu.com/9284564/ The stackdump makes me believe that the issue also appeared since "[media] cx88: convert to vb2" (still to confirm) Regards, Luis On Fri, Jan 16, 2015 at 4:20 PM, Ra

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Jurgen Kramer
On Fri, 2015-01-16 at 15:58 +0100, Hans Verkuil wrote: > On 01/15/2015 05:32 PM, Jurgen Kramer wrote: > > Hi Hans, > > > > On Tue, 2015-01-13 at 17:59 +0100, Jurgen Kramer wrote: > >> Hi Hans, > >> On Tue, 2015-01-13 at 15:01 +0100, Hans Verkuil wrote: > >>> Hi Raimonds, Jurgen, > >>> > >>> Can yo

randconfig build error with next-20150116, in drivers/media/v4l2-core/tuner-core.c

2015-01-16 Thread Jim Davis
Building with the attached random configuration file, drivers/built-in.o: In function `set_type': tuner-core.c:(.text+0x245dd0): undefined reference to `tea5767_attach' tuner-core.c:(.text+0x245f5f): undefined reference to `xc2028_attach' tuner-core.c:(.text+0x2460e3): undefined reference to `tda1

[RFC PATCH 0/5] media: rcar_vin: Fixes for buffer management

2015-01-16 Thread William Towle
The following is a subset of our work in progress branch for video support on the Renesas "Lager" board, comprising hotfixes for video buffer management. We are successfully capturing single frames and video with the complete branch, and intend to follow up with stable patches from the branch

[PATCH 1/2] media: rcar_vin: helper function for streaming stop

2015-01-16 Thread William Towle
From: Ian Molton The code that tests that capture from a stream has stopped is presently insufficient and the potential for a race condition exists where frame capture may generate an interrupt between requesting the capture process halt and freeing buffers. This patch refactors code out of rcar

[PATCH 2/2] media: rcar_vin: move buffer management to .stop_streaming handler

2015-01-16 Thread William Towle
This commit moves the "buffer in use" logic from the .buf_cleanup handler into .stop_streaming, based on advice that this is its proper logical home. By ensuring the list of pointers in priv->queue_buf[] is managed as soon as possible, we avoid warnings concerning buffers in ACTIVE state when the

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Raimonds Cicans
On 16.01.2015 16:54, Hans Verkuil wrote: On 01/13/2015 06:55 PM, Raimonds Cicans wrote: On 13.01.2015 16:01, Hans Verkuil wrote: Can you both test this patch? It should (I hope) solve the problems you both had with the cx23885 driver. Can you check that the function cx23885_risc_field in driv

Re: [PATCH/RFC v10 03/19] DT: leds: Add led-sources property

2015-01-16 Thread Jacek Anaszewski
On 01/16/2015 02:48 PM, Rob Herring wrote: On Fri, Jan 16, 2015 at 3:07 AM, Jacek Anaszewski wrote: On 01/15/2015 03:24 PM, Rob Herring wrote: On Tue, Jan 13, 2015 at 2:42 AM, Jacek Anaszewski wrote: On 01/12/2015 05:55 PM, Rob Herring wrote: Adding Mark B and Liam... On Mon, Jan 12, 2

[PATCH 1/2] media: rcar_vin: Fix race condition terminating stream

2015-01-16 Thread William Towle
From: Ian Molton The potential for a race condition exists where frame capture may generate an interrupt between requesting the capture process halt and freeing buffers. Introduce rcar_vin_wait_stop_streaming() and call it in appropriate places so we ensure capturing has finished where this is c

[PATCH 2/2] media: rcar_vin: move buffer management to .stop_streaming handler

2015-01-16 Thread William Towle
This commit moves the "buffer in use" logic from the .buf_cleanup handler into .stop_streaming, based on advice that this is its proper logical home. By ensuring the list of pointers in priv->queue_buf[] is managed as soon as possible, we avoid warnings concerning buffers in ACTIVE state when the

Re: [PATCH 00/15] media: blackfin: bfin_capture enhancements

2015-01-16 Thread Hans Verkuil
Hi Scott, Any idea when you will have time to test this? Regards, Hans On 12/26/2014 08:13 AM, Scott Jiang wrote: > Hi Lad, > > I'm on holiday these days. I will test these patches later. > > Thanks, > Scott > > 2014-12-20 18:47 GMT+08:00 Lad, Prabhakar : >> Hi Scott, >> >> Although

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Hans Verkuil
On 01/15/2015 05:32 PM, Jurgen Kramer wrote: > Hi Hans, > > On Tue, 2015-01-13 at 17:59 +0100, Jurgen Kramer wrote: >> Hi Hans, >> On Tue, 2015-01-13 at 15:01 +0100, Hans Verkuil wrote: >>> Hi Raimonds, Jurgen, >>> >>> Can you both test this patch? It should (I hope) solve the problems you >>> bot

Re: [PATCH] cx23885/vb2 regression: please test this patch

2015-01-16 Thread Hans Verkuil
On 01/13/2015 06:55 PM, Raimonds Cicans wrote: > On 13.01.2015 16:01, Hans Verkuil wrote: >> Hi Raimonds, Jurgen, >> >> Can you both test this patch? It should (I hope) solve the problems you >> both had with the cx23885 driver. >> >> This patch fixes a race condition in the vb2_thread that occurs

Re: [PATCH 1/2] si2168: return error if set_frontend is called with invalid parameters

2015-01-16 Thread Prabhakar Lad
Hi Olli, Thanks for the patch. On Fri, Jan 16, 2015 at 12:35 PM, Olli Salonen wrote: > This patch should is based on Antti's silabs branch. > > According to dvb-frontend.h set_frontend may be called with bandwidth_hz set > to 0 if automatic bandwidth is required. Si2168 does not support automat

Re: [PATCH 2/2] si2168: add support for 1.7MHz bandwidth

2015-01-16 Thread Tycho Lürsen
Hi Olli, did you commit this anywhere? Regards, Tycho. Op 16-01-15 om 13:35 schreef Olli Salonen: This patch is based on Antti's silabs branch. Add support for 1.7 MHz bandwidth. Supported in all versions of Si2168 according to short data sheets. Signed-off-by: Olli Salonen --- drivers/med

Re: [PATCH 2/2] si2168: add support for 1.7MHz bandwidth

2015-01-16 Thread Antti Palosaari
On 01/16/2015 02:35 PM, Olli Salonen wrote: This patch is based on Antti's silabs branch. Add support for 1.7 MHz bandwidth. Supported in all versions of Si2168 according to short data sheets. Signed-off-by: Olli Salonen Reviewed-by: Antti Palosaari How about tuner driver filters? Having

Re: [PATCH 1/2] si2168: return error if set_frontend is called with invalid parameters

2015-01-16 Thread Antti Palosaari
On 01/16/2015 02:35 PM, Olli Salonen wrote: This patch should is based on Antti's silabs branch. According to dvb-frontend.h set_frontend may be called with bandwidth_hz set to 0 if automatic bandwidth is required. Si2168 does not support automatic bandwidth and does not declare FE_CAN_BANDWID

Re: [PATCH/RFC v10 03/19] DT: leds: Add led-sources property

2015-01-16 Thread Rob Herring
On Fri, Jan 16, 2015 at 3:07 AM, Jacek Anaszewski wrote: > On 01/15/2015 03:24 PM, Rob Herring wrote: >> >> On Tue, Jan 13, 2015 at 2:42 AM, Jacek Anaszewski >> wrote: >>> >>> On 01/12/2015 05:55 PM, Rob Herring wrote: Adding Mark B and Liam... On Mon, Jan 12, 2015 at 10:

[PATCH 2/2] si2168: add support for 1.7MHz bandwidth

2015-01-16 Thread Olli Salonen
This patch is based on Antti's silabs branch. Add support for 1.7 MHz bandwidth. Supported in all versions of Si2168 according to short data sheets. Signed-off-by: Olli Salonen --- drivers/media/dvb-frontends/si2168.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/dvb-fron

[PATCH 1/2] si2168: return error if set_frontend is called with invalid parameters

2015-01-16 Thread Olli Salonen
This patch should is based on Antti's silabs branch. According to dvb-frontend.h set_frontend may be called with bandwidth_hz set to 0 if automatic bandwidth is required. Si2168 does not support automatic bandwidth and does not declare FE_CAN_BANDWIDTH_AUTO in caps. This patch will change the b

[GIT FIXES FOR v3.19] Fixes for 3.19

2015-01-16 Thread Hans Verkuil
The following changes since commit 99f3cd52aee21091ce62442285a68873e3be833f: [media] vb2-vmalloc: Protect DMA-specific code by #ifdef CONFIG_HAS_DMA (2014-12-23 16:28:09 -0200) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v3.19a for you to fetch cha

[GIT PULL FOR v3.20] Fixes, cleanups, improvements

2015-01-16 Thread Hans Verkuil
Hi Mauro, This pull request contains various fixes, cleanups and improvements. The only notable change is the addition of unpacking and logging functions for InfoFrames to drivers/video/hdmi.c. Thierry was OK with taking this via the media tree (http://www.spinics.net/lists/linux-media/msg84655.h

Re: [PATCH] [media] [pvrusb2]: remove dead retry cmd code

2015-01-16 Thread Haim Daniel
Removing 8 years old dead code seemed right to silly me. On Fri, 2015-01-16 at 12:37 +0100, Hans Verkuil wrote: > On 01/16/2015 12:29 PM, Haim Daniel wrote: > > It looks that "if (try_count < 20) continue" jumps to end of the do ... > > while(0) loop and goes out. > > Ah, you are right. But that

Re: [PATCH] [media] [pvrusb2]: remove dead retry cmd code

2015-01-16 Thread Hans Verkuil
On 01/16/2015 12:29 PM, Haim Daniel wrote: > It looks that "if (try_count < 20) continue" jumps to end of the do ... > while(0) loop and goes out. Ah, you are right. But that is obviously not what was intended, so just removing it is not a proper 'fix'. Mike, can you take a look at this? Regard

Re: [PATCH] [media] [pvrusb2]: remove dead retry cmd code

2015-01-16 Thread Haim Daniel
It looks that "if (try_count < 20) continue" jumps to end of the do ... while(0) loop and goes out. --hd. On Fri, 2015-01-16 at 11:57 +0100, Hans Verkuil wrote: > On 01/05/2015 11:38 PM, Haim Daniel wrote: > > In case a command is timed out, current flow sets the retry_flag > > and does nothing.

Re: [PATCH] BLACKFIN MEDIA DRIVER: rewrite the blackfin style of read/write into common style

2015-01-16 Thread Hans Verkuil
Hi Hao, Why would you do this? read/writew() is AFAICT not the same as bfin_read/write16 (defined in arch/blackfin/include/asm/def_LPBlackfin.h). And all other blackfin sources I've seen all use bfin_read/write. So unless there is a good reason for this change I am not going to accept this. Rega

[PATCH v2 1/2] dvb: tua6034: add a new driver for Infineon tua6034 tuner

2015-01-16 Thread tskd08
From: Akihiro Tsukada this 3-band tuner is used in Friio (dvb-usb-friio), and it was buried in friio-fe.c. Signed-off-by: Akihiro Tsukada --- drivers/media/tuners/Kconfig | 7 + drivers/media/tuners/Makefile | 1 + drivers/media/tuners/tua6034.c | 464 +++

[PATCH v2 2/2] dvb-usb-friio: split and merge into dvb-usbv2-gl861

2015-01-16 Thread tskd08
From: Akihiro Tsukada A Friio device consists of a GL861 adapter/bridge chip, a TC90522 demod chip and a TUA6034 tuner chip, but the friio driver was implemented as one combined driver. This patch separates off the each chip drivers and re-uses the existing modules: dvb-usbv2-gl861,tc90522. It a

[PATCH v2 0/2] split dvb-usb-friio into parts

2015-01-16 Thread tskd08
From: Akihiro Tsukada This patch series decomposes the friio driver which was monolithic into adapter,demod,tuner modules. Changes in v2: - dvb-usbv2-gl861: adapt to i2c template patch v3, fe_i2c_client was moved from dvb_frontend* to priv Akihiro Tsukada (2): dvb: tua6034: add a new dri

Re: [PATCH 2/2] [media] adv7180: Remove the unneeded 'err' label

2015-01-16 Thread Lars-Peter Clausen
On 12/16/2014 05:49 PM, Fabio Estevam wrote: There is no need to jump to the 'err' label as we can simply return the error code directly and make the code shorter. Signed-off-by: Fabio Estevam Acked-by: Lars-Peter Clausen -- To unsubscribe from this list: send the line "unsubscribe linux-me

[PATCH v3 3/4] dvb: tc90522: use dvb-core i2c binding model template

2015-01-16 Thread tskd08
From: Akihiro Tsukada Signed-off-by: Akihiro Tsukada --- drivers/media/dvb-frontends/tc90522.c | 64 --- drivers/media/dvb-frontends/tc90522.h | 8 ++--- 2 files changed, 34 insertions(+), 38 deletions(-) diff --git a/drivers/media/dvb-frontends/tc90522.c b/dr

[PATCH v3 4/4] dvb: earth-pt3: use dvb-core i2c binding model template

2015-01-16 Thread tskd08
From: Akihiro Tsukada Signed-off-by: Akihiro Tsukada --- drivers/media/pci/pt3/pt3.c | 85 ++--- drivers/media/pci/pt3/pt3.h | 11 +++--- 2 files changed, 32 insertions(+), 64 deletions(-) diff --git a/drivers/media/pci/pt3/pt3.c b/drivers/media/pci/pt3/

[PATCH v3 1/4] dvb: qm1d1c0042: use dvb-core i2c binding model template

2015-01-16 Thread tskd08
From: Akihiro Tsukada Signed-off-by: Akihiro Tsukada --- drivers/media/tuners/qm1d1c0042.c | 60 +-- drivers/media/tuners/qm1d1c0042.h | 2 -- 2 files changed, 19 insertions(+), 43 deletions(-) diff --git a/drivers/media/tuners/qm1d1c0042.c b/drivers/media

[PATCH v3 2/4] dvb: mxl301rf: use dvb-core i2c binding model template

2015-01-16 Thread tskd08
From: Akihiro Tsukada Signed-off-by: Akihiro Tsukada --- drivers/media/tuners/mxl301rf.c | 50 +++-- drivers/media/tuners/mxl301rf.h | 2 +- 2 files changed, 14 insertions(+), 38 deletions(-) diff --git a/drivers/media/tuners/mxl301rf.c b/drivers/media/tune

[PATCH v3 0/4] modify earth-pt3 and its dependees to use i2c template

2015-01-16 Thread tskd08
From: Akihiro Tsukada This patch series depends on the previous patch: "[PATCH v3]dvb-core: add template code for i2c binding model" The adapter(earth-pt3), its demod (tc90522) and tuners (mxl301rf, qm1d1c0042) are ported to dvb-core i2c template. Changes in v3: - tc90522,earth-pt3: adapt to i2

[PATCH v3] dvb-core: add template code for i2c binding model

2015-01-16 Thread tskd08
From: Akihiro Tsukada Define a standard interface for demod/tuner i2c driver modules. A module client calls dvb_i2c_attach_{fe,tuner}(), and a module driver defines struct dvb_i2c_module_param and calls DEFINE_DVB_I2C_MODULE() macro. This template provides implicit module requests and ref-counti

Re: [PATCHv3 0/3] hdmi: add unpack and logging functions

2015-01-16 Thread Thierry Reding
On Fri, Dec 19, 2014 at 01:14:19PM +0100, Hans Verkuil wrote: > This patch series adds new HDMI 2.0/CEA-861-F defines to hdmi.h and > adds unpacking and logging functions to hdmi.c. It also uses those > in the V4L2 adv7842 driver (and they will be used in other HDMI drivers > once this functionalit

Re: [PATCH] [media] [pvrusb2]: remove dead retry cmd code

2015-01-16 Thread Hans Verkuil
On 01/05/2015 11:38 PM, Haim Daniel wrote: > In case a command is timed out, current flow sets the retry_flag > and does nothing. Really? That's not how I read the code: it retries up to 20 times before bailing out. Perhaps you missed the "if (try_count < 20) continue;" line? Regards, H

Re: [PATCH] media: pci: solo6x10: solo6x10-enc.c: Remove unused function

2015-01-16 Thread Hans Verkuil
(resent with correct email address for Ismael) Ismael, Andrey, Can you take a look at this? Shouldn't solo_s_jpeg_qp() be hooked up to something? Regards, Hans On 12/21/2014 06:58 PM, Rickard Strandqvist wrote: > Remove the function solo_s_jpeg_qp() that is not used anywhere. > > Thi

Re: [PATCH] media: pci: solo6x10: solo6x10-enc.c: Remove unused function

2015-01-16 Thread Hans Verkuil
Ismael, Andrey, Can you take a look at this? Shouldn't solo_s_jpeg_qp() be hooked up to something? Regards, Hans On 12/21/2014 06:58 PM, Rickard Strandqvist wrote: > Remove the function solo_s_jpeg_qp() that is not used anywhere. > > This was partially found by using a static code ana

Re: [PATCH 15/16] [media] adv7180: Add free run mode controls

2015-01-16 Thread Hans Verkuil
Hi Lars, On 01/13/2015 01:01 PM, Lars-Peter Clausen wrote: > The adv7180 (and similar) has support for a so called free run mode in which > it will output a predefined test signal. This patch adds support for > configuring the various aspects of the so called free run mode. > > The patch adds thr

Re: [PATCH/RFC v10 03/19] DT: leds: Add led-sources property

2015-01-16 Thread Sylwester Nawrocki
Hi, On 15/01/15 22:03, Pavel Machek wrote: >> Perhaps we could use the 'reg' property to describe actual connections, >> > I'm not sure if it's better than a LED specific property, e.g. >> > >> > max77387@52 { >> > compatible = "nxp,max77387"; >> > #address-cells = <2>; >> >

Re: [PATCH 08/16] [media] adv7180: Consolidate video mode setting

2015-01-16 Thread Hans Verkuil
Hi Lars, On 01/13/2015 01:01 PM, Lars-Peter Clausen wrote: > We have basically the same code to set the video standard in init_device() > and adv7180_s_std(). Factor this out into a common helper function. > > Signed-off-by: Lars-Peter Clausen > --- > drivers/media/i2c/adv7180.c | 67 > +++

[PATCH 3/3 v2] bttv: Improve TEA575x support

2015-01-16 Thread Ondrej Zary
Improve g_tuner and add s_hw_freq_seek and enum_freq_bands support for cards with TEA575x radio. This allows signal/stereo detection and HW seek to work on these cards. Signed-off-by: Ondrej Zary --- drivers/media/pci/bt8xx/bttv-driver.c | 31 +++ 1 file changed, 3

Re: [PATCH 3/3] bttv: Improve TEA575x support

2015-01-16 Thread Hans Verkuil
Hi Ondrej, Just two small comments: On 01/15/2015 09:10 PM, Ondrej Zary wrote: > Improve g_tuner and add s_hw_freq_seek and enum_freq_bands support for cards > with TEA575x radio. > > This allows signal/stereo detection and HW seek to work on these cards. > > Signed-off-by: Ondrej Zary > --- >

[GIT PULL FOR v3.20] Remove deprecated drivers

2015-01-16 Thread Hans Verkuil
As promised, remove the deprecated tlg2300, vino, saa7191, bw-qcam, c-qcam and pms drivers. Regards, Hans The following changes since commit 99f3cd52aee21091ce62442285a68873e3be833f: [media] vb2-vmalloc: Protect DMA-specific code by #ifdef CONFIG_HAS_DMA (2014-12-23 16:28:09 -0200)

Re: [PATCH/RFC v10 03/19] DT: leds: Add led-sources property

2015-01-16 Thread Jacek Anaszewski
On 01/15/2015 03:24 PM, Rob Herring wrote: On Tue, Jan 13, 2015 at 2:42 AM, Jacek Anaszewski wrote: On 01/12/2015 05:55 PM, Rob Herring wrote: Adding Mark B and Liam... On Mon, Jan 12, 2015 at 10:10 AM, Jacek Anaszewski wrote: On 01/12/2015 02:52 PM, Rob Herring wrote: On Mon, Jan 12,