[PATCH] build: add fwnode_property_get_reference_args if not defined

2017-11-05 Thread Matthias Schwarzott
Add function dummy that returns -ENODATA. Copied struct fwnode_reference_args from include/linux/fwnode.h. Signed-off-by: Matthias Schwarzott --- v4l/compat.h | 19 +++ v4l/scripts/make_config_compat.pl | 1 + 2 files changed, 20

[PATCH 03/15] si2165: Make checkpatch happy

2017-11-05 Thread Matthias Schwarzott
Fix almost all of checkpatch --strict warnings. The remaining warnings are about: * macro REG16 (should be enclosed in parentheses) * macro REG16 (Macro argument reuse) Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 87

[PATCH 08/15] si2165: Use constellation from property cache instead of hardcoded QAM256

2017-11-05 Thread Matthias Schwarzott
Use constellation from property cache instead of always setting it to QAM256. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 29 ++--- 1 file changed, 26 insertions(+), 3 deletions(-) diff --git

[PATCH 04/15] si2165: define register macros

2017-11-05 Thread Matthias Schwarzott
Convert register numbers to macros. Correctness verified by comparing the disassembly before and after. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 206 +++--- drivers/media/dvb-frontends/si2165_priv.h | 65

[PATCH 05/15] si2165: move ts parallel mode setting to the ts init code

2017-11-05 Thread Matthias Schwarzott
The TS parallel mode setting should be where all other TS settings are written. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb-frontends/si2165.c

[PATCH 02/15] si2165: Convert debug printk to dev_dbg

2017-11-05 Thread Matthias Schwarzott
Removed module parameter debug and the conditions based on it. Now it can be configured via dynamic debug. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 79 +++- 1 file changed, 15 insertions(+), 64 deletions(-)

[PATCH 07/15] si2165: Write const value for lock timeout

2017-11-05 Thread Matthias Schwarzott
The lock timeout should not depend on the bandwidth. It should be either constant or depend on xtal frequency. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 01/15] si2165: Remove redundant KBUILD_MODNAME from dev_* logging

2017-11-05 Thread Matthias Schwarzott
Remove redundant repeated module name from messages. Before: si2165 8-0064: si2165: fw load finished After: si2165 8-0064: fw load finished Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 69 1 file

[PATCH 09/15] cx231xx: Use semicolon after assignment instead of comma

2017-11-05 Thread Matthias Schwarzott
End assignments by semicolon instead of comma. Signed-off-by: Matthias Schwarzott --- drivers/media/usb/cx231xx/cx231xx-dvb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/usb/cx231xx/cx231xx-dvb.c

[PATCH 15/15] media: MAINTAINERS: add si2165 driver

2017-11-05 Thread Matthias Schwarzott
Silicon Labs Si2165 DVB-C/T demod driver Signed-off-by: Matthias Schwarzott --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index adbf69306e9e..b3dc695bcfa4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -12193,6 +12193,14 @@

[PATCH 14/15] si2165: Add DVBv3 wrapper for ber statistics

2017-11-05 Thread Matthias Schwarzott
Add read_ber function that reads from property cache to support DVBv3. The implementation is inspired by the cx24120 driver. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 26 ++ 1 file changed, 26 insertions(+) diff --git

[PATCH 06/15] si2165: improve read_status

2017-11-05 Thread Matthias Schwarzott
Use check_signal register for DVB-T additionally. For DVB-C use ps_lock additionally. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 41 ++- drivers/media/dvb-frontends/si2165_priv.h | 2 ++ 2 files changed, 37

[PATCH 10/15] cx23885: Use semicolon after assignment instead of comma

2017-11-05 Thread Matthias Schwarzott
End assignments by semicolon instead of comma. Signed-off-by: Matthias Schwarzott --- drivers/media/pci/cx23885/cx23885-dvb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/pci/cx23885/cx23885-dvb.c

[PATCH 12/15] si2165: add DVBv5 BER statistics

2017-11-05 Thread Matthias Schwarzott
Add support for BER statistics. Configure a measurement period of 3 packets. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 57 +-- drivers/media/dvb-frontends/si2165_priv.h | 11 ++ 2 files changed, 66

[PATCH 13/15] si2165: add DVBv3 wrapper for C/N statistics

