Re: [PATCH 3/3] [media] uvcvideo: skip non-extension unit controls on Oculus Rift Sensors

2017-07-23 Thread Philipp Zabel
Hi Laurent, Am Montag, den 17.07.2017, 05:25 +0300 schrieb Laurent Pinchart: > Hi Philipp, > > On Saturday 15 Jul 2017 15:13:45 Philipp Zabel wrote: > > Am Samstag, den 15.07.2017, 12:54 +0300 schrieb Laurent Pinchart: > > > On Friday 14 Jul 2017 22:14:24 Philipp Zabel wrote: > > > > The Oculus R

Re: [ragnatech:media-tree 2075/2144] drivers/media/dvb-frontends/stv0910.c:1185:2-8: preceding lock on line 1176 (fwd)

2017-07-23 Thread Daniel Scheller
Hello Jula, Am 24.07.2017 um 07:02 schrieb Julia Lawall: Is a lock release needed before line 1185? Is the !enable in line 1189 correct? If the code is correct as is, perhaps it could be good to add some comments. Yes, it's needed because the I2C gate is shared for the two tuner chips atta

Re: [Patch v5 11/12] [media] s5p-mfc: Add support for HEVC encoder

2017-07-23 Thread Smitha T Murthy
On Thu, 2017-07-20 at 15:32 +0200, Hans Verkuil wrote: > On 19/06/17 07:10, Smitha T Murthy wrote: > > Add HEVC encoder support and necessary registers, V4L2 CIDs, > > and hevc encoder parameters > > > > Signed-off-by: Smitha T Murthy > > --- > > drivers/media/platform/s5p-mfc/regs-mfc-v10.h |

[ragnatech:media-tree 2075/2144] drivers/media/dvb-frontends/stv0910.c:1185:2-8: preceding lock on line 1176 (fwd)

2017-07-23 Thread Julia Lawall
Is a lock release needed before line 1185? Is the !enable in line 1189 correct? If the code is correct as is, perhaps it could be good to add some comments. julia -- Forwarded message -- Date: Mon, 24 Jul 2017 12:55:30 +0800 From: kbuild test robot To: kbu...@01.org Cc: Julia L

