[PATCH] [media] coda: add missing header dependencies

2016-09-23 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/media/platform/coda/coda-h264.c:21:5: warning: no previous prototype for 'coda_h264_padding' [-Wmissing-prototypes] In fact, this function is declared in drivers/media/platform/coda/coda.h, so this patch adds missing header dependencies. Si

[PATCH] dma-buf/sw_sync: mark sync_timeline_create() static

2016-09-23 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/dma-buf/sw_sync.c:87:23: warning: no previous prototype for 'sync_timeline_create' [-Wmissing-prototypes] In fact, this function is only used in the file in which it is declared and don't need a declaration, but can be made static. So this p

cron job: media_tree daily build: ERRORS

2016-09-23 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: Sat Sep 24 04:00:17 CEST 2016 git branch: test git hash: e3ea5e94489bc8c711d422dfa311cfa310553a1b gcc versi

Re: [PATCH] [media/input] rc: report rc protocol type to userspace through input

2016-09-23 Thread Dmitry Torokhov
On Thu, Sep 22, 2016 at 11:57:13AM -0300, Mauro Carvalho Chehab wrote: > Em Wed, 21 Sep 2016 10:54:21 +0100 > Sean Young escreveu: > > > We might want to know what protocol a remote uses when we do not know. With > > this patch and another patch for v4l-utils (follows), you can do that with: > >

[PATCH] [media] VPU: mediatek: Fix return value in case of error

2016-09-23 Thread Christophe JAILLET
If 'dma_alloc_coherent()' returns NULL, 'vpu_alloc_ext_mem()' will return 0 which means success. Return -ENOMEM instead. Signed-off-by: Christophe JAILLET --- drivers/media/platform/mtk-vpu/mtk_vpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/mtk-v

Re: [PATCH v2 1/2] media: adv7604: fix bindings inconsistency for default-input

2016-09-23 Thread Rob Herring
On Thu, Sep 22, 2016 at 03:18:59PM +0200, Ulrich Hecht wrote: > The text states that default-input is an endpoint property, but in the > example it is a device property. > > The default input is a property of the chip, not of a particular port, so > the example makes more sense. > > Signed-off-by

[PATCH 2/2] media: mt9m111: add regulator support

2016-09-23 Thread Robert Jarzmik
In the soc_camera removal, the board specific power callback was dropped. This at least will remove the power optimization from ezx and em-x270 pxa based boards. As to recreate the same level of functionality, make the mt9m111 have a regulator providing it its power, so that board designers can pl

[PATCH 1/2] media: platform: pxa_camera: add missing sensor power on

2016-09-23 Thread Robert Jarzmik
During sensors binding, there is a window where the sensor is switched off, while there is a call it to set a new format, which can end up in an access to the sensor, especially an I2C based sensor. Remove this window by activating the sensor. Signed-off-by: Robert Jarzmik --- drivers/media/pla

Re: [PATCH] pxa_camera: merge soc_mediabus.c into pxa_camera.c

2016-09-23 Thread Robert Jarzmik
Hans Verkuil writes: >> Hi Hans, >> >> I wonder why you chose to copy-paste this code instead of adding in the >> Kconfig >> a "depends on !SOC_CAMERA". Any specific reason ? As this will have to be >> dealt >> with later anyway as you pointed out earlier, this format translation I >> mean, I

Re: [Intel-gfx] [PATCH 11/11] dma-buf: Do a fast lockless check for poll with timeout=0

2016-09-23 Thread Christian König
Am 23.09.2016 um 17:20 schrieb Chris Wilson: On Fri, Sep 23, 2016 at 03:50:44PM +0200, Daniel Vetter wrote: On Mon, Aug 29, 2016 at 08:08:34AM +0100, Chris Wilson wrote: Currently we install a callback for performing poll on a dma-buf, irrespective of the timeout. This involves taking a spinloc

Re: [Intel-gfx] [PATCH 11/11] dma-buf: Do a fast lockless check for poll with timeout=0

2016-09-23 Thread Chris Wilson
On Fri, Sep 23, 2016 at 03:50:44PM +0200, Daniel Vetter wrote: > On Mon, Aug 29, 2016 at 08:08:34AM +0100, Chris Wilson wrote: > > Currently we install a callback for performing poll on a dma-buf, > > irrespective of the timeout. This involves taking a spinlock, as well as > > unnecessary work, and

Re: [Intel-gfx] [PATCH 11/11] dma-buf: Do a fast lockless check for poll with timeout=0

2016-09-23 Thread Chris Wilson
On Fri, Sep 23, 2016 at 03:50:44PM +0200, Daniel Vetter wrote: > On Mon, Aug 29, 2016 at 08:08:34AM +0100, Chris Wilson wrote: > > Currently we install a callback for performing poll on a dma-buf, > > irrespective of the timeout. This involves taking a spinlock, as well as > > unnecessary work, and

Re: [Intel-gfx] [PATCH 11/11] dma-buf: Do a fast lockless check for poll with timeout=0

2016-09-23 Thread Chris Wilson
On Fri, Sep 23, 2016 at 03:50:44PM +0200, Daniel Vetter wrote: > On Mon, Aug 29, 2016 at 08:08:34AM +0100, Chris Wilson wrote: > > Currently we install a callback for performing poll on a dma-buf, > > irrespective of the timeout. This involves taking a spinlock, as well as > > unnecessary work, and

