Hi Kieran,
Thank you for the patch.
On Thursday, 14 December 2017 01:10:10 EET Kieran Bingham wrote:
> From: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
>
> The videodevice module defines a stream_off function call, but this is
> not available in the python bindings interface.
>
> Provide the binding of VideoStreamer::stream_off.
>
> Signed-off-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
> ---
> py/pykms/pyvid.cpp | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/py/pykms/pyvid.cpp b/py/pykms/pyvid.cpp
> index 6a6080e4b2b7..92006c404038 100644
> --- a/py/pykms/pyvid.cpp
> +++ b/py/pykms/pyvid.cpp
> @@ -34,5 +34,6 @@ void init_pyvid(py::module &m)
> .def("queue", &VideoStreamer::queue)
> .def("dequeue", &VideoStreamer::dequeue)
> .def("stream_on", &VideoStreamer::stream_on)
> + .def("stream_off", &VideoStreamer::stream_off)
I've always wondered why V4L2 has two ioctls for stream on and stream off,
when a single one with an on/off parameter would be perfectly fine. I'm thus
tempted to implement stream control through a single function, but that's a
matter of taste. Feel free to give it a go if you want, otherwise
Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ;
> }
--
Regards,
Laurent Pinchart
