[PATCH 2/2] [media] si2157: Improve a size determination in two functions

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 08:32:17 +0200 Replace the specification of data structures by variable references as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus

[PATCH 1/2] [media] si2157: Delete an error message for a failed memory allocation in si2157_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 08:20:04 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/tuners/si2157.c | 1 - 1 file changed, 1 deletion(-) di

[PATCH 0/2] [media] Si2157: Adjustments for two function implementations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 08:48:24 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination in two functions drivers/media/tuners/si2157.c

Re: dvb_usb_rtl2832u vs dvb_usb_rtl28xxu

2017-09-16 Thread Eyal Lebedinsky
On 10/09/17 09:29, Eyal Lebedinsky wrote: I was running f19 and had dvb_usb_rtl2832u built from git handling my "Leadtek Winfast DTV2000 DS PLUS TV" tuners. https://www.linuxtv.org/wiki/index.php/Leadtek_WinFast_DTV2000DS_PLUS I am using jaredquinn's driver Yesterday I upgraded to f22

cron job: media_tree daily build: WARNINGS

2017-09-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: Sun Sep 17 05:00:21 CEST 2017 media-tree git hash:1efdf1776e2253b77413c997bed862410e4b6aaf media_build gi

[PATCH 1/2] Store device structure in dvb_register_device

2017-09-16 Thread Jasmin J.
From: Jasmin Jessich The device created by device_create in dvb_register_device was not available for DVB device drivers. Added "struct device *dev" to "struct dvb_device" and store the created device. Signed-off-by: Jasmin Jessich --- drivers/media/dvb-core/dvbdev.c | 1 + drivers/media/dvb-c

[PATCH 2/2] Added timers for dvb_ca_en50221_write_data

2017-09-16 Thread Jasmin J.
From: Jasmin Jessich Some (older) CAMs are really slow in accepting data. The CI interface specification doesn't define a handshake for accepted data. Thus, the en50221 protocol driver can't control if a data byte has been correctly written to the CAM. The current implementation writes the lengt

[PATCH 0/2] Add timers to en50221 protocol driver

2017-09-16 Thread Jasmin J.
From: Jasmin Jessich Some (older) CAMs are really slow in accepting data. I got sometimes the already known error "CAM tried to send a buffer larger than the ecount size". I could track it down to the dvb_ca_en50221_write_data function not waiting between sending the data length high/low and d

Re: IMX6 ADV7180 no /dev/media

2017-09-16 Thread Fabio Estevam
On Fri, Sep 15, 2017 at 8:46 PM, Fabio Estevam wrote: > Tim, > > Care to send the patch enabling CONFIG_VIDEO_MUX? Nevermind. I have just sent it.

[PATCH 2/2] [media] m88rs6000t: Improve three size determinations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 21:38:03 +0200 Replace the specification of data structures by variable references as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus

[PATCH 1/2] [media] tuners: Delete an error message for a failed memory allocation in m88rs6000t_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 21:24:27 +0200 * Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. * Add a jump target so that the function "kfree" will be always called with a non-null pointer. Sig

[PATCH 0/2] [media] M88RS6000T: Adjustments for two function implementations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 21:48:45 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve three size determinations drivers/media/tuners/m88rs6000t.c | 13 ++

[PATCH 2/2] [media] it913x: Improve three size determinations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 20:06:01 +0200 Replace the specification of data structures by variable references as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus

[PATCH 1/2] [media] it913x: Delete two error messages for a failed memory allocation in it913x_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 19:40:47 +0200 * Omit extra messages for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. * Delete the label "err" and the variable "ret" which became unnecessary with this refactoring. Signe

[PATCH 0/2] [media] IT913X: Adjustments for two function implementations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 20:12:34 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete two error messages for a failed memory allocation Improve three size determinations drivers/media/tuners/it913x.c | 3 +--

Re: [GIT PULL] SAA716x DVB driver

