media: mc: Expand MUST_CONNECT flag to always require an enabled link
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Sun, 14 Jan 2024 23:04:52 +0000 (01:04 +0200)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Mon, 5 Feb 2024 13:29:34 +0000 (14:29 +0100)
commitb3decc5ce7d778224d266423b542326ad469cb5f
tree3f52600fc61bb3aadb7f6a77b984609ef764bb34
parent9ec9109cf9f611e3ec9ed0355afcc7aae5e73176
media: mc: Expand MUST_CONNECT flag to always require an enabled link

The MEDIA_PAD_FL_MUST_CONNECT flag indicates that the pad requires an
enabled link to stream, but only if it has any link at all. This makes
little sense, as if a pad is part of a pipeline, there are very few use
cases for an active link to be mandatory only if links exist at all. A
review of in-tree drivers confirms they all need an enabled link for
pads marked with the MEDIA_PAD_FL_MUST_CONNECT flag.

Expand the scope of the flag by rejecting pads that have no links at
all. This requires modifying the pipeline build code to add those pads
to the pipeline.

Cc: stable@vger.kernel.org # 6.1
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Documentation/userspace-api/media/mediactl/media-types.rst
drivers/media/mc/mc-entity.c