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

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 10:03: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

Re: dvb_usb_rtl2832u vs dvb_usb_rtl28xxu

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

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

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

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

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

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

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

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

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 09:33:58 +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

[PATCH 2/2] [media] airspy: Improve a size determination in airspy_probe()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 11:03:21 +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

[PATCH] [media] au0828: Delete an error message for a failed memory allocation in au0828_usb_probe()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 11:40:31 +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 ---

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

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

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

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

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

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 09:42: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

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

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 11:00:09 +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 ---

[PATCH 0/2] [media] AirSpy: Adjustments for airspy_probe()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 11:11:02 +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

[PATCH 2/4] [media] cpia2: Adjust two function calls together with a variable assignment

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 12:56:50 +0200 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code places. Signed-off-by: Markus Elfring

[PATCH 1/4] [media] cpia2: Use common error handling code in cpia2_usb_probe()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 12:40:14 +0200 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 3/4] [media] cpia2: Delete unnecessary null pointer checks in free_sbufs()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 13:23:47 +0200 The script "checkpatch.pl" pointed information out like the following. WARNING: kfree(NULL) is safe and this check is probably not required Thus fix the affected source code place. Signed-off-by: Markus

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

2017-09-17 Thread Linus Walleij
On Sat, Sep 16, 2017 at 4:28 PM, Hans Verkuil wrote: > 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

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

2017-09-17 Thread Linus Walleij
On Sat, Sep 16, 2017 at 4:28 PM, Hans Verkuil wrote: > 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

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

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

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

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

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

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

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

2017-09-17 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 Reviewed-by: Linus Walleij ---

[PATCH 0/4] [media] CPia2: Fine-tuning for four function implementations

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 15:25:35 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): Use common error handling code in cpia2_usb_probe() Adjust two function calls together with a

[PATCH 4/4] [media] cpia2: Delete an unnecessary return statement in process_frame()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 13:32:45 +0200 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function.

'LITE-ON USB2.0 DVB-T Tune' not working with kernel 4.10 / ubuntu 17.04

2017-09-17 Thread Laurent Caumont
Hello, The hardware was working with kernel 4.4. I'm trying to make my DVB-T to work with my Ubuntu 17.04 and kernel 4.10. The complied version installed didn't work (with error: dvb_usb_dibusb_mc_common: Unknown symbol __symbol_get (err 0)) So I'm trying to make it work with the latest code

[PATCH] scan-tables: add scan file for Shostka, Ukraine

2017-09-17 Thread Oleh Kravchenko
Signed-off-by: Oleh Kravchenko --- dvb-t/ua-Shostka | 55 +++ 1 file changed, 55 insertions(+) create mode 100644 dvb-t/ua-Shostka diff --git a/dvb-t/ua-Shostka b/dvb-t/ua-Shostka new file mode 100644 index 000..696a428

Re: [PATCH v1 1/3] media: vsp1: Prevent resuming DRM pipelines

2017-09-17 Thread Kieran Bingham
Hi Laurent, On 15/09/17 17:58, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Friday, 15 September 2017 19:42:05 EEST Kieran Bingham wrote: >> DRM pipelines utilising the VSP must stop all frame processing as part >> of the suspend operation to ensure the hardware is

[PATCH 3/8] [media] cx231xx: Improve six size determinations

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 18:38:50 +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

[PATCH 0/8] [media] Cx231xx: Adjustments for several function implementations

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 21:30:12 +0200 Some update suggestions were taken into account from static source code analysis. Markus Elfring (8): Delete eight error messages for a failed memory allocation Adjust 56 checks for null pointers

[PATCH 5/8] [media] cx231xx: Use common error handling code in dvb_init()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 20:10:34 +0200 Add jump targets so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring ---

[PATCH 4/8] [media] cx231xx: Delete an unnecessary variable initialisation in dvb_init()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 18:53:16 +0200 The variable "result" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring ---

[PATCH 6/8] [media] cx231xx: Use common error handling code in read_eeprom()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 20:22:15 +0200 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 7/8] [media] cx231xx: Delete an unnecessary variable initialisation in read_eeprom()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 20:28:00 +0200 The local variable "ret" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring ---

[PATCH] Support HVR-1200 analog video as a clone of HVR-1500. Tested, composite and s-video inputs.

2017-09-17 Thread Nigel Kettlewell
I propose the following patch to support Hauppauge HVR-1200 analog video, nothing more than a clone of HVR-1500. Patch based on Linux 4.9 commit 69973b830859bc6529a7a0468ba0d80ee5117826 I have tested composite and S-Video inputs. With the change, HVR-1200 devices have a /dev/video entry which

[PATCH 2/8] [media] cx231xx: Adjust 56 checks for null pointers

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 18:23:06 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus

[PATCH 1/8] [media] cx231xx: Delete eight error messages for a failed memory allocation

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 17:43:47 +0200 Omit extra messages for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 8/8] [media] cx231xx: Use common error handling code in cx231xx_load_firmware()

2017-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 17 Sep 2017 21:07:39 +0200 Add jump targets so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring ---

[PATCH] media: i2c: tc358743: fix spelling mistake

2017-09-17 Thread Jacob Chen
It should be "LP-11". Signed-off-by: Jacob Chen --- drivers/media/i2c/tc358743.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c index e6f5c36..c208c30 100644 ---

Urgent: dvb_usb_rtl28xxu not tuning "Leadtek Winfast DTV2000 DS PLUS TV"

2017-09-17 Thread Eyal Lebedinsky
I have just upgraded to f24. I am now using the standard dvb_usb_rtl28xxu fe which logs messages suggesting all is well (I get the /dev/dvb/adapter? etc.) but I get no channels tuned when I run mythfrontend or scandvb. Is anyone using this combination? Is this the correct way to use this tuner?

[PATCH v3 0/3] media: ov7670: Add entity init and power operation

2017-09-17 Thread Wenyou Yang
This patch set is to add the media entity pads initialization, the s_power operation and get_fmt callback support. Changes in v3: - Keep tried format info in the try_fmt member of v4l2_subdev__pad_config struct. - Add the internal_ops callback to set default format. Changes in v2: - Add

[PATCH v3 1/3] media: ov7670: Add entity pads initialization

2017-09-17 Thread Wenyou Yang
Add the media entity pads initialization. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: None drivers/media/i2c/ov7670.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov7670.c

[PATCH v3 3/3] media: ov7670: Add the s_power operation

2017-09-17 Thread Wenyou Yang
Add the s_power operation which is responsible for manipulating the power dowm mode through the PWDN pin and the reset operation through the RESET pin. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: - Add the patch to support the get_fmt ops. -

[PATCH v3 2/3] media: ov7670: Add the get_fmt callback

2017-09-17 Thread Wenyou Yang
Add the get_fmt callback, also enable V4L2_SUBDEV_FL_HAS_DEVNODE flag to make this subdev has device node. Signed-off-by: Wenyou Yang --- Changes in v3: - Keep tried format info in the try_fmt member of v4l2_subdev__pad_config struct. - Add the internal_ops

Re: [PATCH v3 1/3] media: ov7670: Add entity pads initialization

2017-09-17 Thread kbuild test robot
Hi Wenyou, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.14-rc1 next-20170915] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH v3 2/3] media: ov7670: Add the get_fmt callback

2017-09-17 Thread kbuild test robot
Hi Wenyou, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.14-rc1 next-20170915] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

cron job: media_tree daily build: WARNINGS

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