Moi, On Fri, Apr 23, 2021 at 03:37:03PM +0300, Tomi Valkeinen wrote: > On 20/04/2021 14:41, Sakari Ailus wrote: > > Hi Laurent, > > > > On Sun, Apr 18, 2021 at 09:06:11PM +0300, Laurent Pinchart wrote: > > > Hi Tomi and Sakari, > > > > > > Thank you for the patch. > > > > > > There's an extra "to" in the subject line. > > > > > > On Thu, Apr 15, 2021 at 04:04:37PM +0300, Tomi Valkeinen wrote: > > > > From: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > > > > > > > Links are validated along the pipeline which is about to start streaming. > > > > Not all the pads in entities that are traversed along that pipeline are > > > > part of the pipeline, however. Skip the link validation for such pads, > > > > and while at there rename "other_pad" to "local_pad" to convey the fact > > > > the route to be checked is internal to the entity. > > > > > > Both "pad" and "local_pad" are local. I would have kept the "other_pad" > > > > The pad that in the remote entity is not local. The other one could be > > called remote_pad though. > > I'm not sure what you mean here. Aren't both pad and local_pad pads of a > single entity here? If so, I think Laurent's comment makes sense, and > other_pad is a better name. I guess you could argue for either. I'm fine dropping the patch. -- Sakari Ailus
