Re: [PATCH v3 00/31] v4l: add support for multiplexed streams

From: Sakari Ailus <sakari.ailus@linux.intel.com>
To: Jacopo Mondi <jacopo@jmondi.org>
Cc: niklas soderlund <niklas.soderlund@ragnatech.se>,
	Jacopo Mondi <jacopo+renesas@jmondi.org>,
	laurent.pinchart@ideasonboard.com,
	niklas.soderlund+renesas@ragnatech.se,
	linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH v3 00/31] v4l: add support for multiplexed streams
Date: Fri, 8 Mar 2019 16:12:29 +0200	[thread overview]
Message-ID: <20190308141229.qflsxk2lhhdohn6a@paasikivi.fi.intel.com> (raw)
In-Reply-To: <20190308131903.daja4zzfp47pucsf@uno.localdomain>

On Fri, Mar 08, 2019 at 02:19:03PM +0100, Jacopo Mondi wrote:
> Hi Sakari and Niklas,
> 
> On Thu, Mar 07, 2019 at 11:47:26AM +0200, Sakari Ailus wrote:
> > Hi Jacopo,
> >
> > On Tue, Mar 05, 2019 at 07:51:19PM +0100, Jacopo Mondi wrote:
> > > Hello,
> > >    third version of multiplexed stream support patch series.
> > >
> > > V2 sent by Niklas is available at:
> > > https://patchwork.kernel.org/cover/10573817/
> > >
> > > As per v2, most of the core patches are work from Sakari and Laurent, with
> > > Niklas' support on top for adv748x and rcar-csi2.
> > >
> > > The use case of the series remains the same: support for virtual channel
> > > selection implemented on R-Car Gen3 and adv748x. Quoting the v2 cover letter:
> > >
> > > -------------------------------------------------------------------------------
> > > I have added driver support for the devices used on the Renesas Gen3
> > > platforms, a ADV7482 connected to the R-Car CSI-2 receiver. With these
> > > changes I can control which of the analog inputs of the ADV7482 the
> > > video source is captured from and on which CSI-2 virtual channel the
> > > video is transmitted on to the R-Car CSI-2 receiver.
> > >
> > > The series adds two new subdev IOCTLs [GS]_ROUTING which allows
> > > user-space to get and set routes inside a subdevice. I have added RFC
> > > support for these to v4l-utils [2] which can be used to test this
> > > series, example:
> > >
> > >     Check the internal routing of the adv748x csi-2 transmitter:
> > >     v4l2-ctl -d /dev/v4l-subdev24 --get-routing
> > >     0/0 -> 1/0 [ENABLED]
> > >     0/0 -> 1/1 []
> > >     0/0 -> 1/2 []
> > >     0/0 -> 1/3 []
> > >
> > >
> > >     Select that video should be outputed on VC 2 and check the result:
> > >     $ v4l2-ctl -d /dev/v4l-subdev24 --set-routing '0/0 -> 1/2 [1]'
> > >
> > >     $ v4l2-ctl -d /dev/v4l-subdev24 --get-routing
> > >     0/0 -> 1/0 []
> > >     0/0 -> 1/1 []
> > >     0/0 -> 1/2 [ENABLED]
> > >     0/0 -> 1/3 []
> > > -------------------------------------------------------------------------------
> > >
> > > Below is reported the media graph of the system used for testing [1].
> > >
> > > v4l2-ctl patches to handle the newly introduced IOCTLs are available from
> > > Niklas' repository at:
> > > git://git.ragnatech.se/v4l-utils routing
> >
> > Could you send the v4l2-ctl patches out as well, please?
> >
> 
> Niklas sent them on late 2017... time flies :)
> https://patchwork.kernel.org/patch/10113189/
> 
> Would you like to have them re-sent?

Oh, well, sure, if there are no changes, no need to resend. Thanks for the
pointer.

-- 
Sakari Ailus
sakari.ailus@linux.intel.com