cron job: media_tree daily build: OK

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 Mar 15 04:00:22 CET 2016 git branch: test git hash: da470473c9cf9c4ebb40d046b306c76427b6df94 gcc

Re: [PATCH v2] [media] media-device: Don't call notify callbacks holding a spinlock

On 03/14/2016 01:19 PM, Mauro Carvalho Chehab wrote: > The notify routines may sleep. So, they can't be called in spinlock > context. Also, they may want to call other media routines that might > be spinning the spinlock, like creating a link. > > This fixes the following bug: > > [ 1839.510587]

[PATCH v4 0/2] media: soc_camera: rcar_vin: add fallback and r8a7792 bindings

Hi, this short series adds add fallback and r8a7792 bindings to rcar_vin. Based on media-tree/master Changes since v3: * Add Acks * Correct typo in changelog Simon Horman (1): media: soc_camera: rcar_vin: add device tree support for r8a7792 Yoshihiro Kaneko (1): media: soc_camera:

[PATCH v4 2/2] media: soc_camera: rcar_vin: add device tree support for r8a7792

Simply document new compatibility string. As a previous patch adds a generic R-Car Gen2 compatibility string there appears to be no need for a driver updates. By documenting this compat string it may be used in DTSs shipped, for example as part of ROMs. It must be used in conjunction with the

[PATCH v4 1/2] media: soc_camera: rcar_vin: add R-Car Gen 2 and 3 fallback compatibility strings

From: Yoshihiro Kaneko Add fallback compatibility string for R-Car Gen 1 and 2. In the case of Renesas R-Car hardware we know that there are generations of SoCs, e.g. Gen 2 and 3. But beyond that its not clear what the relationship between IP blocks might be. For example,

[PATCH 3/3] [media] v4l2-mc: remove unused dtv_demod variable

A recent patch removed the only user of the 'dtv_demod' variable in v4l2_mc_create_media_graph, but did not remove the declaration, possibly as a result of an incorrect rebase: drivers/media/v4l2-core/v4l2-mc.c: In function 'v4l2_mc_create_media_graph': drivers/media/v4l2-core/v4l2-mc.c:37:55:

[PATCH 2/3] [media] am437x-vfpe: fix typo in vpfe_get_app_input_index

gcc-6 points out an obviously silly comparison in vpfe_get_app_input_index(): drivers/media/platform/am437x/am437x-vpfe.c: In function 'vpfe_get_app_input_index': drivers/media/platform/am437x/am437x-vpfe.c:1709:27: warning: self-comparison always evaluats to true [-Wtautological-compare]

[PATCH 1/3] [media] cobalt: add MTD dependency

The cobalt driver fails to link when it is built-in and MTD is disabled or a loadable module: drivers/media/built-in.o: In function `cobalt_flash_probe': :(.text+0xb8b46): undefined reference to `mtd_device_parse_register' :(.text+0xb8b88): undefined reference to `do_map_probe'

[PATCH] media: Update documentation for media_entity_notify

Update documentation for media_entity_notify to clearly state the usage restrictions. This handler is intended for creating links between exiting entities and should not used to create and register entities. Signed-off-by: Shuah Khan --- include/media/media-device.h | 6

[PATCH] include sys/sysmacros.h for major() & minor()

Linux C libraries are looking to disentangle sysmacros.h from the sys/types.h header to clean up namespace pollution. Since these macros are provided in glibc/etc... today, switch to pulling in this header directly. Signed-off-by: Mike Frysinger ---

Re: [PATCH] [media] media-device: Don't call notify callbacks holding a spinlock

Hi Mauro, [auto build test WARNING on sailus-media/master] [also build test WARNING on next-20160314] [cannot apply to v4.5] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Mauro-Carvalho

[PATCH v2] [media] media-device: Don't call notify callbacks holding a spinlock

The notify routines may sleep. So, they can't be called in spinlock context. Also, they may want to call other media routines that might be spinning the spinlock, like creating a link. This fixes the following bug: [ 1839.510587] BUG: sleeping function called from invalid context at

[PATCH] [media] media-device: Don't call notify callbacks holding a spinlock

