cron job: media_tree daily build: WARNINGS

2017-02-18 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 Feb 19 05:00:22 CET 2017 media-tree git hash:9eeb0ed0f30938f31a3d9135a88b9502192c18dd media_build git

Re: [RFC 04/13] omap3isp: add support for CSI1 bus

2017-02-18 Thread Pavel Machek
Hi! I guess I'll need some help here. > @@ -160,6 +163,33 @@ static int ccp2_if_enable(struct isp_ccp2_device *ccp2, > u8 enable) > return ret; > } > > + if (isp->revision == ISP_REVISION_2_0) { > + struct media_pad *pad; > + struct v4l2_

Re: [PATCH] v4l-utils: fix invalid protocol in streamzap keymap

2017-02-18 Thread Sean Young
On Fri, Feb 17, 2017 at 10:19:16AM +0100, Matthias Reichl wrote: > ir-keytable can't load the streamzap keymap because the > protocol type RC5_SZ is invalid: > > ./ir-keytable -w rc_keymaps/streamzap > Protocol RC5_SZ invalid > ... > > Fix this by changing the protocol type to RC-5-SZ which > mat

Re: [PATCH v4 29/36] media: imx: mipi-csi2: enable setting and getting of frame rates

2017-02-18 Thread Russell King - ARM Linux
On Sat, Feb 18, 2017 at 09:29:17AM -0800, Steve Longerbeam wrote: > On 02/18/2017 01:23 AM, Russell King - ARM Linux wrote: > >On Fri, Feb 17, 2017 at 05:12:44PM -0800, Steve Longerbeam wrote: > >>Hi Russell, > >> > >>I signed-off on this but after more review I'm not sure this is right. > >> > >>T

Re: [PATCH v4 00/36] i.MX Media Driver

2017-02-18 Thread Steve Longerbeam
On 02/16/2017 02:57 PM, Russell King - ARM Linux wrote: On Thu, Feb 16, 2017 at 02:27:41PM -0800, Steve Longerbeam wrote: On 02/16/2017 02:20 PM, Russell King - ARM Linux wrote: On Wed, Feb 15, 2017 at 06:19:02PM -0800, Steve Longerbeam wrote: In version 4: With this version, I get: [28

Re: [PATCH v4 29/36] media: imx: mipi-csi2: enable setting and getting of frame rates

2017-02-18 Thread Steve Longerbeam
On 02/18/2017 01:23 AM, Russell King - ARM Linux wrote: On Fri, Feb 17, 2017 at 05:12:44PM -0800, Steve Longerbeam wrote: Hi Russell, I signed-off on this but after more review I'm not sure this is right. The CSI-2 receiver really has no control over frame rate. It's output frame rate is the

Re: [PATCH v4 00/36] i.MX Media Driver

2017-02-18 Thread Sakari Ailus
Hi Philipp and Russell, On Fri, Feb 17, 2017 at 04:04:30PM +0100, Philipp Zabel wrote: > On Fri, 2017-02-17 at 14:22 +0200, Sakari Ailus wrote: > > Hi Philipp, Steve and Russell, > > > > On Fri, Feb 17, 2017 at 12:43:38PM +0100, Philipp Zabel wrote: > > > On Thu, 2017-02-16 at 14:27 -0800, Steve

Re: [patch v2] staging: bcm2835-camera: fix error handling in init

2017-02-18 Thread walter harms
looks more readable now :) Acked-by: wha...@bfs.de Am 18.02.2017 00:20, schrieb Dan Carpenter: > The unwinding here isn't right. We don't free gdev[0] and instead > free 1 step past what was allocated. Also we can't allocate "dev" then > we should unwind instead of returning directly. > > Fixe

Re: Dead code in v4l2-mem2mem.c?

2017-02-18 Thread Laurent Pinchart
Hi Shaobo, On Friday 17 Feb 2017 11:42:25 Shaobo wrote: > Hi Laurent, > > Thanks a lot for your reply. > > I would like to also point out the inconsistency of using `v4l2_m2m_get_vq` > inside drivers/media/v4l2-core/v4l2-mem2mem.c and inside other files. It > appears to me almost all call sites

Re: [PATCH v4 29/36] media: imx: mipi-csi2: enable setting and getting of frame rates

2017-02-18 Thread Russell King - ARM Linux
On Fri, Feb 17, 2017 at 05:12:44PM -0800, Steve Longerbeam wrote: > Hi Russell, > > I signed-off on this but after more review I'm not sure this is right. > > The CSI-2 receiver really has no control over frame rate. It's output > frame rate is the same as the rate that is delivered to it. > > S