Hi Sakari,
Thank you for the patch.
On Fri, Sep 20, 2024 at 10:35:02AM +0300, Sakari Ailus wrote:
> Document the expected {enable,disable}_streams callback behaviour for
> drivers that are stream-unaware i.e. don't specify the
> V4L2_SUBDEV_CAP_STREAMS sub-device capability flat. In this specific case,
s/flat/flag/
> the mask argument can be ignored.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> ---
> include/media/v4l2-subdev.h | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
> index 3cc6b4a5935f..ff63fb6046b1 100644
> --- a/include/media/v4l2-subdev.h
> +++ b/include/media/v4l2-subdev.h
> @@ -834,11 +834,19 @@ struct v4l2_subdev_state {
> * v4l2_subdev_init_finalize() at initialization time). Do not call
> * directly, use v4l2_subdev_enable_streams() instead.
> *
> + * Drivers that support only a single stream without setting the
> + * V4L2_SUBDEV_CAP_STREAMS sub-device capability flag can ignore the mask
> + * argument.
> + *
> * @disable_streams: Disable the streams defined in streams_mask on the given
> * source pad. Subdevs that implement this operation must use the active
> * state management provided by the subdev core (enabled through a call to
> * v4l2_subdev_init_finalize() at initialization time). Do not call
> * directly, use v4l2_subdev_disable_streams() instead.
> + *
> + * Drivers that support only a single stream without setting the
> + * V4L2_SUBDEV_CAP_STREAMS sub-device capability flag can ignore the mask
> + * argument.
> */
> struct v4l2_subdev_pad_ops {
> int (*enum_mbus_code)(struct v4l2_subdev *sd,
--
Regards,
Laurent Pinchart
