From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1D5DFC07E9E for ; Mon, 12 Jul 2021 07:43:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id EE22F61186 for ; Mon, 12 Jul 2021 07:43:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346584AbhGLHqk (ORCPT ); Mon, 12 Jul 2021 03:46:40 -0400 Received: from relay11.mail.gandi.net ([217.70.178.231]:44279 "EHLO relay11.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1349637AbhGLHo1 (ORCPT ); Mon, 12 Jul 2021 03:44:27 -0400 Received: (Authenticated sender: jacopo@jmondi.org) by relay11.mail.gandi.net (Postfix) with ESMTPSA id B94CE100008; Mon, 12 Jul 2021 07:41:30 +0000 (UTC) Date: Mon, 12 Jul 2021 09:42:20 +0200 From: Jacopo Mondi To: Sakari Ailus Cc: Tomi Valkeinen , linux-media@vger.kernel.org, Jacopo Mondi , Laurent Pinchart , niklas.soderlund+renesas@ragnatech.se, Mauro Carvalho Chehab , Hans Verkuil , Pratyush Yadav , Lokesh Vutla , Michal Simek Subject: Re: [PATCH v7 08/27] media: entity: Add has_route entity operation Message-ID: <20210712074220.wwpullv252hdwpnv@uno.localdomain> References: <20210524104408.599645-1-tomi.valkeinen@ideasonboard.com> <20210524104408.599645-9-tomi.valkeinen@ideasonboard.com> <20210708124310.ievjt7effrkc6g6d@uno.localdomain> <20210711152626.GC3@paasikivi.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210711152626.GC3@paasikivi.fi.intel.com> Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Sakari, On Sun, Jul 11, 2021 at 06:26:26PM +0300, Sakari Ailus wrote: > On Thu, Jul 08, 2021 at 02:43:10PM +0200, Jacopo Mondi wrote: > > Hi Tomi, > > a small note > > > > On Mon, May 24, 2021 at 01:43:49PM +0300, Tomi Valkeinen wrote: > > > From: Laurent Pinchart > > > > > > The optional operation can be used by entities to report whether two > > > pads are internally connected. > > > > > > While at there, fix a Sphinx compiler warning in a comment block a few > > > lines above. > > > > > > Signed-off-by: Laurent Pinchart > > > Signed-off-by: Michal Simek > > > Signed-off-by: Sakari Ailus > > > Signed-off-by: Jacopo Mondi > > > Signed-off-by: Tomi Valkeinen > > > --- > > > include/media/media-entity.h | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > > > index 516d73a2941e..ad4020b2df65 100644 > > > --- a/include/media/media-entity.h > > > +++ b/include/media/media-entity.h > > > @@ -187,6 +187,7 @@ enum media_pad_signal_type { > > > * @flags: Pad flags, as defined in > > > * :ref:`include/uapi/linux/media.h ` > > > * (seek for ``MEDIA_PAD_FL_*``) > > > + * > > > * .. note:: > > > * > > > * @stream_count reference count must never be negative, but is a signed > > > @@ -214,6 +215,10 @@ struct media_pad { > > > * @link_validate: Return whether a link is valid from the entity point of > > > * view. The media_pipeline_start() function > > > * validates all links by calling this operation. Optional. > > > + * @has_route: Return whether a route exists inside the entity between > > > + * two given pads. Pads are passed to the operation ordered > > > + * by index. Optional: If the operation isn't implemented > > > > According to the next patch, this doesn't seem to be 'Optional:' :) > > How? Few drivers will implement this in the end, and that's been taken into > account in code AFAIU. What I meant is that it's not optional that all pads are considered connected if the op is not implemented. Or maybe this should have read as "Implementing the operation is optional" ? > > -- > Sakari Ailus