Hi Sakari, On Tuesday 01 October 2013 02:08:47 Sakari Ailus wrote: > On Fri, Sep 20, 2013 at 11:08:47PM +0200, Laurent Pinchart wrote: > > On Thursday 19 September 2013 01:01:05 Sakari Ailus wrote: > > > Pads that set this flag must be connected by an active link for the > > > entity > > > to stream. > > > > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> > > > Acked-by: Sylwester Nawrocki <sylvester.nawrocki@xxxxxxxxx> > > > --- > > > > > > Documentation/DocBook/media/v4l/media-ioc-enum-links.xml | 8 +++++++ > > > include/uapi/linux/media.h | 1 + > > > 2 files changed, 9 insertions(+) > > > > > > diff --git a/Documentation/DocBook/media/v4l/media-ioc-enum-links.xml > > > b/Documentation/DocBook/media/v4l/media-ioc-enum-links.xml index > > > 355df43..59b212a 100644 > > > --- a/Documentation/DocBook/media/v4l/media-ioc-enum-links.xml > > > +++ b/Documentation/DocBook/media/v4l/media-ioc-enum-links.xml > > > @@ -134,6 +134,14 @@ > > > <entry>Output pad, relative to the entity. Output pads source > > > data and are origins of links.</entry> > > > </row> > > > + <row> > > > + <entry><constant>MEDIA_PAD_FL_MUST_CONNECT</constant></entry> > > > + <entry>A pad must be connected with an enabled link for the > > > > s/A pad/The pad/ ? > > Fixed. > > > > + entity to be able to stream. There could be temporary reasons > > > + (e.g. device configuration dependent) for the pad to need > > > + connecting; the absence of the flag won't say there > > > + may not be any.</entry> > > > > I believe the description doesn't make it very explicit that a > > MUST_CONNECT pad with no existing link is valid, as opposed to existing > > links with no enabled link, which would be invalid. Do you think we should > > fix that ? > > Yes. I propose to add this: "The flag has no effect on pads without > connected links." What about If the pad is linked to any other pad, at least one of the links must be enabled for the entity to be able to stream. There could be temporary reasons (e.g. device configuration dependent) for the pad to need enabled links; the absence of the flag doesn't imply there is none. The flag has no effect on pads without connected links. > > > + </row> > > > </tbody> > > > </tgroup> > > > </table> -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
