Re: [PATCH 1/2] media/dvb-core: fix inverted check

2016-03-20 Thread Olli Salonen
Hi Max, Already in the tree: http://git.linuxtv.org/media_tree.git/commit/drivers/media/dvb-core?id=711f3fba6ffd3914fd1b5ed9faf8d22bab6f2203 Cheers, -olli On 18 March 2016 at 23:31, Max Kellermann wrote: > Breakage caused by commit f50d51661a > > Signed-off-by: Max Kellermann > --- > drivers/

Re: [PATCH] dma-buf, drm, ion: Propagate error code from dma_buf_start_cpu_access()

2016-03-20 Thread Sumit Semwal
On 19 March 2016 at 15:39, Daniel Vetter wrote: > On Fri, Mar 18, 2016 at 08:02:39PM +, Chris Wilson wrote: >> Drivers, especially i915.ko, can fail during the initial migration of a >> dma-buf for CPU access. However, the error code from the driver was not >> being propagated back to ioctl an

cron job: media_tree daily build: OK

2016-03-20 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 Mar 21 04:00:22 CET 2016 git branch: test git hash: b39950960d2b890c21465c69c7c0e4ff6253c6b5 gcc versio

Re: [PATCHv13 01/17] dts: exynos4*: add HDMI CEC pin definition to pinctrl

2016-03-20 Thread Krzysztof Kozlowski
On 19.03.2016 17:15, Hans Verkuil wrote: > On 03/19/2016 03:50 AM, Krzysztof Kozlowski wrote: >> On Fri, Mar 18, 2016 at 03:07:00PM +0100, Hans Verkuil wrote: >>> From: Kamil Debski >>> >>> Add pinctrl nodes for the HDMI CEC device to the Exynos4210 and >>> Exynos4x12 SoCs. These are required by t

Re: [PATCH 13/15] mediactl: Add media device ioctl API

2016-03-20 Thread Sakari Ailus
Hi Jacek, On Thu, Feb 18, 2016 at 02:14:40PM +0100, Jacek Anaszewski wrote: > Hi Sakari, > > On 02/18/2016 01:09 PM, Sakari Ailus wrote: > >Hi Jacek, > > > >On Mon, Feb 15, 2016 at 02:06:06PM +0100, Jacek Anaszewski wrote: > >>Hi Sakari, > >> > >>Thanks for the review. > >> > >>On 02/15/2016 01:4

Re: [PATCH 01/15] mediactl: Introduce v4l2_subdev structure

2016-03-20 Thread Sakari Ailus
Hi Jacek, On Thu, Feb 18, 2016 at 03:15:32PM +0100, Jacek Anaszewski wrote: > Hi Sakari, > > Thanks for the review. > > On 02/12/2016 01:42 PM, Sakari Ailus wrote: > >Hi Jacek, > > > >Thanks for continuing this work! And my apologies for reviewing only > >now... please see the comments below. >

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-20 Thread Shuah Khan
On 03/16/2016 08:25 AM, Mauro Carvalho Chehab wrote: > Em Wed, 16 Mar 2016 08:05:15 -0600 > Shuah Khan escreveu: > >> On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: >>> Now that the media_device can be used by multiple drivers, >>> via devres, we need to be sure that it will be dropped only

Re: [PATCH] [media] media: rename media unregister function

2016-03-20 Thread Shuah Khan
On 03/18/2016 08:12 AM, Javier Martinez Canillas wrote: > Hello Shuah, > > On 03/18/2016 11:01 AM, Shuah Khan wrote: >> On 03/18/2016 07:05 AM, Mauro Carvalho Chehab wrote: >>> Now that media_device_unregister() also does a cleanup, rename it >>> to media_device_unregister_cleanup(). >>> >>> Signe

[v4l-utils PATCH 1/1] libmediactl: Drop length argument from

2016-03-20 Thread Sakari Ailus
Recently it was decided that the API dealing with string operations would be better to just receive a nul-terminated string rather than a string the length of which is defined. This change was implemented for v4l2_subdev_string_to_pixelcode() and v4l2_subdev_string_to_field() functions by patch "v4

Re: cron job: media_tree daily build: ABI WARNING

2016-03-20 Thread Hans Verkuil
On 03/20/2016 05:01 AM, Hans Verkuil wrote: > ABI WARNING: change for arm-at91 > ABI WARNING: change for arm-davinci > ABI WARNING: change for arm-exynos > ABI WARNING: change for arm-mx > ABI WARNING: change for arm-omap > ABI WARNING: change for arm-omap1 > ABI WARNING: change for arm-pxa > ABI W

Re: [PATCH] media: rc: remove unneeded mutex in rc_register_device

2016-03-20 Thread Heiner Kallweit
Am 16.03.2016 um 20:33 schrieb Vladimir Zapolskiy: > On 16.03.2016 20:53, Heiner Kallweit wrote: >> Access to dev->initialized is atomic, therefore we don't have to >> protect it with a mutex. > > Mutexes are used to split the code to mutually exclusive execution blocks, > so not arguing about the

Re: [PATCH 5/5] [media] media-device: make media_device_cleanup() static

2016-03-20 Thread Mauro Carvalho Chehab
Em Wed, 16 Mar 2016 11:03:38 -0300 Javier Martinez Canillas escreveu: > Hello Mauro, > > The patch looks mostly good to me, I just have a question below: > > On Wed, Mar 16, 2016 at 9:04 AM, Mauro Carvalho Chehab > wrote: > > [snip] > > > > > -void media_device_cleanup(struct media_device *m

[PATCHv13 08/17] cec: add compat32 ioctl support

2016-03-20 Thread Hans Verkuil
The CEC ioctls didn't have compat32 support, so they returned -ENOTTY when used in a 32 bit application on a 64 bit kernel. Since all the CEC ioctls are 32-bit compatible adding support for this API is trivial. Signed-off-by: Hans Verkuil --- fs/compat_ioctl.c | 17 + 1 file cha