Re: [PATCH 5/5] [media] cxusb: add analog mode support for Medion MD95700

2017-08-12 Thread kbuild test robot
Hi Maciej, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.13-rc4 next-20170811] [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/Maciej-S-Szmigiero/Add-analog-mod

cron job: media_tree daily build: ERRORS

2017-08-12 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: Sun Aug 13 05:00:20 CEST 2017 media-tree git hash:ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0 media_build gi

[linux-next:master 1210/6359] drivers/media//cec/cec-adap.c:167: error: unknown field 'lost_msgs' specified in initializer

2017-08-12 Thread kbuild test robot
Hi Hans, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 91dfed74eabcdae9378131546c446442c29bf769 commit: 6b2bbb08747a56dcf4ee33606a06025eca571260 [1210/6359] media: cec: rework the cec event handling config: x86_6

Re: [PATCH v7] rockchip/rga: v4l2 m2m support

2017-08-12 Thread kbuild test robot
Hi Jacob, [auto build test ERROR on rockchip/for-next] [also build test ERROR on v4.13-rc4 next-20170811] [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/Jacob-Chen/rockchip-rga-v4l2-m2m-support/

Re: [PATCH v7] rockchip/rga: v4l2 m2m support

2017-08-12 Thread kbuild test robot
Hi Jacob, [auto build test ERROR on rockchip/for-next] [also build test ERROR on v4.13-rc4 next-20170811] [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/Jacob-Chen/rockchip-rga-v4l2-m2m-support/

Re: [PATCH 5/5] [media] cxusb: add analog mode support for Medion MD95700

2017-08-12 Thread kbuild test robot
Hi Maciej, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.13-rc4 next-20170811] [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/Maciej-S-Szmigiero/Add-analog-mod

Re: [linux-media] Patch notification: 3 patches updated

2017-08-12 Thread Peter Rosin
On 2017-08-09 17:01, Patchwork wrote: > Hello, > > The following patches (submitted by you) have been updated in patchwork: > > * linux-media: [2/3,media] cx231xx: drop return value of > cx231xx_i2c_unregister > - http://patchwork.linuxtv.org/patch/42858/ > - for: Linux Media kernel p

Re: [PATCH 5/5] [media] cxusb: add analog mode support for Medion MD95700

2017-08-12 Thread kbuild test robot
Hi Maciej, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.13-rc4 next-20170811] [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/Maciej-S-Szmigiero/Add-analog-mod

[PATCH] au0828: fix unbalanced lock/unlock in au0828_usb_probe

2017-08-12 Thread Gustavo A. R. Silva
Call mutex_unlock and free dev on failure. Reported-by: Julia Lawall Signed-off-by: Gustavo A. R. Silva --- drivers/media/usb/au0828/au0828-core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/media/usb/au0828/au0828-core.c index 739df61

[PATCH] [media] tuners: make tda18271_std_map const

2017-08-12 Thread Bhumika Goyal
Make these const as they are only used during a copy operation. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/tuners/tda18271-maps.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/tuners/tda18271-maps.c b/drivers/media/tuners/tda182

[PATCH v4 09/11] [media] ddbridge: fix dereference before check

2017-08-12 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 v4 10/11] [media] ddbridge: Kconfig option to control the MSI modparam default

2017-08-12 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 v4 11/11] [media] MAINTAINERS: add entry for ddbridge

2017-08-12 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 931abca006b7..0453a1365c3a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8421,6 +8421,14 @@ T: git git://linuxtv.org/media_tree

[PATCH v4 07/11] [media] ddbridge: remove unreachable code

2017-08-12 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 v4 06/11] [media] ddbridge: fix possible buffer overflow in ddb_ports_init()

2017-08-12 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 v4 08/11] [media] ddbridge: fix impossible condition warning

2017-08-12 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 v4 04/11] [media] ddbridge: check pointers before dereferencing

2017-08-12 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 v4 02/11] [media] ddbridge: split I/O related functions off from ddbridge.h

2017-08-12 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 v4 05/11] [media] ddbridge: only register frontends in fe2 if fe is not NULL

2017-08-12 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 v4 03/11] [media] ddbridge: split off hardware definitions and mappings

2017-08-12 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 v4 00/11] ddbridge: bump to ddbridge-0.9.29

2017-08-12 Thread Daniel Scheller
From: Daniel Scheller Still, target: 4.14. Changes from v3 to v4: * IRQ handler splitoff removed * Prefixed irq_handler{,0,1} with ddb_ to avoid potential conflicts in the global namespace wrt things like allyesconfig due to very generic function naming * Removed MSI_IRQHANDLERS #ifdef'ery

