Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-19 Thread Laurent Pinchart
Hi Hans, On Thursday, 19 October 2017 10:23:06 EEST Hans Yang wrote: > On 2017年10月18日 16:52, Guennadi Liakhovetski wrote: > > On Mon, 16 Oct 2017, Laurent Pinchart wrote: > >> > >> On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > >>> For bulk-based devices, when disabling the video

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-19 Thread Laurent Pinchart
Hi Hans, On Thursday, 19 October 2017 10:23:06 EEST Hans Yang wrote: > On 2017年10月18日 16:52, Guennadi Liakhovetski wrote: > > On Mon, 16 Oct 2017, Laurent Pinchart wrote: > >> > >> On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > >>> For bulk-based devices, when disabling the video

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-19 Thread Hans Yang
Hi Laurent and Guennadi, On 2017年10月18日 16:52, Guennadi Liakhovetski wrote: Hi Laurent, On Mon, 16 Oct 2017, Laurent Pinchart wrote: Hi Hans, (CC'ing Guennadi Liakhovetski) Thank you for the patch. On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: For bulk-based devices, when

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-19 Thread Hans Yang
Hi Laurent and Guennadi, On 2017年10月18日 16:52, Guennadi Liakhovetski wrote: Hi Laurent, On Mon, 16 Oct 2017, Laurent Pinchart wrote: Hi Hans, (CC'ing Guennadi Liakhovetski) Thank you for the patch. On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: For bulk-based devices, when

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-18 Thread Guennadi Liakhovetski
Hi Laurent, On Mon, 16 Oct 2017, Laurent Pinchart wrote: > Hi Hans, > > (CC'ing Guennadi Liakhovetski) > > Thank you for the patch. > > On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > > For bulk-based devices, when disabling the video stream, > > in addition to issue

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-18 Thread Guennadi Liakhovetski
Hi Laurent, On Mon, 16 Oct 2017, Laurent Pinchart wrote: > Hi Hans, > > (CC'ing Guennadi Liakhovetski) > > Thank you for the patch. > > On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > > For bulk-based devices, when disabling the video stream, > > in addition to issue

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-16 Thread Laurent Pinchart
Hi Hans, (CC'ing Guennadi Liakhovetski) Thank you for the patch. On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > For bulk-based devices, when disabling the video stream, > in addition to issue CLEAR_FEATURE(HALT), it is better to set > alternate setting 0 as well or the sequnce

Re: [PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-10-16 Thread Laurent Pinchart
Hi Hans, (CC'ing Guennadi Liakhovetski) Thank you for the patch. On Friday, 15 September 2017 09:27:51 EEST Hans Yang wrote: > For bulk-based devices, when disabling the video stream, > in addition to issue CLEAR_FEATURE(HALT), it is better to set > alternate setting 0 as well or the sequnce

[PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-09-15 Thread Hans Yang
For bulk-based devices, when disabling the video stream, in addition to issue CLEAR_FEATURE(HALT), it is better to set alternate setting 0 as well or the sequnce number in host side will probably not reset to zero. Then in next time video stream start, the device will expect host starts packet

[PATCH resend] [media] uvcvideo: zero seq number when disabling stream

2017-09-15 Thread Hans Yang
For bulk-based devices, when disabling the video stream, in addition to issue CLEAR_FEATURE(HALT), it is better to set alternate setting 0 as well or the sequnce number in host side will probably not reset to zero. Then in next time video stream start, the device will expect host starts packet