2017-11-05 Thread Matthias Schwarzott
Add read_snr function that reads from property cache to support DVBv3. Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/media/dvb-frontends/si2165.c

[PATCH 11/15] si2165: add DVBv5 C/N statistics for DVB-C

2017-11-05 Thread Matthias Schwarzott
Add C/N statistics in dB to read_status (DVBv5). Signed-off-by: Matthias Schwarzott --- drivers/media/dvb-frontends/si2165.c | 43 +-- drivers/media/dvb-frontends/si2165_priv.h | 1 + 2 files changed, 42 insertions(+), 2 deletions(-) diff

[PATCH] cec: add the adap_monitor_pin_enable op

2017-11-05 Thread Hans Verkuil
Some devices can monitor the CEC pin using an interrupt, but you only want to enable the interrupt if you actually switch to pin monitoring mode. So add a new op that is called when pin monitoring needs to be switched on or off. Also fix a small bug where the initial CEC pin event was sent again

Re: broken build against 4.4.0

2017-11-05 Thread Jasmin J.
Hello Vincent! > can someone take a look please? Well, Matthias may have fixed that (I didn't try). See: https://www.mail-archive.com/linux-media@vger.kernel.org/msg121610.html Maybe you need that also: https://www.mail-archive.com/linux-media@vger.kernel.org/msg121625.html BR, Jasmin

[PATCH 1/1] ov13858: Select V4L2_FWNODE

2017-11-05 Thread Sakari Ailus
The ov13858 driver depends on the V4L2 fwnode, thus add that to Kconfig. Signed-off-by: Sakari Ailus --- drivers/media/i2c/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig index

Re: [PATCH 1/1] media: i2c: as3645a: Remove driver

2017-11-05 Thread Sakari Ailus
On Thu, Nov 02, 2017 at 07:44:24PM +0200, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Friday, 8 September 2017 16:51:40 EET Sakari Ailus wrote: > > Remove the V4L2 AS3645A sub-device driver in favour of the LED flash class > > driver for the same hardware,

[PATCH 1/1] et8ek8: Select V4L2_FWNODE

2017-11-05 Thread Sakari Ailus
The et8ek8 driver depends on the V4L2 fwnode, thus add that to Kconfig. Signed-off-by: Sakari Ailus --- drivers/media/i2c/et8ek8/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/i2c/et8ek8/Kconfig b/drivers/media/i2c/et8ek8/Kconfig index

cron job: media_tree daily build: ERRORS

2017-11-05 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: Mon Nov 6 05:00:15 CET 2017 media-tree git hash:9917fbcfa20ab987d6381fd0365665e5c1402d75 media_build

Re: [PATCH v7 3/3] intel-ipu3: cio2: Add new MIPI-CSI2 driver

2017-11-05 Thread Sakari Ailus
Hi Yong, Thanks for the update! I took a final glance, there are a few matters that still need to be addressed but then I think we're done. Please see below. Then we'll need an entry in the MAINTAINERS file in the kernel root directory. Could you add an entry for this driver in v8? On Thu, Nov

Re: [PATCH v1] [media] v4l2-ctrls: Don't validate BITMASK twice

2017-11-05 Thread Sakari Ailus
On Fri, Nov 03, 2017 at 03:35:39PM +0200, Andy Shevchenko wrote: > There is no need to repeat what check_range() does for us, i.e. BITMASK > validation in v4l2_ctrl_new(). > > Signed-off-by: Andy Shevchenko Acked-by: Sakari Ailus

Re: [PATCH 1/2] media: usb: uvc: remove duplicate & operation

2017-11-05 Thread Jaejoong Kim
Hi, Laurent Could you please review this patch? Thanks, jaejoong 2017-10-20 16:25 GMT+09:00 Jaejoong Kim : > usb_endpoint_maxp() has an inline keyword and searches for bits[10:0] > by & operation with 0x7ff. So, we can remove the duplicate & operation > with 0x7ff. > >

[GIT PULL for 4.15] V4L2 async NULL dereference fix

2017-11-05 Thread Sakari Ailus
Hi Mauro, There was a rather subtle problem in the recent V4L2 async patches that could lead to NULL pointer dereference in some situations. This simple patch from Niklas addresses that. Please pull. The following changes since commit 9917fbcfa20ab987d6381fd0365665e5c1402d75: media: