Re: Help required for TVP5151 on Overo

2014-12-22 Thread Francesco Marletta
Il giorno Sat, 13 Dec 2014 00:50:10 +0200 Laurent Pinchart laurent.pinch...@ideasonboard.com ha scritto: Hi Francesco, On Friday 12 December 2014 16:38:02 Francesco Marletta wrote: Hi Laurent, I'll check the patches you indicated on git://linuxtv.org/pinchartl/media.git

Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-22 Thread Josh Wu
Hi, Guennadi On 12/20/2014 6:16 AM, Guennadi Liakhovetski wrote: On Fri, 19 Dec 2014, Josh Wu wrote: Hi, Guennadi Thanks for the review. On 12/19/2014 5:59 AM, Guennadi Liakhovetski wrote: Hi Josh, Thanks for your patches! On Thu, 18 Dec 2014, Josh Wu wrote: To support async probe for

Re: [PATCH v4 5/5] media: ov2640: dt: add the device tree binding document

2014-12-22 Thread Josh Wu
Hi, Sylwester On 12/18/2014 8:13 PM, Sylwester Nawrocki wrote: Hi Josh, On 18/12/14 03:27, Josh Wu wrote: Add the document for ov2640 dt. Cc: devicet...@vger.kernel.org Signed-off-by: Josh Wu josh...@atmel.com Acked-by: Sylwester Nawrocki s.nawro...@samsung.com Thanks. It seems ovti is

SuperH Mobile CEU driver warning

2014-12-22 Thread Geert Uytterhoeven
commit 454a4e728dd56c8515b80381c14168099279e7fc ([media] v4l2-ioctl: WARN_ON if querycap didn't fill device_caps) causes on r8a7740/armadillo-legacy: sh_mobile_ceu sh_mobile_ceu.0: SuperH Mobile CEU driver attached to camera 0 sh_mobile_ceu sh_mobile_ceu.0: pm_clk_resume() [ cut here

Re: [PATCH v4 5/5] media: ov2640: dt: add the device tree binding document

2014-12-22 Thread Sylwester Nawrocki
Hi Josh, On 22/12/14 11:32, Josh Wu wrote: +Required Properties: +- compatible: Must be ovti,ov2640 I believe it is preferred to put it as Should contain, rather than Must be. I don't have a strong opinion here. After check many documents, it seems many people use Should be. Is it

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mark Brown
On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: Lockdep validator complains recursive locking and deadlock when two different regmap instances are called in a nested order, as regmap groups locks by default. That happens easily for example when both I don't know what regmap

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Antti Palosaari
On 12/22/2014 02:44 PM, Mark Brown wrote: On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: Lockdep validator complains recursive locking and deadlock when two different regmap instances are called in a nested order, as regmap groups locks by default. That happens easily for

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Steven Toth
On Sun, Dec 21, 2014 at 2:59 PM, Matthias Schwarzott z...@gentoo.org wrote: Unconditionally attaching Si2161/Si2165 demod driver breaks Hauppauge WinTV Starburst. So create own card entry for this. Add card name comments to the subsystem ids. Signed-off-by: Matthias Schwarzott

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Matthias Schwarzott
On 22.12.2014 13:59, Steven Toth wrote: On Sun, Dec 21, 2014 at 2:59 PM, Matthias Schwarzott z...@gentoo.org wrote: Unconditionally attaching Si2161/Si2165 demod driver breaks Hauppauge WinTV Starburst. So create own card entry for this. Add card name comments to the subsystem ids.

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Steven Toth
Please change CX23885_BOARD_HAUPPAUGE_STARBURST to CX23885_BOARD_HAUPPAUGE_HVR5500. Thanks, Hi Steven, thank you for your feedback. I rechecked the names and this are the more or less supported devices: * Starburst supports DVB-S2 only * HVR-4400 supports DVB-S2 + DVB-T (Si2161) *

Re: [PATCH 6/7] ARM: at91: dts: sama5d3: add ov2640 camera sensor support

2014-12-22 Thread Alexandre Belloni
Hi Josh, On 22/12/2014 at 15:06:07 +0800, Josh Wu wrote : I've acked your previous patch but maybe it should be named pinctrl_isi_pck1_as_mck to be clearer (you used the handle to pck1 below). It's a good idea. Maybe I prefer to use the name: pinctrl_pck1_as_isi_mck ? If you are ok with this

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Mauro Carvalho Chehab
Em Sun, 21 Dec 2014 21:07:02 +0100 Matthias Schwarzott z...@gentoo.org escreveu: Hi! Should the commit message directly point to the breaking commit 36efec48e2e6016e05364906720a0ec350a5d768? Yes, if this fixes an issue that happened on a previous commit, then you should add the original

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mark Brown
On Mon, Dec 22, 2014 at 02:55:23PM +0200, Antti Palosaari wrote: On 12/22/2014 02:44 PM, Mark Brown wrote: On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: I2C client and I2C adapter are using regmap. As a solution, add configuration option to pass custom lock class key for

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Matthias Schwarzott
On 22.12.2014 14:25, Mauro Carvalho Chehab wrote: Em Sun, 21 Dec 2014 21:07:02 +0100 Matthias Schwarzott z...@gentoo.org escreveu: Hi! Should the commit message directly point to the breaking commit 36efec48e2e6016e05364906720a0ec350a5d768? Yes, if this fixes an issue that happened on a

Re: [PATCH v2 12/13] ARM: dts: sun6i: Add sun6i-a31s.dtsi

2014-12-22 Thread Hans de Goede
Hi, On 21-12-14 23:39, Maxime Ripard wrote: On Sat, Dec 20, 2014 at 11:24:55AM +0100, Hans de Goede wrote: Hi, On 19-12-14 19:34, Maxime Ripard wrote: On Wed, Dec 17, 2014 at 06:18:23PM +0100, Hans de Goede wrote: Add a dtsi file for A31s based boards. Since the A31s is the same die as

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Antti Palosaari
On 12/22/2014 03:31 PM, Mark Brown wrote: On Mon, Dec 22, 2014 at 02:55:23PM +0200, Antti Palosaari wrote: On 12/22/2014 02:44 PM, Mark Brown wrote: On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: I2C client and I2C adapter are using regmap. As a solution, add configuration

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Antti Palosaari
On 12/22/2014 03:36 PM, Matthias Schwarzott wrote: On 22.12.2014 14:25, Mauro Carvalho Chehab wrote: Em Sun, 21 Dec 2014 21:07:02 +0100 Matthias Schwarzott z...@gentoo.org escreveu: Hi! Should the commit message directly point to the breaking commit

Re: [PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Steven Toth
In my understanding Starburst is HVR-4400, but only satellite tuner is installed to PCB - whilst terrestrial/cable is left out. I went back through my engineering notes from my Hauppauge engineering days, this statement is correct. -- Steven Toth - Kernel Labs http://www.kernellabs.com -- To

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mauro Carvalho Chehab
Em Mon, 22 Dec 2014 13:31:42 + Mark Brown broo...@kernel.org escreveu: On Mon, Dec 22, 2014 at 02:55:23PM +0200, Antti Palosaari wrote: On 12/22/2014 02:44 PM, Mark Brown wrote: On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: I2C client and I2C adapter are using

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mark Brown
On Mon, Dec 22, 2014 at 03:53:10PM +0200, Antti Palosaari wrote: On 12/22/2014 03:31 PM, Mark Brown wrote: Why is this configurable, how would a device know if the system it is in needs a custom locking class and can safely use one? If RegMap instance is bus master, eg. I2C adapter, then you

[PATCH v6 2/3] of: Add for_each_endpoint_of_node helper macro

2014-12-22 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel p.za...@pengutronix.de Acked-by: Laurent Pinchart

[PATCH v6 3/3] of: Add of_graph_get_port_by_id function

2014-12-22 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel p.za...@pengutronix.de Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- drivers/of/base.c| 26 ++ include/linux/of_graph.h |

[PATCH v6 1/3] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-12-22 Thread Philipp Zabel
Decrementing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. All current users of this function that pass a non-NULL prev parameter (that is, soc_camera and imx-drm) are changed to not decrement the passed prev

[PATCH v6 0/3] Add of-graph helpers to loop over endpoints and find ports by id

2014-12-22 Thread Philipp Zabel
Hi, next try for v3.20. After the merge window we have a few new users of of_graph_get_next_endpoint, could I please get some acks from the respective maintainers for this to go in through Grant's tree? This series converts all existing users of of_graph_get_next_endpoint that pass a non-NULL

Re: coda: Unable to use encoder video_bitrate

2014-12-22 Thread Frédéric Sureau
Hi Philipp, Le 18/12/2014 17:52, Philipp Zabel a écrit : Hi Frédéric, Am Donnerstag, den 18.12.2014, 17:44 +0100 schrieb Frédéric Sureau: Hi I am trying to use the coda encoder through Gstreamer on an iMX6-based board. I use the (rebased and slightly modified) gstv4l2h264enc plugin from:

Re: coda: Unable to use encoder video_bitrate

2014-12-22 Thread Nicolas Dufresne
Le 2014-12-22 10:24, Frédéric Sureau a écrit : Thanks for the patch! It works fine now after forcing framerate to 30fps (which seems to be hardcoded in the driver) Can you comment about this on gnome bug. Would make sense for the encoder element in GStreamer to relay the framerate to the

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mark Brown
On Mon, Dec 22, 2014 at 12:23:19PM -0200, Mauro Carvalho Chehab wrote: What this patch does is to offer a way for drivers B and C to define different mutex groups (e. g. different mutex IDs) that will teach the lockdep code to threat regmap mutex on drivers B and C as different mutexes. I

[RFC PATCH] [media] coda: Use S_PARM to set nominal framerate for h.264 encoder

2014-12-22 Thread Philipp Zabel
The encoder needs to know the nominal framerate for the constant bitrate control mechanism to work. Currently the only way to set the framerate is by using VIDIOC_S_PARM on the output queue. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/platform/coda/coda-common.c | 29

Re: coda: Unable to use encoder video_bitrate

2014-12-22 Thread Philipp Zabel
Hi Frédéric, On Mon, Dec 22, 2014 at 04:24:20PM +0100, Frédéric Sureau wrote: Thanks for the patch! It works fine now after forcing framerate to 30fps (which seems to be hardcoded in the driver) That is a good point, rate control can only work if the encoder has an idea about the framerate.

Re: coda: Unable to use encoder video_bitrate

2014-12-22 Thread Nicolas Dufresne
Le 2014-12-22 11:02, Philipp Zabel a écrit : That is a good point, rate control can only work if the encoder has an idea about the framerate. I've sent a patch that allows to use VIDIOC_S_PARM to set it: [media] coda: Use S_PARM to set nominal framerate for h.264 encoder Thanks, I'll sort out

It's important i discuss with you.

2014-12-22 Thread Mrs Abdam Mohammed
Assalamualaikum, Peace, mercy and blessing of Allah, I am Mrs. Abdam Mohammed from Benghazi Libya.I have an important private discussion to make with you and for privacy sake, I will be glad that you please confirm to me if your email is secured or not. Alternatively please can you provide me with

It's important i discuss with you.

2014-12-22 Thread Mrs Abdam Mohammed
Assalamualaikum, Peace, mercy and blessing of Allah, I am Mrs. Abdam Mohammed from Benghazi Libya.I have an important private discussion to make with you and for privacy sake, I will be glad that you please confirm to me if your email is secured or not. Alternatively please can you provide me with

pd-video.c:undefined reference to `usb_autopm_put_interface'

2014-12-22 Thread Fengguang Wu
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 97bf6af1f928216fd6c5a66e8a57bfa95a659672 commit: 0ec28c37c21a2b4393692e832e11a7573ac545e2 Merge tag 'media/v3.19-2' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media date: 3 days ago

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2014-12-22 Thread Mark Brown
On Sun, Dec 21, 2014 at 12:34:51AM +0200, Antti Palosaari wrote: + * @lock_class_key: Custom lock class key for lockdep validator. Use that when + *regmap in question is used for bus master IO in order to avoid + *false lockdep nested locking warning. Valid

[PATCH] media: tlg2300: disable building the driver

2014-12-22 Thread Jim Davis
This driver doesn't build with the current kernel, as reported in linux-next (https://lkml.org/lkml/2014/12/18/483) and by the 0-day build system (https://www.mail-archive.com/linux-media@vger.kernel.org/msg83501.html). Since it's scheduled for removal, disable building it for now. Signed-off-by:

Re: [PATCH v6 1/3] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-12-22 Thread Mathieu Poirier
On 22 December 2014 at 08:11, Philipp Zabel p.za...@pengutronix.de wrote: Decrementing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. All current users of this function that pass a non-NULL prev parameter

Re: Support for Elgato Game Capture HD / MStar MST3367CMK

2014-12-22 Thread Tolga Cakir
Am 09.05.2014 um 14:38 schrieb Steven Toth: On Thu, May 8, 2014 at 9:48 PM, Tolga Cakir to...@cevel.net wrote: Hello everyone! Hi Tolga! Over the past weeks, I've been busy capturing USB packets between the Elgato Game Capture HD and my PC. It's using the MStar MST3367CMK chip, which seems

[PATCH 2/2] staging: media: bcm2048: Remove obsolete cleanup for clientdata

2014-12-22 Thread Wolfram Sang
A few new i2c-drivers came into the kernel which clear the clientdata-pointer on exit or error. This is obsolete meanwhile, the core will do it. Signed-off-by: Wolfram Sang w...@the-dreams.de --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH] cx23885: Split Hauppauge WinTV Starburst from HVR4400 card entry

2014-12-22 Thread Matthias Schwarzott
Unconditionally attaching Si2161/Si2165 demod driver breaks Hauppauge WinTV Starburst. So create own card entry for this. Add card name comments to the subsystem ids. This fixes a regression introduced in 3.17 by 36efec48e2e6016e05364906720a0ec350a5d768 ([media] cx23885: Add si2165 support for

Re: Support for Elgato Game Capture HD / MStar MST3367CMK

2014-12-22 Thread Steven Toth
Comments inline On Mon, Dec 22, 2014 at 4:07 PM, Tolga Cakir to...@cevel.net wrote: Am 09.05.2014 um 14:38 schrieb Steven Toth: On Thu, May 8, 2014 at 9:48 PM, Tolga Cakir to...@cevel.net wrote: Hello everyone! Hi Tolga! Over the past weeks, I've been busy capturing USB packets between

cron job: media_tree daily build: ERRORS

2014-12-22 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: Tue Dec 23 04:00:24 CET 2014 git branch: test git hash: 654a731be1a0b6f606f3f3d12b50db08f2ae3c34 gcc