Re: [PATCH v2] media: mali-c55: Require an enabled link to start a video node

[Linus Walleij]

On Fri, Nov 6, 2026 at 12:16 AM David Carlier <devnexen@xxxxxxxxx> wrote:

> mali_c55_pipeline_ready() skips queues whose link is disabled, but such
> a queue can still start streaming on its own, as the video node pads
> lack MEDIA_PAD_FL_MUST_CONNECT. Stopping it later sees a ready pipeline
> and disables the ISP under the connected queues still capturing.
>
> Mark the video node pads MUST_CONNECT so that starting a disconnected
> node fails with -ENOLINK.
>
> Fixes: d5f281f3dd29 ("media: mali-c55: Add Mali-C55 ISP driver")
> Signed-off-by: David Carlier <devnexen@xxxxxxxxx>

It checks out:
Reviewed-by: Linus Walleij <linusw@xxxxxxxxxx>

> Changes in v2:
> - Add missing Signed-off-by.
> - Cc stable, as d5f281f3dd29 is in the stable tree.

No Cc: stable@xxxxxxxxxxxxxxx actually added?

But Jacopo can certainly add that when applying the patch.

Yours,
Linus Walleij