[PATCH v2 04/15] [media] lirc_dev: replace printk with pr_* or dev_*

2016-07-01 Thread Andi Shyti
This patch mutes also all the checkpatch warnings related to printk. Reword all the printouts so that the string doesn't need to be split, which fixes the following checkpatch warning: WARNING: quoted string split across lines Signed-off-by: Andi Shyti ---

[PATCH v2 02/15] [media] lirc_dev: allow bufferless driver registration

2016-07-01 Thread Andi Shyti
Some drivers don't necessarily need to have a FIFO managed buffer for their transfers. Drivers now should call lirc_register_bufferless_driver in order to handle the buffer themselves. The function works exaclty like lirc_register_driver except of the buffer allocation. Signed-off-by: Andi Shyti

[PATCH v2 03/15] [media] lirc_dev: remove unnecessary debug prints

2016-07-01 Thread Andi Shyti
Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 25 - 1 file changed, 25 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index fa562a3..ee997ab 100644 --- a/drivers/media/rc/lirc_dev.c +++

[PATCH v2 05/15] [media] lirc_dev: simplify goto paths

2016-07-01 Thread Andi Shyti
The code can be rearranged so that some goto paths can be removed Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 34 -- 1 file changed, 12 insertions(+), 22 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH v2 13/15] [media] lirc_dev: extremely trivial comment style fix

2016-07-01 Thread Andi Shyti
Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index 99d1f98..4b3efcf 100644 --- a/drivers/media/rc/lirc_dev.c +++

[PATCH v2 06/15] [media] lirc_dev: do not use goto to create loops

2016-07-01 Thread Andi Shyti
... use "do .. while" instead. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index 0d988c9..da1777c 100644 ---

[PATCH v2 00/15] lirc_dev fixes and beautification

2016-07-01 Thread Andi Shyti
Hi, After applying Joe's suggestion, the next patches had some conflicts, therefore I have to send all the 15 patches again. This is a collection of fixes, added functionality, coding rework and trivial coding style fixes. The first patch is preparatory to the second, which allows the user to

[PATCH v2 01/15] [media] lirc_dev: place buffer allocation on separate function

2016-07-01 Thread Andi Shyti
During the driver registration, move the buffer allocation on a separate function. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 57 +++-- 1 file changed, 34 insertions(+), 23 deletions(-) diff --git

[PATCH v2 14/15] [media] lirc_dev: fix potential segfault

2016-07-01 Thread Andi Shyti
When opening or closing a lirc character device, the framework provides to the user the possibility to keep track of opening or closing of the device by calling two functions: - set_use_inc() when opening the device - set_use_dec() when closing the device if those are not set by the lirc user,

Re: square-only image on Nokia N900 camera -- pipeline setup in python (was Re: [RFC PATCH 00/24] Make Nokia N900 cameras working)

2016-07-01 Thread Pavel Machek
Hi! > On gitlab is the latest version of pipeline setup if python. I also > got fcam to work (slowly) on the camera, with autofocus and > autogain. Capturing from preview modes works fine, but image quality > is not good, as expected. Capturing raw GRBG10 images works, but > images are square,

Re: [PATCH v6 10/11] Input: synaptics-rmi4 - add support for F54 diagnostics

2016-07-01 Thread Nick Dyer
On Thu, Jun 30, 2016 at 06:38:53PM +0100, Nick Dyer wrote: > Function 54 implements access to various RMI4 diagnostic features. > > This patch adds support for retrieving this data. It registers a V4L2 > device to output the data to user space. > > Signed-off-by: Nick Dyer >

Re: [PATCH v2 15/15] [media] include: lirc: add LIRC_GET_LENGTH command

2016-07-01 Thread Sean Young
On Fri, Jul 01, 2016 at 05:01:38PM +0900, Andi Shyti wrote: > Added the get length command to allow userspace users to check on > the data length. So what does LIRC_GET_LENGTH do? If you want to add an ioctl, it need justification, documenting in