2017-09-16 Thread Mauro Carvalho Chehab
Em Sat, 16 Sep 2017 14:54:15 +0200 Soeren Moch escreveu: > On 09.09.2017 23:20, Mauro Carvalho Chehab wrote: > > Em Sat, 9 Sep 2017 14:52:18 +0200 > > Soeren Moch escreveu: > > > >> You explicitly > >> want to discourage new driver and application implementations. > > Me? It is just the op

[PATCH 2/2] [media] fc0012: Improve a size determination in fc0012_attach()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 17:55:27 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was det

[PATCH 1/2] [media] fc0012: Delete an error message for a failed memory allocation in fc0012_attach()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 17:47:52 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/tuners/fc0012.c | 1 - 1 file changed, 1 deletion(-) di

[PATCH 0/2] [media] FC0012: Adjustments for fc0012_attach()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 18:03:21 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination drivers/media/tuners/fc0012.c | 3 +-- 1 file

[PATCH] [media] Si4713: Delete an error message for a failed memory allocation in two functions

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 16:15:44 +0200 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/radio/si4713/radio-platform-si4713.c | 1 - drivers/me

[PATCHv5 2/5] cec-ioc-dqevent.rst: document new CEC_EVENT_PIN_HPD_LOW/HIGH events

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil Document these new CEC events. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/cec/cec-ioc-dqevent.rst | 18 ++ 1 file changed, 18 insertions(+) diff --git a/Documentation/media/uapi/cec/cec-ioc-dqevent.rst b/Documentation/media/uapi/cec/cec-ioc-dq

[PATCHv5 0/5] cec-gpio: add HDMI CEC GPIO-based driver

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil This driver adds support for CEC implementations that use a pull-up GPIO line. While SoCs exist that do this, the primary use-case is to turn a single-board computer into a cheap CEC debugger. Together with 'cec-ctl --monitor-pin' you can do low-level CEC bus monitoring and do

[PATCHv5 4/5] cec-gpio: add HDMI CEC GPIO driver

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil Add a simple HDMI CEC GPIO driver that sits on top of the cec-pin framework. While I have heard of SoCs that use the GPIO pin for CEC (apparently an early RockChip SoC used that), the main use-case of this driver is to function as a debugging tool. By connecting the CEC line

[PATCHv5 1/5] cec: add CEC_EVENT_PIN_HPD_LOW/HIGH events

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil Add support for two new low-level events: PIN_HPD_LOW and PIN_HPD_HIGH. This is specifically meant for use with the upcoming cec-gpio driver and makes it possible to trace when the HPD pin changes. Some HDMI sinks do strange things with the HPD and this makes it easy to debug

[PATCHv5 5/5] MAINTAINERS: add cec-gpio entry

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil Add an entry for the CEC GPIO driver. Signed-off-by: Hans Verkuil --- MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index eb930ebecfcb..5ef0d34ef502 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3205,6 +3205,15 @@ F:

[PATCHv5 3/5] dt-bindings: document the CEC GPIO bindings

2017-09-16 Thread Hans Verkuil
From: Hans Verkuil Document the bindings for the cec-gpio module for hardware where the CEC line and optionally the HPD line are connected to GPIO lines. Signed-off-by: Hans Verkuil --- .../devicetree/bindings/media/cec-gpio.txt | 29 ++ 1 file changed, 29 insertion

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

2017-09-16 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

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

2017-09-16 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

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

2017-09-16 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 the current pre-4.14-rc1 mainline which has all the needed cec 4.14 patches merged. This patch series has been tested with my NUC7i5BNK and a Samsung USB-C to HDMI

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

2017-09-16 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

[PATCH 3/3] [media] si470x: Delete an unnecessary variable initialisation in si470x_usb_driver_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 15:08:39 +0200 The variable "retval" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/radio/si470x/radio-si470x-usb.c | 2 +- 1 file ch

[PATCH 2/3] [media] si470x: Improve a size determination in si470x_usb_driver_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 14:58:06 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was det

[PATCH 1/3] [media] si470x: Delete an error message for a failed memory allocation in si470x_usb_driver_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 14:53:49 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/radio/si470x/radio-si470x-usb.c | 1 - 1 file changed, 1

