On Fri, 2018-06-15 at 10:43 +0200, Hans Verkuil wrote: > On 14/06/18 17:34, Ezequiel Garcia wrote: > > Implementing job_abort() does not make sense on some drivers. > > This is not a problem, as the abort is not required to > > wait for the job to finish. Quite the opposite, drivers > > are encouraged not to wait. > > > > Demote v4l2_m2m_ops.job_abort from required to optional, and > > clean all drivers with dummy or wrong implementations. > > Can you split off the rcar_jpu.c and g2d.c changes into separate > patches? > The others are trivial, but those two need a more precise commit log > and > I would like to have Acks of the driver maintainers before merging. > Yes, that makes sense. > I'm going to take the first and third patches of this series, so you > only have to post a v2 for this patch. > OK. Thanks, Eze