Re: [Linaro-mm-sig] [PATCH 10/11] dma-buf: Use seqlock to close RCU race in test_signaled_single

2016-09-23 Thread Chris Wilson
On Fri, Sep 23, 2016 at 03:49:26PM +0200, Daniel Vetter wrote: > On Mon, Aug 29, 2016 at 08:08:33AM +0100, Chris Wilson wrote: > > With the seqlock now extended to cover the lookup of the fence and its > > testing, we can perform that testing solely under the seqlock guard and > > avoid the effecti

Re: [Intel-gfx] [PATCH 11/11] dma-buf: Do a fast lockless check for poll with timeout=0

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:34AM +0100, Chris Wilson wrote: > Currently we install a callback for performing poll on a dma-buf, > irrespective of the timeout. This involves taking a spinlock, as well as > unnecessary work, and greatly reduces scaling of poll(.timeout=0) across > multiple threads.

Re: [Linaro-mm-sig] [PATCH 10/11] dma-buf: Use seqlock to close RCU race in test_signaled_single

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:33AM +0100, Chris Wilson wrote: > With the seqlock now extended to cover the lookup of the fence and its > testing, we can perform that testing solely under the seqlock guard and > avoid the effective locking and serialisation of acquiring a reference to > the request.

Re: [PATCH 09/11] dma-buf: Restart reservation_object_test_signaled_rcu() after writes

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:32AM +0100, Chris Wilson wrote: > In order to be completely generic, we have to double check the read > seqlock after acquiring a reference to the fence. If the driver is > allocating fences from a SLAB_DESTROY_BY_RCU, or similar freelist, then > within an RCU grace pe

Re: [PATCH 06/11] dma-buf: Introduce fence_get_rcu_safe()

2016-09-23 Thread Markus Heiser
Am 23.09.2016 um 14:59 schrieb Daniel Vetter : >> >> /** >> - * fence_put - decreases refcount of the fence >> - * @fence: [in]fence to reduce refcount of >> + * fence_get_rcu_safe - acquire a reference to an RCU tracked fence >> + * @fence: [in]pointer to fence to increase refcount o

Re: [PATCH 08/11] dma-buf: Restart reservation_object_wait_timeout_rcu() after writes

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:31AM +0100, Chris Wilson wrote: > In order to be completely generic, we have to double check the read > seqlock after acquiring a reference to the fence. If the driver is > allocating fences from a SLAB_DESTROY_BY_RCU, or similar freelist, then > within an RCU grace pe

Re: [PATCH 07/11] dma-buf: Restart reservation_object_get_fences_rcu() after writes

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:30AM +0100, Chris Wilson wrote: > In order to be completely generic, we have to double check the read > seqlock after acquiring a reference to the fence. If the driver is > allocating fences from a SLAB_DESTROY_BY_RCU, or similar freelist, then > within an RCU grace pe

Re: [PATCH 06/11] dma-buf: Introduce fence_get_rcu_safe()

2016-09-23 Thread Daniel Vetter
On Mon, Aug 29, 2016 at 08:08:29AM +0100, Chris Wilson wrote: > This variant of fence_get_rcu() takes an RCU protected pointer to a > fence and carefully returns a reference to the fence ensuring that it is > not reallocated as it does. This is required when mixing fences and > SLAB_DESTROY_BY_RCU

Re: [PATCH v1.1 5/5] smiapp: Implement support for autosuspend

2016-09-23 Thread Sakari Ailus
Sebastian Reichel wrote: Hi, On Tue, Sep 20, 2016 at 03:29:58PM +0300, Sakari Ailus wrote: Delay suspending the device by 1000 ms by default. Signed-off-by: Sakari Ailus --- since v1: - Increment usage count before register write using pm_runtime_get_noresume(), and decrement it before r

Re: g_webcam Isoch high bandwidth transfer

2016-09-23 Thread Felipe Balbi
Hi, yfw writes: >> Here's one that actually compiles, sorry about that. > > No worries, I was sleeping ;-) > > I will test it out early next week. Thanks. meanwhile, how about some instructions on how to test this out myself? How are you using g_webcam and what

Re: g_webcam Isoch high bandwidth transfer

2016-09-23 Thread yfw
Hi Felipe, On 2016/9/23 15:49, Felipe Balbi wrote: Hi, Bin Liu writes: +Fengwei Yin per his request. On Thu, Sep 22, 2016 at 10:48:40PM +0300, Felipe Balbi wrote: Hi, Bin Liu writes: [...] Here's one that actually compiles, sorry about that. No worries, I was sleeping ;-) I will t

Re: g_webcam Isoch high bandwidth transfer

2016-09-23 Thread Felipe Balbi
Hi, Bin Liu writes: > +Fengwei Yin per his request. > > On Thu, Sep 22, 2016 at 10:48:40PM +0300, Felipe Balbi wrote: >> >> Hi, >> >> Bin Liu writes: >> >> [...] >> >> >> Here's one that actually compiles, sorry about that. >> > >> > No worries, I was sleeping ;-) >> > >> > I will test it o