[PATCH 0/3] [media] si470x: Adjustments for si470x_usb_driver_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 15:16:17 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation Improve a size determination Delete an unnecessary variable initialisatio

Re: [GIT PULL] SAA716x DVB driver

2017-09-16 Thread Soeren Moch
On 09.09.2017 23:20, Mauro Carvalho Chehab wrote: > Em Sat, 9 Sep 2017 14:52:18 +0200 > Soeren Moch escreveu: > >> Hi Mauro, >> >> sorry for the late reply, unfortunately your email was filtered out as spam. > I'll focus talking about saa716x/av7110 here, as this is out of of > scope of the docume

[PATCH 3/3] [media] WL1273: Delete an unnecessary variable initialisation in wl1273_fm_suspend()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 13:55:56 +0200 The local variable "r" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-wl1273.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 2/3] [media] WL1273: Delete an unnecessary goto statement in wl1273_fm_suspend()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 13:53:22 +0200 * Remove an extra goto statement. * Delete the label "out" which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-wl1273.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-)

[PATCH 1/3] [media] WL1273: Delete an error message for a failed memory allocation in wl1273_fm_radio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 13:28:38 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-wl1273.c | 1 - 1 file changed, 1 deletion(-

[PATCH 0/3] [media] WL1273: Adjustments for two function implementations

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 14:05:45 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation Delete an unnecessary goto statement Delete an unnecessary variable initi

[PATCH 3/3] [media] mr800: Delete an unnecessary variable initialisation in usb_amradio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 11:39:50 +0200 The variable "retval" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-mr800.c | 2 +- 1 file changed, 1 ins

Re: [PATCH] [media] solo6x10: hide unused variable

2017-09-16 Thread Anton Sviridenko
On Fri, Sep 15, 2017 at 09:52:04PM +0200, Arnd Bergmann wrote: > When building without CONFIG_GPIOLIB, we get a harmless > warning about an unused variable: > > drivers/media/pci/solo6x10/solo6x10-gpio.c: In function 'solo_gpio_init': > drivers/media/pci/solo6x10/solo6x10-gpio.c:165:6: error: unu

[PATCH 2/3] [media] mr800: Improve a size determination in usb_amradio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 11:34:11 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was det

[PATCH 1/3] [media] mr800: Delete two error messages for a failed memory allocation in usb_amradio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 11:23:53 +0200 Omit extra messages for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-mr800.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH 0/3] [media] MR800: Adjustments for usb_amradio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 12:35:43 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete two error messages for a failed memory allocation Improve a size determination Delete an unnecessary variable initialisat

[PATCH] [media] Maxi Radio: Delete an error message for a failed memory allocation in maxiradio_probe()

2017-09-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Sep 2017 10:15:29 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/radio/radio-maxiradio.c | 4 +--- 1 file changed, 1 inse

Re: [PATCH v13 06/25] omap3isp: Use generic parser for parsing fwnode endpoints

2017-09-16 Thread Pavel Machek
On Sat 2017-09-16 09:04:31, Pavel Machek wrote: > Hi! > > > Instead of using driver implementation, use > > v4l2_async_notifier_parse_fwnode_endpoints() to parse the fwnode endpoints > > of the device. > > > > Signed-off-by: Sakari Ailus > > Acked-by: Hans Verkuil > > Patches at least up to he

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-16 Thread Pavel Machek
On Fri 2017-09-15 17:17:10, Sakari Ailus wrote: > Add three helper functions to call async operations callbacks. Besides > simplifying callbacks, this allows async notifiers to have no ops set, > i.e. it can be left NULL. > > Signed-off-by: Sakari Ailus > Acked-by: Hans Verkuil I'd remove "_cal

Re: [PATCH v13 06/25] omap3isp: Use generic parser for parsing fwnode endpoints

2017-09-16 Thread Pavel Machek
Hi! > Instead of using driver implementation, use > v4l2_async_notifier_parse_fwnode_endpoints() to parse the fwnode endpoints > of the device. > > Signed-off-by: Sakari Ailus > Acked-by: Hans Verkuil Patches at least up to here look fine o me. We are at version 13 of the series... Is merge o