Re: [RFC 1/1] v4l: return POLLERR on V4L2 sub-devices if no events are subscribed

2013-09-30 Thread Sakari Ailus
Hi Hans, Thanks for the review. Hans Verkuil wrote: Hi Teemu, Sakari, I've been thinking about this change and I think it is a reasonable change. I had some doubts earlier, but after thinking it through I agree with this. But it needs a bit more work, see below. On 09/17/2013 08:07 PM, Sakar

Re: [RFC 1/1] v4l: return POLLERR on V4L2 sub-devices if no events are subscribed

2013-09-30 Thread Hans Verkuil
Hi Teemu, Sakari, I've been thinking about this change and I think it is a reasonable change. I had some doubts earlier, but after thinking it through I agree with this. But it needs a bit more work, see below. On 09/17/2013 08:07 PM, Sakari Ailus wrote: > From: Teemu Tuominen > > Add check an

Re: [RFC 1/1] v4l: return POLLERR on V4L2 sub-devices if no events are subscribed

2013-09-17 Thread Laurent Pinchart
Hi Sakari and Teemu, Thank you for the patch. On Tuesday 17 September 2013 21:07:19 Sakari Ailus wrote: > From: Teemu Tuominen > > Add check and return POLLERR from subdev_poll() in case of no events > subscribed and wakeup once the last event subscription is removed. > > This change is essent

[RFC 1/1] v4l: return POLLERR on V4L2 sub-devices if no events are subscribed

2013-09-17 Thread Sakari Ailus
From: Teemu Tuominen Add check and return POLLERR from subdev_poll() in case of no events subscribed and wakeup once the last event subscription is removed. This change is essentially done to add possibility to wakeup polling with concurrent unsubscribe. Signed-off-by: Teemu Tuominen Move the