Re: [Patch v5 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2017-07-23 Thread Smitha T Murthy
On Thu, 2017-07-20 at 15:13 +0200, Hans Verkuil wrote: > On 19/06/17 07:10, Smitha T Murthy wrote: > > Add v4l2 controls for HEVC encoder > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > drivers/media/v4l2-core/v4l2-ctrls.c | 103 > > +++

Re: [Patch v5 05/12] [media] videodev2.h: Add v4l2 definition for HEVC

2017-07-23 Thread Smitha T Murthy
On Thu, 2017-07-20 at 15:07 +0200, Hans Verkuil wrote: > On 19/06/17 07:10, Smitha T Murthy wrote: > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > include/uapi/linux/videodev2.h | 1 + > > 1 file changed, 1 ins

Re: [Patch v5 06/12] [media] v4l2-ioctl: add HEVC format description

2017-07-23 Thread Smitha T Murthy
On Thu, 2017-07-20 at 15:07 +0200, Hans Verkuil wrote: > On 19/06/17 07:10, Smitha T Murthy wrote: > > HEVC is a video coding format > > > > Signed-off-by: Smitha T Murthy > > --- > > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/med

cron job: media_tree daily build: ERRORS

2017-07-23 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 Jul 24 05:00:15 CEST 2017 media-tree git hash:0e50e84a11f4854e9a7e3b7f4443ffb99e6be292 media_build gi

[PATCH 2/3][resend] media: ti-vpe: cal: use of_graph_get_remote_endpoint()

2017-07-23 Thread Kuninori Morimoto
From: Kuninori Morimoto Now, we can use of_graph_get_remote_endpoint(). Let's use it. Signed-off-by: Kuninori Morimoto Reviewed-by: Sylwester Nawrocki Acked-by: Benoit Parrot --- based on 4c9c3d595f1bad021cc126d20879df4016801736 ("of_graph: add of_graph_get_remote_endpoint()") drivers/medi

[PATCH 2/3][resend] media: ti-vpe: cal: use of_graph_get_remote_endpoint()

2017-07-23 Thread Kuninori Morimoto
From: Kuninori Morimoto Now, we can use of_graph_get_remote_endpoint(). Let's use it. Signed-off-by: Kuninori Morimoto Reviewed-by: Sylwester Nawrocki Acked-by: Benoit Parrot --- based on 4c9c3d595f1bad021cc126d20879df4016801736 ("of_graph: add of_graph_get_remote_endpoint()") drivers/medi

Quick Loans

2017-07-23 Thread Sec Capital Loan
Loan Offer at 3%, Feel Free to REPLY back to us for more info.

Re: [PATCH] dvb_frontend: initialize variable s with FE_NONE instead of 0

2017-07-23 Thread kbuild test robot
Hi Colin, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.13-rc1 next-20170721] [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/Colin-King/dvb_frontend-initialize-var

Fw: [PATCH RESEND 00/14] ddbridge: bump to ddbridge-0.9.29

2017-07-23 Thread Daniel Scheller
Manfred kindly asked me to forward his message to the list as he received a bounce from vger.kernel.org on this. Beginn der weitergeleiteten Nachricht: Datum: Sun, 23 Jul 2017 22:10:02 +0200 Von: Manfred_Knick An: Daniel Scheller , linux-media@vger.kernel.org, mche...@kernel.org, mche...@s-ope

?

2017-07-23 Thread Robert
> Did you receive my previous mail ? When and what time can i call you?

[PATCH RESEND 06/14] [media] ddbridge: split off hardware definitions and mappings

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Further cleanup of ddbridge-core and ddbridge-main, and moves all such hw definitions into one single place, making things easier to maintain. Signed-off-by: Daniel Scheller Tested-by: Richard Scobie Tested-by: Jasmin Jessich Tested-by: Dietmar Spingler Tested-by: Manfr

[PATCH RESEND 07/14] [media] ddbridge: check pointers before dereferencing

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Fixes two warnings reported by smatch: drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable dereferenced before check 'idev' (see line 238) drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable dereferenced before check '

[PATCH RESEND 08/14] [media] ddbridge: only register frontends in fe2 if fe is not NULL

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Smatch reported: drivers/media/pci/ddbridge/ddbridge-core.c:1602 dvb_input_attach() error: we previously assumed 'dvb->fe' could be null (see line 1595) dvb->fe2 will ever only be populated when dvb->fe is set. So only handle registration of dvb->fe2 when dvb->fe got se

[PATCH RESEND 09/14] [media] ddbridge: fix possible buffer overflow in ddb_ports_init()

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Report from smatch: drivers/media/pci/ddbridge/ddbridge-core.c:2659 ddb_ports_init() error: buffer overflow 'dev->port' 32 <= u32max Fix by making sure "p" is greater than zero before checking for "dev->port[].type == DDB_CI_EXTERNAL_XO2". Cc: Ralph Metzler Signed-off

[PATCH RESEND 11/14] [media] ddbridge: fix impossible condition warning

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Smatch and gcc complained: drivers/media/pci/ddbridge/ddbridge-core.c:3491 bpsnr_show() warn: impossible condition '(snr[0] == 255) => ((-128)-127 == 255)' drivers/media/pci/ddbridge/ddbridge-core.c: In function ‘bpsnr_show’: drivers/media/pci/ddbridge/ddbridge-core

[PATCH RESEND 10/14] [media] ddbridge: remove unreachable code

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller >From smatch: drivers/media/pci/ddbridge/ddbridge-core.c:3490 snr_store() info: ignoring unreachable code. In fact, the function immediately returns zero, so remove it and update ddb_attrs_snr[] to not reference it anymore. Cc: Ralph Metzler Signed-off-by: Daniel Sche

[PATCH RESEND 13/14] [media] ddbridge: Kconfig option to control the MSI modparam default

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller It is known that MSI interrupts - while working quite well so far - can still cause issues on some hardware platforms (causing I2C timeouts due to unhandled interrupts). The msi variable/option is set to 1 by default. So, add a Kconfig option prefixed with "EXPERIMENTAL" tha

[PATCH RESEND 05/14] [media] ddbridge: split off IRQ handling

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller This not only helps keep the ddbridge-core tidy, but also gets rid of defined-but-unused-function warnings which might be triggered depending of CONFIG_PCI_MSI, without having to clutter the code with #ifdef'ery. Signed-off-by: Daniel Scheller Tested-by: Richard Scobie Te

[PATCH RESEND 01/14] [media] ddbridge: move/reorder functions

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller The functions in ddbridge-core.c have been moved to different positions in newer versions of the dddvb vendor driver package (most notably in version 0.9.9b). Perform the same code move to keep the diff of the upcoming code bump simpler. Signed-off-by: Daniel Scheller Test

[PATCH RESEND 02/14] [media] ddbridge: split code into multiple files

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller As of 0.9.9b, the ddbridge code has been split from one single file (ddbridge-core.c) into multiple files, with the purpose of taking care of different topics, and to be able to reuse code in different kernel modules (ddbridge.ko and octonet.ko). This applies the same code s

[PATCH RESEND 12/14] [media] ddbridge: fix dereference before check

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Both ts_release() and ts_open() can use "output" before check (smatch): drivers/media/pci/ddbridge/ddbridge-core.c:816 ts_release() warn: variable dereferenced before check 'output' (see line 809) drivers/media/pci/ddbridge/ddbridge-core.c:836 ts_open() warn: variable

[PATCH RESEND 14/14] [media] MAINTAINERS: add entry for ddbridge

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9826a918d37a..f25f26b5d9f6 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8428,6 +8428,14 @@ T: git git://linuxtv.org/media_tree

[PATCH RESEND 04/14] [media] ddbridge: split I/O related functions off from ddbridge.h

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller While it seems valid that headers can carry simple oneline static inline annotated functions, move them into their own header file to have the overall code more readable. Also, keep them as header (and don't put in a separate object) and static inline to help the compiler av

[PATCH RESEND 00/14] ddbridge: bump to ddbridge-0.9.29

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Preferrably for Linux 4.14 (to get things done). Resend reasons (resend since no real changes went in): * rebased on latest mediatree-master wrt commit 618e8aac3d7c ("media: ddbridge: constify i2c_algorithm structure") * build error in ddbridge-core.c fixed wrt comm

[PATCH] staging: imx: fix non-static declarations

2017-07-23 Thread JB Van Puyvelde
Add static keywords to fix this kind of sparse warnings: warning: symbol 'imx_t_vcm_timing' was not declared. Should it be static? Signed-off-by: JB Van Puyvelde --- drivers/staging/media/atomisp/i2c/imx/imx.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff

Re: [PATCH 0/7] stv0910/stv6111 updates

2017-07-23 Thread Jasmin J.
Hello! I tested this patch series with: DD Cine S2 V6 DD Duoflex CI (single) DD Duoflex S2 V4 (stv0910/stv6111) VDR 2.3.8, ddci2 Plugin 1.5.0 The SAT cables were connected to the DD DuoFlex S2 V4 card and VDR was instructed to use this device only. I checked the signal values with VDR and

[PATCH] [media] dvb-frontends/cxd2841er: update moddesc wrt new chip support

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Since the driver now recognizes and supports more chip variants, reflect this fact in the module description accordingly. Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/cxd2841er.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

Re: [PATCH 00/14] ddbridge: bump to ddbridge-0.9.29

2017-07-23 Thread Daniel Scheller
Am Sun, 9 Jul 2017 21:42:07 +0200 schrieb Daniel Scheller : > From: Daniel Scheller > > Preferrably for Linux 4.14 (to get things done). > > [...] > > Mauro/Media maintainers, this updates drivers/media/pci/ddbridge to > the very latest code that DD carry in their vendor driver package as > o

[PATCH 1/3] build: Add compat code for pm_runtime_get_if_in_use

2017-07-23 Thread Jasmin J.
From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- v4l/compat.h | 15 +++ v4l/scripts/make_config_compat.pl | 1 + 2 files changed, 16 insertions(+) diff --git a/v4l/compat.h b/v4l/compat.h index e565292..b5b0846 100644 --- a/v4l/compat.h +++ b/v4l/compat.

[PATCH 2/3] build: Disable VIDEO_OV5670 for Kernels older that 3.17

2017-07-23 Thread Jasmin J.
From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- v4l/versions.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/v4l/versions.txt b/v4l/versions.txt index ae4a14f..1dd9a2b 100644 --- a/v4l/versions.txt +++ b/v4l/versions.txt @@ -68,6 +68,7 @@ VIDEO_ADV7180 VIDEO_ET8EK8 VIDEO_OV2640

[PATCH 0/3] fix compile for kernel 3.13

2017-07-23 Thread Jasmin J.
From: Jasmin Jessich This series fixed compilation errors for older kernels. I have tested it with Kernel 3.13 and Daniel with Kernel 4.12. I disabled VIDEO_OV5670 for all kernels older than 3.17, but I am not 100% sure if the required change to compile it was really implemented in that version

[PATCH 3/3] build: fix up build w/kernels <=4.12 by reverting 4.13 patches

2017-07-23 Thread Jasmin J.
From: Daniel Scheller Signed-off-by: Daniel Scheller Signed-off-by: Jasmin Jessich --- backports/backports.txt| 3 + .../v4.12_revert_solo6x10_copykerneluser.patch | 71 ++ 2 files changed, 74 insertions(+) create mode 100644 backports/v4.1

Re: [PATCH v3 2/4] i2c: add docs to clarify DMA handling

2017-07-23 Thread Jonathan Cameron
On Tue, 18 Jul 2017 12:23:37 +0200 Wolfram Sang wrote: > Signed-off-by: Wolfram Sang Is this material not perhaps better placed in the sphinx docs? Up to you of course as your subsystem ;) Text is good though. Acked-by: Jonathan Cameron > --- > Changes since v2: > > * documentation updates.

Re: [PATCH v3 1/4] i2c: add helpers to ease DMA handling

2017-07-23 Thread Jonathan Cameron
On Tue, 18 Jul 2017 12:23:36 +0200 Wolfram Sang wrote: > One helper checks if DMA is suitable and optionally creates a bounce > buffer, if not. The other function returns the bounce buffer and makes > sure the data is properly copied back to the message. > > Signed-off-by: Wolfram Sang My knowl

Re: [PATCH] [media] ddbridge: constify i2c_algorithm structure

2017-07-23 Thread Daniel Scheller
Am Mon, 10 Jul 2017 17:12:27 +0200 schrieb Daniel Scheller : > From: Daniel Scheller > > Original patch and issue identified by Gustavo A. R. Silva > via [1] using Coccinelle. While at it, even > mark the struct static again since it isn't referenced anywhere else. > > [1] http://www.spinics.n

[PATCH 3/7] [media] dvb-frontends/stv0910: further coding style cleanup

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Fixes up all remainders reported by "checkpatch.pl --strict" Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 93 +-- drivers/media/dvb-frontends/stv0910.h | 4 +- 2 files changed, 48 insertions(+), 49 deletions(-

[PATCH 6/7] [media] dvb-frontends/stv6111: cosmetics: comments fixup, misc

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv6111.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/media/dvb-frontends/stv6111.c b/drivers/media/dvb-frontends/stv6111.c index 91e24ba44c30..

[PATCH 5/7] [media] dvb-frontends/stv6111: coding style cleanup

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Fix up all remainders reported by checkpatch-strict. Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv6111.c | 38 --- drivers/media/dvb-frontends/stv6111.h | 7 --- 2 files changed, 26 insertions(+), 19 deletions(-)

[PATCH 7/7] [media] dvb-frontends/stv{0910,6111}: constify tables

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Mark lookup tables and fe_ops things const so the compiler can put them into .rodata. While at it, improve name and identifier strings (moddesc, fe_ops). Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 24 drivers/media

[PATCH 2/7] [media] dvb-frontends/stv0910: implement diseqc_send_burst

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller This implements the diseqc_send_burst frontend op to support sending mini-DISEQC bursts. Picked up from dddvb's driver package where this specific block was disabled via #if 0/#endif, but is still working according to feedback from upstream, so add it. Signed-off-by: Daniel

[PATCH 1/7] [media] dvb-frontends/stv0910: fix STR assignment, remove unneeded var

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller According to the documentation, FE_SCALE_DECIBEL values should be assigned to .svalue and not .uvalue, so let's do this. While at it, remove the unneeded strength var from read_signal_strength(). Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 5

[PATCH 0/7] stv0910/stv6111 updates

2017-07-23 Thread Daniel Scheller
From: Daniel Scheller Several, mostly cosmetic updates and fixes to these two new drivers, and one additional functionality: - Make both drivers 99% checkpatch-strict clean. stv0910_regs.h still has "CHECK: 'VALIDE' may be misspelled - perhaps 'VALID'?", which is left as-is intentionally (un

[PATCH] Add compat code for skb_put_data

2017-07-23 Thread Matthias Schwarzott
Signed-off-by: Matthias Schwarzott --- v4l/compat.h | 12 v4l/scripts/make_config_compat.pl | 1 + 2 files changed, 13 insertions(+) diff --git a/v4l/compat.h b/v4l/compat.h index 47e2694..e565292 100644 --- a/v4l/compat.h +++ b/v4l/compat.h @@ -2072,4 +2072,16

Re: [RFC 08/19] arm: dts: omap3: N9/N950: Add AS3645A camera flash

2017-07-23 Thread Pavel Machek
On Tue 2017-07-18 22:03:50, Sakari Ailus wrote: > From: Sakari Ailus > > Add the as3645a flash controller to the DT source as well as the flash > property with the as3645a device phandle to the sensor DT node. > > Signed-off-by: Sakari Ailus > Reviewed-by: Sebastian Reichel Acked-by: Pavel Ma