Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-29 Thread Laurent Pinchart
Hi Ramzi, On Mon, Jun 29, 2020 at 11:24:13AM +0200, Ramzi Ben Meftah wrote: > On Fri, Jun 26, 2020 at 04:12:51AM +0300, Laurent Pinchart wrote: > > On Thu, Jun 25, 2020 at 10:41:09AM -0700, Steve Longerbeam wrote: > >> On 6/24/20 7:01 PM, Laurent Pinchart wrote: > >>> On Wed, Jun 24, 2020 at 09:53

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-29 Thread Ramzi Ben Meftah
Hi Laurent, On Fri, Jun 26, 2020 at 04:12:51AM +0300, Laurent Pinchart wrote: > Hi Steve, > > On Thu, Jun 25, 2020 at 10:41:09AM -0700, Steve Longerbeam wrote: > > On 6/24/20 7:01 PM, Laurent Pinchart wrote: > > > On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: > > >> On Tue, Jun 16

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-26 Thread Ramzi Ben Meftah
On Fri, Jun 26, 2020 at 11:25:31AM +0200, Hans Verkuil wrote: > On 26/06/2020 11:09, Ramzi Ben Meftah wrote: > > Hi Laurent, Hans, > > > > On Thu, Jun 25, 2020 at 12:35:02PM +0200, Hans Verkuil wrote: > >> On 25/06/2020 12:29, Laurent Pinchart wrote: > >>> Hi Ramzi, > >>> > >>> On Thu, Jun 25, 202

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-26 Thread Hans Verkuil
On 26/06/2020 11:09, Ramzi Ben Meftah wrote: > Hi Laurent, Hans, > > On Thu, Jun 25, 2020 at 12:35:02PM +0200, Hans Verkuil wrote: >> On 25/06/2020 12:29, Laurent Pinchart wrote: >>> Hi Ramzi, >>> >>> On Thu, Jun 25, 2020 at 12:18:35PM +0200, Ramzi Ben Meftah wrote: On Thu, Jun 25, 2020 at 12

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-26 Thread Ramzi Ben Meftah
On Fri, Jun 26, 2020 at 11:09:13AM +0200, Ramzi Ben Meftah wrote: > Hi Laurent, Hans, > > On Thu, Jun 25, 2020 at 12:35:02PM +0200, Hans Verkuil wrote: > > On 25/06/2020 12:29, Laurent Pinchart wrote: > > > Hi Ramzi, > > > > > > On Thu, Jun 25, 2020 at 12:18:35PM +0200, Ramzi Ben Meftah wrote: >

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-26 Thread Ramzi Ben Meftah
Hi Laurent, Hans, On Thu, Jun 25, 2020 at 12:35:02PM +0200, Hans Verkuil wrote: > On 25/06/2020 12:29, Laurent Pinchart wrote: > > Hi Ramzi, > > > > On Thu, Jun 25, 2020 at 12:18:35PM +0200, Ramzi Ben Meftah wrote: > >> On Thu, Jun 25, 2020 at 12:47:24PM +0300, Laurent Pinchart wrote: > >>> On Th

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Laurent Pinchart
Hi Steve, On Thu, Jun 25, 2020 at 10:41:09AM -0700, Steve Longerbeam wrote: > On 6/24/20 7:01 PM, Laurent Pinchart wrote: > > On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: > >> On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: > >>> From: Steve Longerbeam > >> +N

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Steve Longerbeam
Hi Laurent, all, On 6/24/20 7:01 PM, Laurent Pinchart wrote: Hi Jacopo, On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: From: Steve Longerbeam +Niklas, +Laurent Niklas, Laurent, how does this play with CAP_IO

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Hans Verkuil
On 25/06/2020 12:29, Laurent Pinchart wrote: > Hi Ramzi, > > On Thu, Jun 25, 2020 at 12:18:35PM +0200, Ramzi Ben Meftah wrote: >> On Thu, Jun 25, 2020 at 12:47:24PM +0300, Laurent Pinchart wrote: >>> On Thu, Jun 25, 2020 at 11:30:46AM +0200, Ramzi Ben Meftah wrote: On Thu, Jun 25, 2020 at 05:

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Laurent Pinchart
Hi Ramzi, On Thu, Jun 25, 2020 at 12:18:35PM +0200, Ramzi Ben Meftah wrote: > On Thu, Jun 25, 2020 at 12:47:24PM +0300, Laurent Pinchart wrote: > > On Thu, Jun 25, 2020 at 11:30:46AM +0200, Ramzi Ben Meftah wrote: > >> On Thu, Jun 25, 2020 at 05:01:38AM +0300, Laurent Pinchart wrote: > >>> On Wed,

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Ramzi Ben Meftah
Hi Laurent, On Thu, Jun 25, 2020 at 12:47:24PM +0300, Laurent Pinchart wrote: > Hi Ramzi, > > On Thu, Jun 25, 2020 at 11:30:46AM +0200, Ramzi Ben Meftah wrote: > > On Thu, Jun 25, 2020 at 05:01:38AM +0300, Laurent Pinchart wrote: > > > On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote:

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Laurent Pinchart
Hi Ramzi, On Thu, Jun 25, 2020 at 11:30:46AM +0200, Ramzi Ben Meftah wrote: > On Thu, Jun 25, 2020 at 05:01:38AM +0300, Laurent Pinchart wrote: > > On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: > >> On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: > >>> From: Steve

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-25 Thread Ramzi Ben Meftah
Hi Laurent, On Thu, Jun 25, 2020 at 05:01:38AM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: > > On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: > > > From: Steve Longerbeam > > > > +Niklas, +Laurent > > > > Nik

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-24 Thread Laurent Pinchart
Hi Jacopo, On Wed, Jun 24, 2020 at 09:53:07AM +0200, Jacopo Mondi wrote: > On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: > > From: Steve Longerbeam > > +Niklas, +Laurent > > Niklas, Laurent, how does this play with CAP_IO_MC ? I don't think it's related to CAP_IO_MC, but I

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-24 Thread Hans Verkuil
On 24/06/2020 12:08, Hans Verkuil wrote: > On 16/06/2020 12:00, Ramzi BEN MEFTAH wrote: >> From: Steve Longerbeam >> >> This commit enables VIDIOC_ENUMINPUT, VIDIOC_G_INPUT, and VIDIOC_S_INPUT >> ioctls for use via v4l2 subdevice node. >> >> This commit should probably not be pushed upstream, beca

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-24 Thread Hans Verkuil
On 16/06/2020 12:00, Ramzi BEN MEFTAH wrote: > From: Steve Longerbeam > > This commit enables VIDIOC_ENUMINPUT, VIDIOC_G_INPUT, and VIDIOC_S_INPUT > ioctls for use via v4l2 subdevice node. > > This commit should probably not be pushed upstream, because the (old) > idea of video inputs conflicts

Re: [PATCH 1/3] v4l2-subdev: Add subdev ioctl support for ENUM/GET/SET INPUT

2020-06-24 Thread Jacopo Mondi
Hello On Tue, Jun 16, 2020 at 12:00:15PM +0200, Ramzi BEN MEFTAH wrote: > From: Steve Longerbeam > +Niklas, +Laurent Niklas, Laurent, how does this play with CAP_IO_MC ? Thanks j > This commit enables VIDIOC_ENUMINPUT, VIDIOC_G_INPUT, and VIDIOC_S_INPUT > ioctls for use via v4l2 subdevice