Re: [PATCH] media: cedrus: Remove global IRQ spin lock from the driver

2018-11-29 Thread Paul Kocialkowski
Hi, On Fri, 2018-11-16 at 11:47 +0100, Maxime Ripard wrote: > On Thu, Nov 15, 2018 at 03:39:55PM +0100, Paul Kocialkowski wrote: > > We initially introduced a spin lock to ensure that the VPU registers > > are not accessed concurrently between our setup function and IRQ > > handler. Because the V4

Re: [PATCH] media: cedrus: Remove global IRQ spin lock from the driver

2018-11-16 Thread Maxime Ripard
On Thu, Nov 15, 2018 at 03:39:55PM +0100, Paul Kocialkowski wrote: > We initially introduced a spin lock to ensure that the VPU registers > are not accessed concurrently between our setup function and IRQ > handler. Because the V4L2 M2M API only allows one job to run at a time > and our jobs are co

[PATCH] media: cedrus: Remove global IRQ spin lock from the driver

2018-11-15 Thread Paul Kocialkowski
We initially introduced a spin lock to ensure that the VPU registers are not accessed concurrently between our setup function and IRQ handler. Because the V4L2 M2M API only allows one job to run at a time and our jobs are completed following the IRQ handler, there is actually no chance of an interr