Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-02 Thread Mauro Carvalho Chehab
Em Tue, 02 Sep 2014 03:16:00 + (GMT) Changbing Xiong cb.xi...@samsung.com escreveu: Well, we may start returning -ENODEV when such event happens. At the frontend, we could use fe-exit = DVB_FE_DEVICE_REMOVED to signalize it. I don't think that the demod frontend has something

Re: Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-02 Thread Changbing Xiong
Actually, poll() may return an error as well (from poll() manpage): RETURN VALUE On success, a positive number is returned; this is the number of struc‐ tures which have nonzero revents fields (in other words, those descrip‐ tors with events or errors reported). A

Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-02 Thread Mauro Carvalho Chehab
Em Tue, 02 Sep 2014 06:42:42 + (GMT) Changbing Xiong cb.xi...@samsung.com escreveu: Actually, poll() may return an error as well (from poll() manpage): RETURN VALUE On success, a positive number is returned; this is the number of struc‐ tures which have nonzero

Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-01 Thread Antti Palosaari
Moikka Changbing and thanks to working that. I reviewed the first patch and tested all these patches. It does not deadlock USB device anymore because of patch #1 so it is improvement. However, what I expect that patch, it should force device unregister but when I use tzap and unplug running

Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-01 Thread Mauro Carvalho Chehab
Em Tue, 02 Sep 2014 02:58:50 +0300 Antti Palosaari cr...@iki.fi escreveu: Moikka Changbing and thanks to working that. I reviewed the first patch and tested all these patches. It does not deadlock USB device anymore because of patch #1 so it is improvement. However, what I expect that

Re: Re: [PATCH 3/3] media: check status of dmxdev-exit in poll functions of demuxdvr

2014-09-01 Thread Changbing Xiong
Well, we may start returning -ENODEV when such event happens. At the frontend, we could use fe-exit = DVB_FE_DEVICE_REMOVED to signalize it. I don't think that the demod frontend has something similar. Yet, it should be up to the userspace application to properly handle the error codes