[PATCH v2 15/15] [media] include: lirc: add LIRC_GET_LENGTH command

2016-07-01 Thread Andi Shyti
Added the get length command to allow userspace users to check on the data length. Signed-off-by: Andi Shyti --- include/uapi/linux/lirc.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/lirc.h b/include/uapi/linux/lirc.h index 4b3ab29..801e5f8

[PATCH v2 10/15] [media] lirc_dev: remove CONFIG_COMPAT precompiler check

2016-07-01 Thread Andi Shyti
There is no need to check in precompilation whether the ioctl is compat or unlocked, depending on the configuration it will be called the correct one. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread Andi Shyti
The ir-spi is a simple device driver which supports the connection between an IR LED and the MOSI line of an SPI device. The driver, indeed, uses the SPI framework to stream the raw data provided by userspace through a character device. The chardev is handled by the LIRC framework and its

[PATCH v2 09/15] [media] lirc_dev: merge three if statements in only one

2016-07-01 Thread Andi Shyti
The three if statements check the same thing, merge them in only one statement. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH v2 07/15] [media] lirc_dev: simplify if statement in lirc_add_to_buf

2016-07-01 Thread Andi Shyti
The whole function is inside an 'if' statement ("if (ir->d.add_to_buf)"). Check the opposite of that statement at the beginning and exit, this way we can have one level less of indentation. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 31

[PATCH v2 08/15] [media] lirc_dev: remove double if ... else statement

2016-07-01 Thread Andi Shyti
There are two if ... else which check the same thing in different part of the code, they can be merged in a single check. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git

[PATCH] cec: use same build config mode as MEDIA_SUPPORT

2016-07-01 Thread Hans Verkuil
The cec framework should not be build as a module if MEDIA_SUPPORT is set to built-in. Otherwise drivers that are built-in would not be able to reference the exported cec functions. So do the same as is done for the media controller in drivers/media/Makefile: make MEDIA_CEC a bool and use the

[PATCH v2 12/15] [media] lirc_dev: fix error return value

2016-07-01 Thread Andi Shyti
If ioctl is called, it cannot be a case of invalid system call number (ENOSYS), that is an operation not permitted (EPERM). Replace ENOSYS with EPERM. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 8 1 file changed, 4 insertions(+), 4 deletions(-)

[PATCH v2 11/15] [media] lirc_dev: fix variable constant comparisons

2016-07-01 Thread Andi Shyti
When comparing a variable with a constant, the comparison should start from the variable and not from the constant. It's also written in the human DNA. Swap the terms of comparisons whenever the constant comes first and fix the following checkpatch warning: WARNING: Comparisons should place

Re: [PATCH] [media] v4l2-async: Always unregister the subdev on failure

2016-07-01 Thread Hans Verkuil
On 05/11/2016 06:32 PM, Alban Bedel wrote: > On Wed, 11 May 2016 12:22:44 -0400 > Javier Martinez Canillas wrote: > >> Hello Alban, >> >> On 05/11/2016 11:40 AM, Alban Bedel wrote: >>> In v4l2_async_test_notify() if the registered_async callback or the >>> complete

Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver

2016-07-01 Thread Hans Verkuil
On 07/01/2016 01:53 PM, andrew-ct chen wrote: > On Fri, 2016-07-01 at 12:11 +0200, Hans Verkuil wrote: >> On 06/16/2016 12:54 PM, Mauro Carvalho Chehab wrote: >>> Em Tue, 14 Jun 2016 19:08:08 +0800 >>> tiffany lin escreveu: >>> Hi Mauro, On Wed,

Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver

2016-07-01 Thread andrew-ct chen
On Fri, 2016-07-01 at 12:11 +0200, Hans Verkuil wrote: > On 06/16/2016 12:54 PM, Mauro Carvalho Chehab wrote: > > Em Tue, 14 Jun 2016 19:08:08 +0800 > > tiffany lin escreveu: > > > >> Hi Mauro, > >> > >> > >> On Wed, 2016-06-08 at 07:13 +0900, Hans Verkuil wrote: > >>>