The notify routines may sleep. So, they can't be called in spinlock context. Also, they may want to call other media routines that might be spinning the spinlock, like creating a link. This fixes the following bug: [ 1839.510587] BUG: sleeping function called from invalid context at

Re: [PATCH v3 3/9] [media] tvp5150: determine BT.656 or YUV 4:2:2 mode from device tree

Hello Lucas, On Mon, Mar 14, 2016 at 12:23 PM, Lucas Stach wrote: > From: Philipp Zabel > > By looking at the endpoint flags, it can be determined whether the link > should be of V4L2_MBUS_PARALLEL or V4L2_MBUS_BT656 type. Disable the > dedicated

Re: Any reason why media_entity_pads_init() isn't void?

Em Mon, 14 Mar 2016 10:05:01 -0300 Mauro Carvalho Chehab escreveu: > Em Mon, 14 Mar 2016 08:52:51 -0300 > Mauro Carvalho Chehab escreveu: > > > Em Mon, 14 Mar 2016 13:43:33 +0200 > > Sakari Ailus escreveu: > > > > > Hi

Re: [PATCH v3 2/9] [media] tvp5150: add userspace subdev API

Em Mon, 14 Mar 2016 16:23:30 +0100 Lucas Stach escreveu: > From: Philipp Zabel > > This patch adds userspace V4L2 subdevice API support. > > Signed-off-by: Philipp Zabel > Signed-off-by: Lucas Stach

