Hi,
On 10/10/2022 19:53, Dafna Hirschfeld wrote:
On 03.10.2022 15:18, Tomi Valkeinen wrote:
From: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
Add two new subdev pad operations, .enable_streams() and
.disable_streams(), to allow control of individual streams per pad. This
is a superset of what the video .s_stream() operation implements.
To help with handling of backward compatibility, add two wrapper
functions around those operations, and require their usage in drivers.
Hi, what does it mean 'require their usage in drivers'?
Drivers suppose to call these helpers in the s_stream implementation?
The drivers should use v4l2_subdev_enable_streams() and
v4l2_subdev_disable_streams() when they want to enable/disable the
source subdevice (which is most likely in .s_stream or
.enable_streams/.disable_streams).
The "require" there means that drivers should never call
.enable_streams/.disable_streams directly, as mentioned in their help
texts, but instead they should always call the helpers.
Tomi