Re: [PATCH 4/4] drm: adv7511/33: add HDMI CEC support

2017-08-12 Thread Hans Verkuil
On 12/08/17 11:53, Hans Verkuil wrote: > On 10/08/17 10:49, Archit Taneja wrote: >> >> >> On 07/30/2017 06:37 PM, Hans Verkuil wrote: >>> From: Hans Verkuil >>> >>> Add support for HDMI CEC to the drm adv7511/adv7533 drivers. >>> >>> The CEC registers that we need to use are identical for both dri

[PATCH v3] media: v4l2-ctrls.h: better document the arguments for v4l2_ctrl_fill

2017-08-12 Thread Mauro Carvalho Chehab
The arguments for this function are pointers. Make it clear at its documentation. Signed-off-by: Mauro Carvalho Chehab --- Hans, Feel free to pick this patch on your tree, if you're ok with it. Your approach for using v4l_queryctl() sounds better, as it covers private controls too. So I'm not

Re: [PATCH 4/4] drm: adv7511/33: add HDMI CEC support

2017-08-12 Thread Hans Verkuil
On 10/08/17 10:49, Archit Taneja wrote: > > > On 07/30/2017 06:37 PM, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Add support for HDMI CEC to the drm adv7511/adv7533 drivers. >> >> The CEC registers that we need to use are identical for both drivers, >> but they appear at different offsets i

Re: [PATCH] v4l2-compat-ioctl32.c: make ctrl_is_pointer generic

2017-08-12 Thread Mauro Carvalho Chehab
Em Sat, 12 Aug 2017 10:22:07 +0200 Hans Verkuil escreveu: > On 11/08/17 23:08, Mauro Carvalho Chehab wrote: > > Em Fri, 11 Aug 2017 15:26:03 +0200 > > Hans Verkuil escreveu: > > > >> The ctrl_is_pointer used a hard-coded list of control IDs that besides > >> being > >> outdated also wouldn't

Re: [PATCHv2 0/9] omapdrm: hdmi4: add CEC support

2017-08-12 Thread Hans Verkuil
On 11/08/17 12:57, Tomi Valkeinen wrote: > Hi Hans, > > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > > On 02/08/17 11:53, Hans Verkuil wrote: >> From: Hans Verkuil >> >> This patch series adds CEC support f

Re: [GIT PULL FOR v4.14] RC changes (part #2)

2017-08-12 Thread Hans Verkuil
On 11/08/17 15:59, Sean Young wrote: > Hi Mauro, > > Please apply for-v4.14a before this pull request. This pull request includes > a cec fix from Hans, to prevent merge conflicts. > > The last patch is fairly large, and its purpose is get rid of the ugly > RC_TYPE_ and RC_BIT_ names. I realise t

[PATCHv2 1/3] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-08-12 Thread Hans Verkuil
From: Hans Verkuil This adds support for the DisplayPort CEC-Tunneling-over-AUX feature that is part of the DisplayPort 1.3 standard. Unfortunately, not all DisplayPort/USB-C to HDMI adapters with a chip that has this capability actually hook up the CEC pin, so even though a CEC device is create

[PATCHv2 3/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-08-12 Thread Hans Verkuil
From: Hans Verkuil Implement support for this DisplayPort feature. The cec device is created whenever it detects an adapter that has this feature. It is only removed when a new adapter is connected that does not support this. If a new adapter is connected that has different properties than the p

[PATCHv2 2/3] drm-kms-helpers.rst: document the DP CEC helpers

2017-08-12 Thread Hans Verkuil
From: Hans Verkuil Document the Display Port CEC helper functions. Signed-off-by: Hans Verkuil --- Documentation/gpu/drm-kms-helpers.rst | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/gpu/drm-kms-helpers.rst b/Documentation/gpu/drm-kms-helpers.rst index 7c5e2549a58

[PATCHv2 0/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-08-12 Thread Hans Verkuil
From: Hans Verkuil This patch series adds support for the DisplayPort CEC-Tunneling-over-AUX feature. This patch series is based on 4.13-rc4 which has all the needed cec and drm 4.13 patches merged. This patch series has been tested with my NUC7i5BNK and a Samsung USB-C to HDMI adapter. Please

Re: [PATCH] v4l2-compat-ioctl32.c: make ctrl_is_pointer generic

2017-08-12 Thread Hans Verkuil
On 11/08/17 23:08, Mauro Carvalho Chehab wrote: > Em Fri, 11 Aug 2017 15:26:03 +0200 > Hans Verkuil escreveu: > >> The ctrl_is_pointer used a hard-coded list of control IDs that besides being >> outdated also wouldn't work for custom driver controls. >> >> Replaced by calling queryctrl and checki