Re: [PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

Hi, On 14-03-16 16:02, Antonio Ospite wrote: On Thu, 10 Mar 2016 15:54:37 +0100 Hans de Goede wrote: Hi, On 09-03-16 17:03, Antonio Ospite wrote: When calling VIDIOC_REQBUFS v4l2-compliance fails with this message: fail: v4l2-test-buffers.cpp(476): q.reqbufs(node,

[PATCH v3 4/9] [media] tvp5150: fix standard autodetection

From: Philipp Zabel Make sure to not overwrite decoder->norm when setting the standard in hardware, but only when instructed by V4L2 API calls. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach ---

[PATCH v3 3/9] [media] tvp5150: determine BT.656 or YUV 4:2:2 mode from device tree

From: Philipp Zabel By looking at the endpoint flags, it can be determined whether the link should be of V4L2_MBUS_PARALLEL or V4L2_MBUS_BT656 type. Disable the dedicated HSYNC/VSYNC outputs in BT.656 mode. For devices that are not instantiated through DT the current

[PATCH v3 1/9] [media] tvp5150: convert register access to regmap

From: Philipp Zabel Regmap provides built in debugging, caching and provides dedicated accessors for bit manipulations in registers, which make the following changes a lot simpler. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach

[PATCH v3 2/9] [media] tvp5150: add userspace subdev API

From: Philipp Zabel This patch adds userspace V4L2 subdevice API support. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach --- drivers/media/i2c/tvp5150.c | 282 +++- 1 file

[PATCH v3 8/9] [media] tvp5150: Add sync lock interrupt handling

From: Philipp Zabel This patch adds an optional interrupt handler to handle the sync lock interrupt and sync lock status. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach --- drivers/media/i2c/tvp5150.c | 103

[PATCH v3 9/9] [media] tvp5150: disable output while signal not locked

From: Philipp Zabel To avoid short frames on stream start, keep output pins at high impedance while we are not properly locked onto the input signal. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach ---

[PATCH v3 5/9] [media] tvp5150: split reset/enable routine

From: Philipp Zabel To trigger standard autodetection only the reset part of the routine is necessary. Split this out to make it callable on its own. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach ---

[PATCH v3 6/9] [media] tvp5150: trigger autodetection on subdev open to reset cropping

From: Philipp Zabel If cropping isn't set explicitly by userspace, reset it to the maximum possible rectangle in subdevice open if a standard change is detected. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach ---

[PATCH v3 7/9] [media] tvp5150: remove pin configuration from initialization tables

From: Philipp Zabel To allow optional interrupt support, we want to configure the pin settings dynamically. Move those register accesses out of the static initialization tables. Signed-off-by: Philipp Zabel Signed-off-by: Lucas Stach

Re: [PATCH v3 06/22] media: Media Controller enable/disable source handler API

On 03/13/2016 02:11 PM, Sakari Ailus wrote: > Hi Shuah, > > On Thu, Mar 10, 2016 at 07:29:59AM -0700, Shuah Khan wrote: >> On 03/10/2016 12:35 AM, Sakari Ailus wrote: >>> Hi Shuah, >>> >>> On Thu, Feb 11, 2016 at 04:41:22PM -0700, Shuah Khan wrote: Add new fields to struct media_device to

Re: [PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

On Thu, 10 Mar 2016 15:54:37 +0100 Hans de Goede wrote: > Hi, > > On 09-03-16 17:03, Antonio Ospite wrote: > > When calling VIDIOC_REQBUFS v4l2-compliance fails with this message: > > > >fail: v4l2-test-buffers.cpp(476): q.reqbufs(node, 1) > >test

Re: i.mx6 camera interface (CSI) and mainline kernel

Hi Steve and all, I am busy porting to Steve's subdev version a driver that I had written in intdev style, for the freescale imx6 linux version. And I have a problem : My previous driver had the following ioctl function, which used the V4L2_FRMIVAL_TYPE_CONTINUOUS type answer. static int

Re: [PATCH 2/2] v4l2-ioctl: improve cropcap handling

Reviewed-by: Niklas Söderlund On 2016-02-29 11:16:40 +0100, Hans Verkuil wrote: > From: Hans Verkuil > > If cropcap is implemented, then call it first to fill in the pixel > aspect ratio. Don't return if cropcap returns ENOTTY or

Re: [PATCH 1/2] v4l2-ioctl: simplify code

On 2016-03-14 13:52:17 +0100, Hans Verkuil wrote: > On 03/14/2016 01:42 PM, Niklas Söderlund wrote: > > Hi Hans, > > > > On 2016-02-29 11:16:39 +0100, Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> Instead of a big if at the beginning, just check if g_selection ==

Re: Any reason why media_entity_pads_init() isn't void?

Em Mon, 14 Mar 2016 08:52:51 -0300 Mauro Carvalho Chehab escreveu: > Em Mon, 14 Mar 2016 13:43:33 +0200 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > On Mon, Mar 14, 2016 at 08:27:38AM -0300, Mauro Carvalho Chehab wrote: > > > Em Mon, 14 Mar

Re: [PATCH 1/2] v4l2-ioctl: simplify code

On 03/14/2016 01:42 PM, Niklas Söderlund wrote: > Hi Hans, > > On 2016-02-29 11:16:39 +0100, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Instead of a big if at the beginning, just check if g_selection == NULL >> and call the cropcap op immediately and return the

Re: [PATCH 1/2] v4l2-ioctl: simplify code

Hi Hans, On 2016-02-29 11:16:39 +0100, Hans Verkuil wrote: > From: Hans Verkuil > > Instead of a big if at the beginning, just check if g_selection == NULL > and call the cropcap op immediately and return the result. > > No functional changes in this patch. > >

Re: [PATCH] media: v4l2-compat-ioctl32: fix missing reserved field copy in put_v4l2_create32

On Mon, 2016-03-14 at 08:28 -0300, Mauro Carvalho Chehab wrote: > Em Mon, 14 Mar 2016 18:41:46 +0800 > Tiffany Lin escreveu: > > > Change-Id: Idac449fae5059a3ce255340e6da491f8bd83af7a > > We don't need change-id at the Kernel, but we do need a proper patch >

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

Hi Mauro, On Mon, Mar 14, 2016 at 08:46:33AM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 14 Mar 2016 12:52:54 +0200 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > On Mon, Mar 14, 2016 at 07:13:58AM -0300, Mauro Carvalho Chehab wrote: > > > Em Mon, 14 Mar 2016 09:22:37

Re: Any reason why media_entity_pads_init() isn't void?

Em Mon, 14 Mar 2016 13:43:33 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Mon, Mar 14, 2016 at 08:27:38AM -0300, Mauro Carvalho Chehab wrote: > > Em Mon, 14 Mar 2016 12:36:44 +0200 > > Sakari Ailus escreveu: > > > > > Hi Hans, > > > > > > On

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

Em Mon, 14 Mar 2016 12:52:54 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Mon, Mar 14, 2016 at 07:13:58AM -0300, Mauro Carvalho Chehab wrote: > > Em Mon, 14 Mar 2016 09:22:37 +0200 > > Sakari Ailus escreveu: > > > > > Hi Shuah, > > > > > > On

Re: Any reason why media_entity_pads_init() isn't void?

Hi Mauro, On Mon, Mar 14, 2016 at 08:27:38AM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 14 Mar 2016 12:36:44 +0200 > Sakari Ailus escreveu: > > > Hi Hans, > > > > On Mon, Mar 14, 2016 at 09:25:51AM +0100, Hans Verkuil wrote: > > > I was fixing a sparse warning in

DVBv5 Tools: VDR format is broken (recommended patch)

Hi Mauro, sorry for bumping, but could you take a look at my libdvbv5/dvb-vdr-format.c patch? Please give me a short feedback / thanks a lot. --Markus-- Am 10.03.2016 um 15:07 schrieb Markus Heiser : > Hi (Mauro), > > below you will find my recommended patch for

Re: [PATCH] media: v4l2-compat-ioctl32: fix missing reserved field copy in put_v4l2_create32

Em Mon, 14 Mar 2016 18:41:46 +0800 Tiffany Lin escreveu: > Change-Id: Idac449fae5059a3ce255340e6da491f8bd83af7a We don't need change-id at the Kernel, but we do need a proper patch description. Regards, Mauro > --- > drivers/media/v4l2-core/v4l2-compat-ioctl32.c |

Re: Any reason why media_entity_pads_init() isn't void?

Em Mon, 14 Mar 2016 12:36:44 +0200 Sakari Ailus escreveu: > Hi Hans, > > On Mon, Mar 14, 2016 at 09:25:51AM +0100, Hans Verkuil wrote: > > I was fixing a sparse warning in media_entity_pads_init() and I noticed > > that that function always returns 0. Any reason why this

Re: Question regarding internal webcams of tablet devices

Hi, do you know where I can get the source code of these drivers for Baytrail, Anniedale or Cherrytrail? I am not familiar with the Android kernel. I checked git://git.code.sf.net/p/android-x86/kernel but could not find any PCI CSI2 host controller driver. At

[PATCH v2] media: v4l2-compat-ioctl32: fix missing reserved field copy in put_v4l2_create32

In v4l2-compliance utility, test VIDIOC_CREATE_BUFS will check whether reserved filed of v4l2_create_buffers filled with zero Reserved field is filled with zero in v4l_create_bufs. This patch copy reserved field of v4l2_create_buffer from kernel space to user space Signed-off-by: Tiffany Lin

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

Hi Mauro, On Mon, Mar 14, 2016 at 07:13:58AM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 14 Mar 2016 09:22:37 +0200 > Sakari Ailus escreveu: > > > Hi Shuah, > > > > On Sat, Mar 12, 2016 at 06:48:09PM -0700, Shuah Khan wrote: > > > Add GFP flags to

[PATCH] media: v4l2-compat-ioctl32: fix missing reserved field copy in put_v4l2_create32

Change-Id: Idac449fae5059a3ce255340e6da491f8bd83af7a --- drivers/media/v4l2-core/v4l2-compat-ioctl32.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-compat-ioctl32.c b/drivers/media/v4l2-core/v4l2-compat-ioctl32.c index f38c076..109f687

Re: Any reason why media_entity_pads_init() isn't void?

Hi Hans, On Mon, Mar 14, 2016 at 09:25:51AM +0100, Hans Verkuil wrote: > I was fixing a sparse warning in media_entity_pads_init() and I noticed > that that function always returns 0. Any reason why this can't be changed > to a void function? I was thinking of the same function but I had a

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

On Mon, 14 Mar 2016 11:13:58 +0100, Mauro Carvalho Chehab wrote: > > Em Mon, 14 Mar 2016 09:22:37 +0200 > Sakari Ailus escreveu: > > > Hi Shuah, > > > > On Sat, Mar 12, 2016 at 06:48:09PM -0700, Shuah Khan wrote: > > > Add GFP flags to media_create_pad_link(),

Re: Any reason why media_entity_pads_init() isn't void?

Em Mon, 14 Mar 2016 09:25:51 +0100 Hans Verkuil escreveu: > I was fixing a sparse warning in media_entity_pads_init() and I noticed that > that > function always returns 0. Any reason why this can't be changed to a void > function? > > That return value is checked a

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

Em Mon, 14 Mar 2016 09:22:37 +0200 Sakari Ailus escreveu: > Hi Shuah, > > On Sat, Mar 12, 2016 at 06:48:09PM -0700, Shuah Khan wrote: > > Add GFP flags to media_create_pad_link(), media_create_intf_link(), > > media_devnode_create(), and media_add_link() that could get

Re: FW: [PATCH v5 0/8] Add MT8173 Video Encoder Driver and VPU Driver

On 03/14/2016 10:04 AM, tiffany lin wrote: > On Mon, 2016-03-14 at 08:21 +0100, Hans Verkuil wrote: >> On 03/14/2016 08:12 AM, tiffany lin wrote: >>> Hi Hans, >>> >>> After change to use "v4l-utils.git master branch", "V4l2-compliance >>> -d /dev/video1" fail on "fail: v4l2-test-buffers.cpp(555):

Re: FW: [PATCH v5 0/8] Add MT8173 Video Encoder Driver and VPU Driver

On Mon, 2016-03-14 at 08:21 +0100, Hans Verkuil wrote: > On 03/14/2016 08:12 AM, tiffany lin wrote: > > Hi Hans, > > > > After change to use "v4l-utils.git master branch", "V4l2-compliance > > -d /dev/video1" fail on "fail: v4l2-test-buffers.cpp(555): > > check_0(crbufs.reserved,

Any reason why media_entity_pads_init() isn't void?

I was fixing a sparse warning in media_entity_pads_init() and I noticed that that function always returns 0. Any reason why this can't be changed to a void function? That return value is checked a zillion times in the media code. By making it void it should simplify code all over. See e.g.

[PATCH] media-entity: fix sparse warning in media_entity_pads_init()

Fix this sparse warning: drivers/media/media-entity.c:212:5: warning: context imbalance in 'media_entity_pads_init' - different lock contexts for basic block Signed-off-by: Hans Verkuil diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c index

Re: [PATCH] Add tw5864 driver

Hi Andrey, See below for a quick review of the code. I agree with Greg's comment why this is added to staging instead of drivers/media/pci? When you post the v2 patch, can you add the output of 'v4l2-compliance -s' to the cover letter? Please use the latest v4l2-compliance version from the

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

Hi Shuah, On Sat, Mar 12, 2016 at 06:48:09PM -0700, Shuah Khan wrote: > Add GFP flags to media_create_pad_link(), media_create_intf_link(), > media_devnode_create(), and media_add_link() that could get called > in atomic context to allow callers to pass in the right flags for > memory allocation.

Re: FW: [PATCH v5 0/8] Add MT8173 Video Encoder Driver and VPU Driver

On 03/14/2016 08:12 AM, tiffany lin wrote: > Hi Hans, > > After change to use "v4l-utils.git master branch", "V4l2-compliance > -d /dev/video1" fail on "fail: v4l2-test-buffers.cpp(555): > check_0(crbufs.reserved, sizeof(crbufs.reserved))". > > Check the source code and found > > memset(,

Re: FW: [PATCH v5 0/8] Add MT8173 Video Encoder Driver and VPU Driver

On Mon, 2016-03-14 at 14:48 +0800, PoChun Lin (林柏君) wrote: > > -Original Message- > From: tiffany lin [mailto:tiffany@mediatek.com] > Sent: Wednesday, February 24, 2016 6:53 PM > To: Hans Verkuil > Cc: Hans Verkuil; daniel.thomp...@linaro.org; Rob Herring; Mauro Carvalho > Chehab;