Re: [PATCH v4] [media] vimc: Virtual Media Controller core, capture and sensor

2016-07-01 Thread Hans Verkuil
Hi Helen, Better late than never, but I finally have time for a review, mostly with a eye for V4L2 issues. Please note the 'kbuild test robot' mails you received: those issues should be resolved in v5. It looks like it misses the media controller dependency in the Kconfig. On 05/31/2016 05:02

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread Sean Young
On Fri, Jul 01, 2016 at 05:33:42PM +0900, Andi Shyti wrote: > The ir-spi is a simple device driver which supports the > connection between an IR LED and the MOSI line of an SPI device. > > The driver, indeed, uses the SPI framework to stream the raw data > provided by userspace through a

Re: [PATCH] cec: fix Kconfig dependency problems

2016-07-01 Thread Arnd Bergmann
On Friday, July 1, 2016 12:37:06 PM CEST Hans Verkuil wrote: > - Use IS_REACHABLE(RC_CORE) instead of IS_ENABLED: if cec is built-in and > RC_CORE is a module, then CEC can't reach the RC symbols. > - Both cec and cec-edid should be bool and use the same build 'mode' as > MEDIA_SUPPORT (just

Re: [patch V4 14/31] media: use parity functions in saa7115

2016-07-01 Thread Hans Verkuil
On 05/11/2016 11:19 AM, zengzhao...@163.com wrote: > From: Zhaoxiu Zeng > > Signed-off-by: Zhaoxiu Zeng Acked-by: Hans Verkuil Thanks, Hans > --- > drivers/media/i2c/saa7115.c | 17 ++--- > 1 file

Re: [PATCH v5 0/2] [media] atmel-isc: add driver for Atmel ISC

2016-07-01 Thread Hans Verkuil
Hi Songjun, First of all, please CC patch 2/2 to linux-media as well the next time you post this. I only see 1/2 on the mailinglist, and we need both. Secondly, before I can accept it you need to run the v4l2-compliance test first and I need to see the output of that test. The compliance test

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread Andi Shyti
Hi Sean, > > The ir-spi is a simple device driver which supports the > > connection between an IR LED and the MOSI line of an SPI device. > > > > The driver, indeed, uses the SPI framework to stream the raw data > > provided by userspace through a character device. The chardev is > > handled by

[GIT PULL] mem2mem changes

2016-07-01 Thread Kamil Debski
The following changes since commit 0db5c79989de2c68d5abb7ba891bfdb3cd3b7e05: [media] media-devnode.h: Fix documentation (2016-06-16 08:14:56 -0300) are available in the git repository at: git://linuxtv.org/kdebski/media_tree_2.git master-20160627 for you to fetch changes up to

[PATCH] cec: fix Kconfig dependency problems

2016-07-01 Thread Hans Verkuil
- Use IS_REACHABLE(RC_CORE) instead of IS_ENABLED: if cec is built-in and RC_CORE is a module, then CEC can't reach the RC symbols. - Both cec and cec-edid should be bool and use the same build 'mode' as MEDIA_SUPPORT (just as is done for the media controller code). - Add a note to staging

[PATCHv5 05/13] media/pci: convert drivers to use the new vb2_queue dev field

2016-07-01 Thread Hans Verkuil
>From 62ddd1aabe5672541055bc6de3c80ca1e3635729 Mon Sep 17 00:00:00 2001 From: Hans Verkuil Date: Mon, 15 Feb 2016 15:37:15 +0100 Subject: [PATCH 05/13] media/pci: convert drivers to use the new vb2_queue dev field Stop using alloc_ctx and just fill in the device pointer.

Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver

2016-07-01 Thread Hans Verkuil
On 06/16/2016 12:54 PM, Mauro Carvalho Chehab wrote: > Em Tue, 14 Jun 2016 19:08:08 +0800 > tiffany lin escreveu: > >> Hi Mauro, >> >> >> On Wed, 2016-06-08 at 07:13 +0900, Hans Verkuil wrote: >>> >>> On 06/07/2016 11:22 PM, Mauro Carvalho Chehab wrote: Em Mon,

Re: [PATCH] v4l: ioctl: Clear the v4l2_pix_format_mplane reserved field

2016-07-01 Thread Hans Verkuil
On 06/28/2016 03:20 PM, Laurent Pinchart wrote: > The S_FMT and TRY_FMT handlers in multiplane mode attempt at clearing > the reserved fields of the v4l2_format structure after the pix_mp > member. However, the reserved fields are inside pix_mp, not after it. > > Signed-off-by: Laurent Pinchart

Re: square-only image on Nokia N900 camera -- pipeline setup in python (was Re: [RFC PATCH 00/24] Make Nokia N900 cameras working)

2016-07-01 Thread Pavel Machek
On Fri 2016-07-01 10:50:25, Pavel Machek wrote: > Hi! > > > On gitlab is the latest version of pipeline setup if python. I also > > got fcam to work (slowly) on the camera, with autofocus and > > autogain. Capturing from preview modes works fine, but image quality > > is not good, as expected.

Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver

2016-07-01 Thread 李務誠
+Andrew. Last I heard the license was ready and Andrew was preparing the VPU firmware. Andrew. Is the firmware ready to submit? On Fri, Jul 1, 2016 at 6:11 PM, Hans Verkuil wrote: > On 06/16/2016 12:54 PM, Mauro Carvalho Chehab wrote: >> Em Tue, 14 Jun 2016 19:08:08 +0800 >>

Re: [PATCH 2/2] tw686x: Support VIDIOC_{S,G}_PARM ioctls

2016-07-01 Thread Hans Verkuil
On 06/29/2016 04:17 AM, Ezequiel Garcia wrote: > Now that the frame rate can be properly set, this commit adds support > for S_PARM and G_PARM. As mentioned in our irc discussion you also need to add enum_framesize/intervals. Otherwise applications won't know what values to use with s_parm. >

Re: usbvision: problems adding support for ATI TV Wonder USB Edition

2016-07-01 Thread Hans Verkuil
Hi Christopher, On 12/19/2015 07:12 AM, Christopher Chavez wrote: > Hi, > > Not yet an experienced developer here, still working on BSCmpE. > > I have attempted to add long-awaited support for the ATI TV Wonder USB Edition > (NTSC) to usbvision. I have patches of what I've accomplished so far,

[PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Arnd Bergmann
The linux/cec.h header file contains empty inline definitions for a subset of the API for the case in which CEC is not enabled, however we have driver that call other functions as well: drivers/media/i2c/adv7604.c: In function 'adv76xx_cec_tx_raw_status': drivers/media/i2c/adv7604.c:1956:3:

Re: [PATCH 00/38] i.MX5/6 Video Capture

2016-07-01 Thread Hans Verkuil
On 06/28/2016 10:10 PM, Hans Verkuil wrote: > On 06/28/2016 08:54 PM, Tim Harvey wrote: >> On Tue, Jun 14, 2016 at 3:48 PM, Steve Longerbeam >> wrote: >>> Tested on imx6q SabreAuto with ADV7180, and imx6q SabreSD with >>> mipi-csi2 OV5640. There is device-tree support also

Re: [PATCH 28/38] v4l: Add signal lock status to source change events

2016-07-01 Thread Hans Verkuil
On 06/15/2016 12:49 AM, Steve Longerbeam wrote: > Add a signal lock status change to the source changes bitmask. > This indicates there was a signal lock or unlock event detected > at the input of a video decoder. > > Signed-off-by: Steve Longerbeam > --- >

square-only image on Nokia N900 camera -- pipeline setup in python (was Re: [RFC PATCH 00/24] Make Nokia N900 cameras working)

2016-07-01 Thread Pavel Machek
Hi! On gitlab is the latest version of pipeline setup if python. I also got fcam to work (slowly) on the camera, with autofocus and autogain. Capturing from preview modes works fine, but image quality is not good, as expected. Capturing raw GRBG10 images works, but images are square, with values

Re: [RFC] [PATCH 0/3] media: an attempt to refresh omap1_camera driver

2016-07-01 Thread Hans Verkuil
On 06/17/2016 09:04 AM, Hans Verkuil wrote: > Hi Janusz, > > On 06/16/2016 07:21 PM, Janusz Krzysztofik wrote: >> As requested by media subsystem maintainers, here is an attempt to >> convert the omap1_camera driver to the vb2 framework. Also, conversion >> to the dmaengine framework, long

Re: [PATCH v2] Add tw5864 driver

2016-07-01 Thread Hans Verkuil
On 06/10/2016 12:11 AM, Andrey Utkin wrote: > Fixed most of issues discovered by v1 and RFCv0 reviewers. Refactored a lot. > > The only thing from previous review I haven't got my head around yet is > framerate control - Hans Verkuil has told to support 1001/3 frame > interval, > but it's

[PATCH 4/4] cxd2841er: adjust the dB scale for DVB-C

2016-07-01 Thread Mauro Carvalho Chehab
Instead of using a relative frequency range, calibrate it to show the results in dB. The callibration was done getting samples with a signal generated from -50dBm to -12dBm, incremented in steps of 0.5 dB, using 3 frequencies: 175 MHz, 410 MHz and 800 MHz. The modulated signal was using QAM64, and

[PATCH 2/4] cxd2841er: provide signal strength for DVB-C

2016-07-01 Thread Mauro Carvalho Chehab
Currently, there's no stats for DVB-C. Let's at least return signal strength. The scale is different than on DVB-T, so let's use a relative scale, for now. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/cxd2841er.c | 6 ++ 1 file changed, 6

[PATCH 3/4] cxd2841er: fix BER report via DVBv5 stats API

2016-07-01 Thread Mauro Carvalho Chehab
What userspace expects is to receive both bit_error and bit_count counters. So, instead of doing the division at the Kernel, return the counters for userspace to handle it the way it wants. Signed-off-by: Mauro Carvalho Chehab ---

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread Sean Young
On Fri, Jul 01, 2016 at 09:30:35PM +0900, Andi Shyti wrote: > Hi Sean, > > > > The ir-spi is a simple device driver which supports the > > > connection between an IR LED and the MOSI line of an SPI device. > > > > > > The driver, indeed, uses the SPI framework to stream the raw data > > >

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread Andi Shyti
Hi Sean, > > > Also I don't see what justifies this new interface. This can be > > > implemented in rc-core in less lines of code and it will be entirely > > > compatible with existing user-space. > > > > Also here I'm getting a bit confused. When I started writing > > this, I didn't even know

Re: [PATCH] cec: fix Kconfig dependency problems

2016-07-01 Thread Arnd Bergmann
On Friday, July 1, 2016 12:37:06 PM CEST Hans Verkuil wrote: > - Use IS_REACHABLE(RC_CORE) instead of IS_ENABLED: if cec is built-in and > RC_CORE is a module, then CEC can't reach the RC symbols. > - Both cec and cec-edid should be bool and use the same build 'mode' as > MEDIA_SUPPORT (just

[PATCH 1/4] cxd2841er: don't expose a dvbv5 stats to userspace if not available

2016-07-01 Thread Mauro Carvalho Chehab
The current code will expose a zero value if one of the stats is not available, but this is not what userspace expects. Instead, if something goes wrong on providing some stats, it should be changing the scale to FE_SCALE_NOT_AVAILABLE. So, change the logic to do the right thing. Signed-off-by:

Re: [PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Hans Verkuil
On 07/01/2016 01:19 PM, Arnd Bergmann wrote: > The linux/cec.h header file contains empty inline definitions for > a subset of the API for the case in which CEC is not enabled, > however we have driver that call other functions as well: > > drivers/media/i2c/adv7604.c: In function

[GIT PULL FOR v4.8] Various fixes/improvements

2016-07-01 Thread Hans Verkuil
Hi Mauro, Here is my yield of one day of patch processing. On Monday I plan to do another round, then for patches that are not directly my core activity. Although there are one or two dvb patches in this pull request as well. Regards, Hans The following changes since commit

[PATCH] tw686x: make const structs static

2016-07-01 Thread Hans Verkuil
Fix sparse warnings: tw686x-video.c:148:29: warning: symbol 'memcpy_dma_ops' was not declared. Should it be static? tw686x-video.c:195:29: warning: symbol 'contig_dma_ops' was not declared. Should it be static? tw686x-video.c:361:29: warning: symbol 'sg_dma_ops' was not declared. Should it be

[GIT PULL FOR v4.8] vb2: replace allocation context by device pointer

2016-07-01 Thread Hans Verkuil
This pull request is a rebased version of the patch v5 series: http://www.mail-archive.com/linux-media@vger.kernel.org/msg98729.html together with the tw686x fix (needed due to the rebase): https://patchwork.linuxtv.org/patch/34965/ (note: the subject and commit log as posted on the list is a

Re: A potential race

2016-07-01 Thread Hans Verkuil
On 07/01/2016 04:39 PM, Pavel Andrianov wrote: > Hi! > > There is a potential race condition between usbvision_v4l2_close > > and usbvision_disconnect >

Re: [PATCHv5 05/13] media/pci: convert drivers to use the new vb2_queue dev field

2016-07-01 Thread Ezequiel Garcia
On 01 Jul 11:55 AM, Hans Verkuil wrote: > From 62ddd1aabe5672541055bc6de3c80ca1e3635729 Mon Sep 17 00:00:00 2001 > From: Hans Verkuil > Date: Mon, 15 Feb 2016 15:37:15 +0100 > Subject: [PATCH 05/13] media/pci: convert drivers to use the new vb2_queue dev > field > > Stop

Re: [PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Arnd Bergmann
On Friday, July 1, 2016 4:35:09 PM CEST Hans Verkuil wrote: > On 07/01/2016 01:19 PM, Arnd Bergmann wrote: > > The linux/cec.h header file contains empty inline definitions for > > a subset of the API for the case in which CEC is not enabled, > > however we have driver that call other functions as

Re: A potential race

2016-07-01 Thread Pavel Andrianov
01.07.2016 19:53, Hans Verkuil пишет: On 07/01/2016 04:39 PM, Pavel Andrianov wrote: Hi! There is a potential race condition between usbvision_v4l2_close and usbvision_disconnect. The possible scenario may be the following. usbvision_disconnect starts execution, assigns

Re: [PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Arnd Bergmann
On Friday, July 1, 2016 5:22:32 PM CEST Hans Verkuil wrote: > > diff --git a/drivers/media/platform/vivid/Kconfig > > b/drivers/media/platform/vivid/Kconfig > > index 8e6918c5c87c..8e31146d079a 100644 > > --- a/drivers/media/platform/vivid/Kconfig > > +++ b/drivers/media/platform/vivid/Kconfig >

Re: [PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Hans Verkuil
On 07/01/2016 05:37 PM, Arnd Bergmann wrote: > On Friday, July 1, 2016 5:22:32 PM CEST Hans Verkuil wrote: >>> diff --git a/drivers/media/platform/vivid/Kconfig >>> b/drivers/media/platform/vivid/Kconfig >>> index 8e6918c5c87c..8e31146d079a 100644 >>> --- a/drivers/media/platform/vivid/Kconfig

Re: [PATCH] [media] cec: add missing inline stubs

2016-07-01 Thread Hans Verkuil
On 07/01/2016 05:13 PM, Arnd Bergmann wrote: > On Friday, July 1, 2016 4:35:09 PM CEST Hans Verkuil wrote: >> On 07/01/2016 01:19 PM, Arnd Bergmann wrote: >>> The linux/cec.h header file contains empty inline definitions for >>> a subset of the API for the case in which CEC is not enabled, >>>

Re: A potential race

2016-07-01 Thread Hans Verkuil
On 07/01/2016 05:02 PM, Pavel Andrianov wrote: > 01.07.2016 19:53, Hans Verkuil пишет: >> On 07/01/2016 04:39 PM, Pavel Andrianov wrote: >>> Hi! >>> >>> There is a potential race condition between usbvision_v4l2_close and >>> usbvision_disconnect. The possible scenario may be the following.

Re: square-only image on Nokia N900 camera -- pipeline setup in python (was Re: [RFC PATCH 00/24] Make Nokia N900 cameras working)

2016-07-01 Thread Pavel Machek
On Fri 2016-07-01 13:01:54, Pavel Machek wrote: > On Fri 2016-07-01 10:50:25, Pavel Machek wrote: > > Hi! > > > > > On gitlab is the latest version of pipeline setup if python. I also > > > got fcam to work (slowly) on the camera, with autofocus and > > > autogain. Capturing from preview modes

[PATCH] cxd2841er: fix signal strength scale for ISDB-T

2016-07-01 Thread Mauro Carvalho Chehab
The scale for ISDB-T was wrong too: it was inverted, and on a relative scale. Use a linear interpolation to make it look better. The formula was empirically determined, using 3 frequencies (175 MHz, 410 MHz and 800 MHz), measuring from -50dBm to -12dBm in steps of 0.5dB. Signed-off-by: Mauro

Re: [PATCH v6 10/11] Input: synaptics-rmi4 - add support for F54 diagnostics

2016-07-01 Thread Andrew Duggan
On 06/30/2016 10:38 AM, Nick Dyer wrote: Function 54 implements access to various RMI4 diagnostic features. This patch adds support for retrieving this data. It registers a V4L2 device to output the data to user space. Signed-off-by: Nick Dyer Tested-by: Andrew Duggan

cron job: media_tree daily build: ERRORS

2016-07-01 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 Jul 2 04:00:27 CEST 2016 git branch: test git hash: d81295d1bed850335f9f4ccb6b1aa4f6a123d4f0 gcc

Re: [PATCH 00/38] i.MX5/6 Video Capture

2016-07-01 Thread Steve Longerbeam
Hi Tim, Pardon late reply, I'm on vacation. See below... On 06/28/2016 11:54 AM, Tim Harvey wrote: On Tue, Jun 14, 2016 at 3:48 PM, Steve Longerbeam wrote: Tested on imx6q SabreAuto with ADV7180, and imx6q SabreSD with mipi-csi2 OV5640. There is device-tree support

Re: [PATCH 28/38] v4l: Add signal lock status to source change events

2016-07-01 Thread Steve Longerbeam
On 07/01/2016 12:24 AM, Hans Verkuil wrote: On 06/15/2016 12:49 AM, Steve Longerbeam wrote: Add a signal lock status change to the source changes bitmask. This indicates there was a signal lock or unlock event detected at the input of a video decoder. Signed-off-by: Steve Longerbeam

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread kbuild test robot
Hi, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.7-rc5 next-20160701] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Andi-Shyti/rc-ir-spi-add-support-for-IR-LEDs

Re: [PATCH] [media] rc: ir-spi: add support for IR LEDs connected with SPI

2016-07-01 Thread kbuild test robot
Hi, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.7-rc5 next-20160701] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Andi-Shyti/rc-ir-spi-add-support-for-IR